Add #include <vector> for vector<> #1946
Merged
Conversation
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
To fix cpplint
SteveMacenski
approved these changes
Aug 14, 2020
SteveMacenski
added a commit
that referenced
this pull request
Aug 24, 2020
* waypoint_follower node has _rclcpp_node as well as base node (#1940) * Add #include <vector> for vector<> (#1946) To fix cpplint * Add 'angles' dependency to nav2_costmap_2d package.xml (#1947) * transform goal to costmap frame (#1949) The plan recieved is usually in global frame, but our local costmap is often in odom frame. This fixes a regression from #1857 * Add mutex lock into inflation layer to avoid thread issue in updating footprint (#1952) Signed-off-by: Daisuke Sato <[email protected]> * Fix being unable to change StandardTrajectoryGenerator parameter vtheta_samples (#1619) * Fix tests declaring parameters real nodes don't Signed-off-by: Shane Loretz <[email protected]> * Fix loadParameterWithDeprecation not getting initial parameter values Signed-off-by: Shane Loretz <[email protected]> * Create sim_time variable before using it Signed-off-by: Shane Loretz <[email protected]> * Line length < 100 Signed-off-by: Shane Loretz <[email protected]> * Add missing { Signed-off-by: Shane Loretz <[email protected]> * Linter fixes Signed-off-by: Shane Loretz <[email protected]> * sim_granularity -> time_granularity Signed-off-by: Shane Loretz <[email protected]> * Linter fix Signed-off-by: Shane Loretz <[email protected]> * update version to 0.4.3 * removing redundant dep on angles Co-authored-by: Ruffin <[email protected]> Co-authored-by: Sarthak Mittal <[email protected]> Co-authored-by: Michael Ferguson <[email protected]> Co-authored-by: Daisuke Sato <[email protected]> Co-authored-by: Shane Loretz <[email protected]>
ruffsl
added a commit
to ruffsl/navigation2
that referenced
this pull request
Jul 2, 2021
To fix cpplint
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.
To fix cpplint
See: https://app.circleci.com/pipelines/github/ros-planning/navigation2/3602/workflows/c6d8b916-c656-4f75-a89b-381d44b02903/jobs/14110