forked from ros2/ros1_bridge
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Combined branch #1
Merged
Merged
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
Signed-off-by: Jacob Perron <[email protected]>
… rosidl_adapter and .msg files (ros2#296) * update to use rosidl_parser and .idl files rather than rosidl_adapter and .msg files Signed-off-by: William Woodall <[email protected]> * fix quote style linting error Signed-off-by: William Woodall <[email protected]>
Signed-off-by: William Woodall <[email protected]>
* very minor typo in comments * Fix other copy-paste typos. Co-authored-by: Steven! Ragnarök <[email protected]> Co-authored-by: tomoya <[email protected]>
* Fix logging for updated rclcpp interface Signed-off-by: Michael Carroll <[email protected]> * Update src/bridge.cpp Signed-off-by: Michael Carroll <[email protected]> Co-authored-by: Ivan Santiago Paunovic <[email protected]> * Update to pass char * Signed-off-by: Michael Carroll <[email protected]> * Uncrustify Signed-off-by: Michael Carroll <[email protected]> Co-authored-by: Ivan Santiago Paunovic <[email protected]>
Signed-off-by: Chris Lalancette <[email protected]>
Signed-off-by: Christophe Bedard <[email protected]>
* init docker based CI Signed-off-by: Harsh Deshpande <[email protected]> * target rolling only Signed-off-by: Harsh Deshpande <[email protected]> * use setup-ros and action-ros-ci instead of custom scripts Signed-off-by: Harsh Deshpande <[email protected]> * quiet blind except warnings ros2#310 (comment) Signed-off-by: Harsh Deshpande <[email protected]> * remove ccache build times are not a concern and it is not significantly improved when using ros-tooling actions Signed-off-by: Harsh Deshpande <[email protected]>
Signed-off-by: Chen Lihui <[email protected]>
Signed-off-by: Chen Lihui <[email protected]>
…ubsciption Signed-off-by: Chen Lihui <[email protected]>
Signed-off-by: Chen Lihui <[email protected]>
and resotre the original main loop logic Signed-off-by: Chen Lihui <[email protected]>
to fix 'Not all nodes were finished before finishing the context' since ros2 PublisherBase passed into ros1 subscriber as parameter which might be freed after ros2 context. Signed-off-by: Chen Lihui <[email protected]>
Signed-off-by: Chen Lihui <[email protected]>
Signed-off-by: Chen Lihui <[email protected]>
Co-authored-by: Tomoya Fujita <[email protected]> Signed-off-by: Chen Lihui <[email protected]>
Signed-off-by: Chen Lihui <[email protected]>
See if that fixes the apt-update part of CI Signed-off-by: Shane Loretz <[email protected]>
Signed-off-by: Shane Loretz <[email protected]>
Signed-off-by: Christophe Bedard <[email protected]>
* Add example for using the parameter_bridge Signed-off-by: Loy van Beek <[email protected]> Signed-off-by: Chris Lalancette <[email protected]>
Signed-off-by: Loy van Beek <[email protected]>
Signed-off-by: Loy van Beek <[email protected]>
Signed-off-by: Loy van Beek <[email protected]>
Signed-off-by: Loy van Beek <[email protected]>
Signed-off-by: Loy van Beek <[email protected]>
Signed-off-by: Loy van Beek <[email protected]>
Signed-off-by: Loy van Beek <[email protected]>
Signed-off-by: Loy van Beek <[email protected]>
Signed-off-by: Loy van Beek <[email protected]>
…representation of liveliness enum values Signed-off-by: Loy van Beek <[email protected]>
Signed-off-by: Loy van Beek <[email protected]>
In 2 cases the formatting conflicts with what uncrustify wants; // NOLINT got rid of the complaints of cpplint, favouring uncrustify Signed-off-by: Loy van Beek <[email protected]>
Signed-off-by: Chris Lalancette <[email protected]>
Co-authored-by: Geoffrey Biggs <[email protected]> Signed-off-by: Loy van Beek <[email protected]>
Signed-off-by: Loy van Beek <[email protected]>
* Update package maintainers Signed-off-by: Geoffrey Biggs <[email protected]>
Signed-off-by: Servando German Serrano <[email protected]>
…thread-services' into combined_branch
…o combined_branch
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.
This PR updates the develop branch of our fork to include the following PRs
ros2#331
ros2#315
ros2#339