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

Add route reverse feature #784

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ycy-0510
Copy link

@ycy-0510 ycy-0510 commented Oct 2, 2024

There are two feature is this PR:

  1. add route reverse feature:
    user and build route for alliances

  2. and add auto save when clone and reverse:
    use don’t need to edit before close app

add route reverse feature: user and build route for alliances
and add auto save when clone and reverse: use don’t need to edit before close app
@github-actions github-actions bot added the GUI Changes to the PathPlanner GUI label Oct 2, 2024
@ycy-0510 ycy-0510 changed the title add route reverse feature and add auto save when clone and reverse add route reverse feature Oct 2, 2024
@ycy-0510 ycy-0510 changed the title add route reverse feature Add route reverse feature Oct 2, 2024
@mjansen4857
Copy link
Owner

So this looks like its for flipping paths to the other side of the field, which is not something that will be supported in the GUI. PathPlannerLib already takes care of flipping paths, which prevents the need to maintain a different path for each alliance.

@ycy-0510
Copy link
Author

ycy-0510 commented Oct 3, 2024

Some teams maybe need to flip and adjust some value. They can use this feature.

@mjansen4857
Copy link
Owner

In that case they can just duplicate the path and let PPLib flip it. I don't want to include any flipping logic in the GUI since it overcomplicates things like switching field images and I just wouldn't recommend people use it over the PPLib flipping anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GUI Changes to the PathPlanner GUI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants