0.2.4 sync - Ready for testing#1096
Merged
222 commits merged intodashing-develfrom Sep 9, 2019
unknown repository
Merged
Conversation
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.
removing old maintainers from navigation1
* change primitives to recoveries * changing all over new bringups with nav2_recoveries * renaming primitive_name -> recovery_name
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
Removing deprecation warning
#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 #1079 (comment)
changing pose topics to goal_pose
This pull request was closed.
This file contains hidden or 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
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!