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

modified frenet optimal trajectory generator to consider the left and… #1055

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

Conversation

MaheshKumar92
Copy link

… right boundary. Plotting the same also.

Reference issue

What does this implement/fix?

Additional information

CheckList

  • Did you add an unittest for your new example or defect fix?
  • Did you add documents for your new example?
  • All CIs are green? (You can check it after submitting)

Copy link
Owner

@AtsushiSakai AtsushiSakai left a comment

Choose a reason for hiding this comment

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

@MaheshKumar92 Sorry for very late review.
Instead of adding a separate script, could you modify the original script to account for boundaries?
I believe this is a great additional implementation.

plt.grid(True)
plt.pause(0.0001)
# Save the current frame
plt.savefig(f'frames/frame_{frame_number:04d}.png')
Copy link
Owner

Choose a reason for hiding this comment

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

Please remove this line.

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