Skip to content

Release Candidate 0.2.3#977

Merged
42 commits merged intodashing-develfrom
dashing-sync
Aug 1, 2019
Merged

Release Candidate 0.2.3#977
42 commits merged intodashing-develfrom
dashing-sync

Conversation

@ghost
Copy link

@ghost ghost commented Jul 24, 2019

This is the current 0.2.3 release candidate. This PR merges master onto the dashing-devel branch.

We expect a few more changes to be added to this PR as additional PRs get merged to master this week. The goal is to test this early next week and then merge to dashing-devel.

SteveMacenski and others added 30 commits June 28, 2019 16:06
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
SteveMacenski and others added 12 commits July 23, 2019 17:32
* I added a timeout to spin_until_future_complete, so rviz doesn't get frozen if something goes wrong in the navigation stack. I did some changes in the start navigation; - changed the funtion type to boolean - added returns to the funtion, so it will return false if something goes wrong. I changed goal_tool added a if statement to check if start_navigation doesn't return false before calling any other functions in goal_tool

* made the code cleaner and changed the error messages. Reduced the timeout value to 3 seconds. The timeouts will be parametrized in the future.

* added timeout parameter
Build tools - Run build_all.sh from any nested repo workspace & rename dependencies workspace for "nav" autocomplete
* I added a timeout to spin_until_future_complete, so rviz doesn't get frozen if something goes wrong in the navigation stack. I did some changes in the start navigation; - changed the funtion type to boolean - added returns to the funtion, so it will return false if something goes wrong. I changed goal_tool added a if statement to check if start_navigation doesn't return false before calling any other functions in goal_tool

* made the code cleaner and changed the error messages. Reduced the timeout value to 3 seconds. The timeouts will be parametrized in the future.

* added timeout parameter

* Update navigation_dialog.cpp

* Update navigation_dialog.hpp

* Update navigation_dialog.cpp

* Update navigation_dialog.hpp
* Fixing typo in dependencies

* Also remove some duplicate dependencies

* Fixing robot_state_publisher dependency
* Add back parameter to configure DWA rate

* Report out if we missed our control loop time.
All our builds have boost installed due to other dependencies, so
we've never run into this. The build farm must build with all other
dependencies removed somehow.
Signed-off-by: vinnamkim <vinnam.kim@gmail.com>
@ghost ghost changed the title [WIP] Release Candidate 0.2.3 Release Candidate 0.2.3 Jul 29, 2019
@ghost
Copy link
Author

ghost commented Aug 1, 2019

The general consensus is this is good enough to release. I'm merging the PR to dashing-devel

@SteveMacenski Can you please do a 0.2.3 release with the tip of dashing-devel when you get a chance?

@ghost ghost merged commit 05517fb into dashing-devel Aug 1, 2019
@ghost ghost deleted the dashing-sync branch August 1, 2019 18:20
@SteveMacenski
Copy link
Member

@crdelsey sure thing, it probably won't happen before we talk this afternoon though

@SteveMacenski
Copy link
Member

ros/rosdistro#21896

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants