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

Sw/update velocity #34

Merged
merged 14 commits into from
Nov 18, 2023
Merged

Sw/update velocity #34

merged 14 commits into from
Nov 18, 2023

Conversation

PatXue
Copy link
Collaborator

@PatXue PatXue commented Nov 16, 2023

Code for automatically changing velocity over time based on checkpoints on track

Basic framework of VelocityUpdater class, just applies constant acceleration to the buggy
Added velocity_updater node to launch file
Removed mpc_calculator node from launch file to remove terminal error message when running simulation
Added calculate_accel function to vary velocity over time (currently accelerates for 20s and then stops)
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 but can you also add function/class headers like the other files?

Copy link
Collaborator

@Jackack Jackack left a comment

Choose a reason for hiding this comment

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

Everything looks good but you need to pull the changes in main and merge them. Keep in mind that the topic names are remapped after the 2-buggy sim was merged into main. You will need to take the name of the buggy as a command line argument in your python file and prepend it to your topic paths. Consult the new 2d sim launch files for how to pass command line args via Roslaunch.

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 to me, just Jack's change request left.

@PatXue PatXue dismissed Jackack’s stale review November 18, 2023 22:00

Approved on discord

@Jackack Jackack merged commit ae98072 into main Nov 18, 2023
@Jackack Jackack deleted the sw/update_velocity branch November 18, 2023 22:48
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

3 participants