Skip to content
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

Option for ObstacleLayer to not overwrite StaticLayer's unknown parts #438

Conversation

RBT22
Copy link
Contributor

@RBT22 RBT22 commented Jun 20, 2023

The documentation for #3612

@RBT22 RBT22 changed the title Add enum options to Obstacle Layer params Option for ObstacleLayer to not overwrite StaticLayer's unknown parts Jun 20, 2023
@RBT22
Copy link
Contributor Author

RBT22 commented Jun 20, 2023

Need to add to migration guide on website that this new feature exists to folks can use it!

Where should I put the migration guide?

Need to add on configuration guide for costmap 2d the new setting so folks in the future can know its there

What should be this one? Isn't it enough to put info about it in the layer params docs?

Copy link
Member

@SteveMacenski SteveMacenski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We also need an entry in the migration guide announcing this change.

Make a new file iron.rst like https://github.com/ros-planning/navigation.ros.org/blob/master/migration/Humble.rst and start its first entry! This is the first big change for J-turtle

configuration/packages/costmap-plugins/obstacle.rst Outdated Show resolved Hide resolved
configuration/packages/costmap-plugins/voxel.rst Outdated Show resolved Hide resolved
@SteveMacenski SteveMacenski merged commit 892fab9 into ros-navigation:master Jun 21, 2023
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.

2 participants