forked from DFKI-NI/mir_robot
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add emergency halt publisher #19
Open
niemsoen
wants to merge
96
commits into
relffok:galactic-devel
Choose a base branch
from
niemsoen:galactic-devel
base: galactic-devel
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added headless launch file to mir_driver
added mir_bridge_ready publisher
…into galactic-devel
This reverts commit f31efb8.
…msoen/mir_robot into mir-bridge-as-service-server
Mir bridge as service server
…into galactic-devel
…gfix Launch mir_restapi_server in external driver
Co-authored-by: Sönke Niemann <[email protected]>
…robot into mir-restapi-sounds
…to mir-restapi-sounds
* add publisher for emergency halt * fix cyclic emergency halt checking interrupts rosbridge * improved terminal output Co-authored-by: Sönke Niemann <[email protected]>
It's not necessary to use the REST API to get the emergency status. The emergency status is already published as part of the RobotState message: mir_robot/mir_msgs/msg/RobotState.msg Line 12 in 89e4598
All we need to do is uncomment this line to get it published: mir_robot/mir_driver/mir_driver/mir_bridge.py Line 294 in 89e4598
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adding a publisher to the rosbridge, that publishes a message, should the emergency halt be triggered, so that this can be used as trigger in other ROS nodes.