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

Remove replanning #709

Closed
mjansen4857 opened this issue Sep 10, 2024 · 0 comments · Fixed by #714
Closed

Remove replanning #709

mjansen4857 opened this issue Sep 10, 2024 · 0 comments · Fixed by #714
Labels
enhancement New feature or request GUI Changes to the PathPlanner GUI PathPlannerLib Changes to PathPlannerLib

Comments

@mjansen4857
Copy link
Owner

mjansen4857 commented Sep 10, 2024

Replanning seems to be misused more often than it is actually helpful, so it should probably just be removed to the point of only re-generating a trajectory if the robot does not match the path's ideal starting state.

The current implementation adds a lot of potential footguns for the ability to join a path anywhere, which really isn't something the user should be wanting to do anyways. Any more complex on-the-fly path planning should be left up to the user instead of creating a general solution that doesn't work that great.

@mjansen4857 mjansen4857 added enhancement New feature or request GUI Changes to the PathPlanner GUI PathPlannerLib Changes to PathPlannerLib labels Sep 10, 2024
@mjansen4857 mjansen4857 added this to the 2025 milestone Sep 10, 2024
@mjansen4857 mjansen4857 changed the title Significantly simplify replanning Remove replanning Sep 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request GUI Changes to the PathPlanner GUI PathPlannerLib Changes to PathPlannerLib
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant