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

Partial PV support #115

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

Conversation

icolwell-as
Copy link
Collaborator

This PR resolves #112 by adding partial support for Path Vehicles.
By "partial support" I mean everything in the official geoscenario spec is implemented except for:

  • agentacceleration and timetoacceleration tags for path nodes.
  • cycles and usespeedprofile tags for vehicles. I don't think the existing TV implementation supported these vehicle tags either.

This PR contains the following changes:

  • Add a new PV class of type Vehicle
  • Update the Actor class to include a new follow_path function similar to the existing follow_trajectory function
  • The new follow_path function uses the frenet distance s to travel along the defined path, which also allows the PV to provide some data in frenet frame as well
  • Updates the dashboard to be able to display frenet frame information without the rest of the debug data
  • Adds a brief scenario for testing PVs
  • Cleans up some broken btree references and deletes some old duplicated btrees

@rodrigoqueiroz rodrigoqueiroz self-requested a review December 20, 2023 01:07
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.

Add support for velocity path following
1 participant