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

Added path projection class and logic #23

Merged
merged 2 commits into from
Oct 13, 2023

Conversation

gzaragoza314
Copy link
Collaborator

Added a class and logic for path projection:

  • Currently, assumes steering angle and velocity are constant for the curve it projects
  • Has functions for projecting analytically via integrated kinematics and projecting discretely by timesteps
    • Tested that both give nearly the same numbers for a reasonable input

@christianvluu
Copy link
Collaborator

Can you check if the outputs for these area in the correct coordinate frame? We're using UTM coordinates, zone 17T.

Looks like X = northing
Y = easting

If turning right, steering radius is positive
If turning left, steering radius is negative

Heading = 0 means north
Heading = 90 means east and so forth (but you might want to check the logic of this in engine.py for the simulator as that's our coordinate frame)

Copy link
Collaborator

@christianvluu christianvluu left a comment

Choose a reason for hiding this comment

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

Looks good - thanks!

@christianvluu
Copy link
Collaborator

normally gabe would click squash and merge but i'll do it since it's break.

@christianvluu christianvluu merged commit fd4c0a5 into main Oct 13, 2023
@christianvluu christianvluu deleted the gzaragoza314/path_projection branch October 13, 2023 17:53
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