0.2.4 sync - Ready for testing #1096
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
deleting robot package
* Add parameter to disable map server * Move parameter check to getParameters * Uncrustify * Don't mark static layer as current until we have a map * Make sure static layer gets initialized Make sure static layer gets initialized, even with an empty layer, so it doesn't crash if you disable the map service and enable the obstacle layer * remove map service from static layer Only use map topic instead Subscribe to map topic with default transient local volatility Add parameter "map_subscribe_transient_local" (default=true) for backwards compatibility with volatile publishers * Change AMCL to use only the map topic, not map service * remove now-unused MapServiceClient * uncrustify * only set current_=true when first map received * Fix nav2_costmap inflation tests. * Fixing collision tester test. * Revert "uncrustify" This reverts commit 82f88a7. This uncrustify failure is enforced differently depending on whether we are on the dashing branch or on the master branch. This particular commit is only appropriate on the dashing branch, but is already present there.
…aintainers removing old maintainers from navigation1
* change primitives to recoveries * changing all over new bringups with nav2_recoveries * renaming primitive_name -> recovery_name
…o_util Robot class to util
…or nav doesnt work
* generic ros service caller BT node * BT specialty service callers using BT node service * rename to reinitialize global localization * uncrustify/lint * changes per PR requests
…i-patch-1 Removing deprecation warning
ros-navigation#1074) * Move AMCL library files back to nav2_amcl package * Fixing up naming so things compile again * Transition copyright disable check to follow eig3 files. * Transition amcl files license to LGPL 2.1 * Adding missing copyright headers to nav2_bringup package * Adding missing copyright headers to nav2_common * Fixing up include guard to match coding style * Reformatting license to pass the copyright check The change here is only whitespace changes - line length and paragraph spacing, as well as the addition of the license title * Relicensing file to better match the existing package * Fixing readme to account for the new change
Adding a link to the upstream documentation on behaviortree.dev
* Add proper SPDX license IDs and repo LICENSE file * Fixing a minor linter typo
possibly responsible for heisenbug ros-navigation#1079 (comment)
changing pose topics to goal_pose
…conds (ros-navigation#1048)" This reverts commit 7c473cd.
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.
Updating dashing-devel branch to the master b629f55
Everything seems to be working as expected. Please test!