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

adjusted dwb_local_planner_params #43

Closed
wants to merge 1 commit into from

Conversation

niniemann
Copy link
Contributor

  • use split_path option to enforce following complex paths (see here)
  • more trajectory samples over a smaller simulated time. This fixes a
    problem where the robot would stop too far away from the goal, as all
    possible trajectories either overshot the goal, or were too short to
    reach into the next gridcell of the critics.
  • removed Oscillation and PathAlign critics during the debugging process. Did not put them back in as they don't seem to be necessary.

- use split_path option to enforce following complex paths
- more trajectory samples over a smaller simulated time. This fixes a
  problem where the robot would stop too far away from the goal, as all
  possible trajectories either overshot the goal, or were too short to
  reach into the next gridcell of the critics.
mintar pushed a commit that referenced this pull request Jan 13, 2020
- use split_path option to enforce following complex paths
- more trajectory samples over a smaller simulated time. This fixes a
  problem where the robot would stop too far away from the goal, as all
  possible trajectories either overshot the goal, or were too short to
  reach into the next gridcell of the critics.
- remove Oscillation critic (never helped)
@mintar
Copy link
Member

mintar commented Jan 13, 2020

Thanks! I've put the PathAlign critic back in (without it, the robot veers more from side to side instead of keeping the nose on the path), changed some comments and merged via bc6c4ff .

@mintar mintar closed this Jan 13, 2020
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.

None yet

2 participants