-
Notifications
You must be signed in to change notification settings - Fork 316
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unlatching topics #674
Unlatching topics #674
Conversation
…/astrobee into unlatching-topics
…/astrobee into unlatching-topics
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix the one comment that's out of date and then looks good.
@@ -98,16 +98,14 @@ void FreeFlyerNodelet::Setup(ros::NodeHandle & nh, ros::NodeHandle & nh_mt, std: | |||
// Immediately, setup a publisher for faults coming from this node | |||
// Topic needs to be latched for initialization faults |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
* Documentation update (#668) * docker docs update * make build instructions ROS1 * updating install instructions, hopefully making them easier to follow * updated from github PR reviews * fix typo * replace list with append (#671) * ci: bump to checkout@v3 (#673) * Unlatching topics (#674) * Changed heartbeat topic to be unlatched. * Changed echoed command to not latching. * no subscribers and info is obtained by service, so no latched topic needed * removing latched on frequently published topics * Made command topic to not atching. * no need for these to be latched * sub and pub in same node, no need to latch * no need to be latched no subs and not recorded, rviz should be opened when published * Made ground bridge topics not latched. * Fixed heartbeat topic latched comment. --------- Co-authored-by: Katie Browne <[email protected]> * support multiple versions for docs (#675) * Make coverage_analyzer.py support Python 3 (#680) Refactor to simplify and make output identical between Python 2 and 3 * Release 0.17.0 (#683) * add missing glog flag that caused problems in some systems (#685) * Make isort repeatable (#694) * make isort repeatable * isort now consistently groups astrobee imports as first party; one-time update to apply to old files * moving ff_names to ff_common, updating headers (#696) --------- Co-authored-by: Trey Smith <[email protected]> Co-authored-by: Katie Browne <[email protected]>
* Changed heartbeat topic to be unlatched. * Changed echoed command to not latching. * no subscribers and info is obtained by service, so no latched topic needed * removing latched on frequently published topics * Made command topic to not atching. * no need for these to be latched * sub and pub in same node, no need to latch * no need to be latched no subs and not recorded, rviz should be opened when published * Made ground bridge topics not latched. * Fixed heartbeat topic latched comment. --------- Co-authored-by: Katie Browne <[email protected]>
No description provided.