Skip to content

fix dwb linear granularity default#900

Closed
EricoMeger wants to merge 1 commit intoros-navigation:masterfrom
EricoMeger:fix/dwb_linear_granularity_defaults
Closed

fix dwb linear granularity default#900
EricoMeger wants to merge 1 commit intoros-navigation:masterfrom
EricoMeger:fix/dwb_linear_granularity_defaults

Conversation

@EricoMeger
Copy link
Copy Markdown
Contributor

Basic Info

Info Please fill out this column
Ticket(s) this addresses (N/A)
Does this PR contain AI-generated software? (No)

Description of contribution in a few bullet points

Signed-off-by: EricoMeger <ericomeger9@gmail.com>
@EricoMeger EricoMeger force-pushed the fix/dwb_linear_granularity_defaults branch from 9b28f78 to 8a63e43 Compare April 10, 2026 18:54
Type Default
------ -------
double 0.5
double 0.05
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think I got kinda biased from my DWA experience, I should have double-checked to see if it was actually correct.

That said, this value seems a bit strange to me, I think this would be way too coarse for the trajectory samples. Can't really say its a typo though, since looking at the git history 0.5 is the default value since the initial implementation in 2018

I also don't think it would be right to change the config example, since it seems like DWB historically has been configured using 0.05 instead. See Humble, for example: https://github.com/ros-navigation/navigation2/blob/e9caa4287b99c17f4ff39fbf2315d4d82e78da93/nav2_bringup/params/nav2_params.yaml#L164

I'll be closing this one then, thanks for taking a look!

@EricoMeger EricoMeger closed this Apr 10, 2026
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