Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

Kart commander #46

Open
wants to merge 34 commits into
base: main
Choose a base branch
from
Open

Kart commander #46

wants to merge 34 commits into from

Conversation

Reschivon
Copy link
Contributor

@Reschivon Reschivon commented May 10, 2023

Brief Description

Kart Commander and Mux for Twist messages

Related Issues

Closes ...

Opens ...

Details

Tasks

  • Finished all sub-tasks (from issues)
  • Updated Documentation
  • Tests written/updated and implemented to CI

berliernj and others added 15 commits April 16, 2023 05:11
Added QoS profile and state transition rules
in hindsight it was simpler. If one mux fails both are likely screwed
See the AKS server for the state graph
It was using the rcs_service package before
No documentation for constants needed some trial n error
tests to be made automated later, see readme in tests
the new `switching_mux` let through one topic at a time
remove debug logger line
install-scripts to install_scripts
script_dir with an underscore
they were causing ci to fail
@Reschivon
Copy link
Contributor Author

@alanssitis approve this

@Reschivon Reschivon marked this pull request as ready for review May 11, 2023 00:58
@alanssitis alanssitis mentioned this pull request May 11, 2023
3 tasks
@alanssitis
Copy link
Contributor

Will test right now. Will add changes that may be needed in a review.

@Reschivon
Copy link
Contributor Author

sg

berliernj and others added 9 commits May 11, 2023 16:08
Added QoS profile and state transition rules
in hindsight it was simpler. If one mux fails both are likely screwed
See the AKS server for the state graph
It was using the rcs_service package before
No documentation for constants needed some trial n error
tests to be made automated later, see readme in tests
the new `switching_mux` let through one topic at a time
remove debug logger line
install-scripts to install_scripts
script_dir with an underscore
they were causing ci to fail
@alanssitis
Copy link
Contributor

Just rebased.

changed msgs package lost in rebase
@alanssitis
Copy link
Contributor

also, fill out package xml with your info and fill TODOs

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants