Skip to content

Conversation

@Patrick8894
Copy link
Collaborator

@Patrick8894 Patrick8894 commented May 15, 2025

Merged with s2025 locally before this PR
Main changes:
[current.yaml] Controller settings
[stanley.py] Stanley controller
[mpc.py] MPC controller
[fixed_route.yaml] Add different controller settings and routes
Some log_plots are added for plotting log file for each controller
Forward, reverse, mixed routes other than p shape for testing performance

@Patrick8894 Patrick8894 requested a review from krishauser as a code owner May 15, 2025 13:26
@Patrick8894 Patrick8894 changed the base branch from main to s2025 May 15, 2025 13:27
Copy link
Owner

@krishauser krishauser left a comment

Choose a reason for hiding this comment

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

Lots of files are considered completely rewritten, even though it looks like there is no change. I also don't know why creep control is another planner, and it doesn't look like it is ever triggered.

@Patrick8894 Patrick8894 requested a review from krishauser May 15, 2025 23:05
@sonarqubecloud
Copy link

Copy link
Owner

@krishauser krishauser left a comment

Choose a reason for hiding this comment

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

LGTM

@krishauser krishauser merged commit 0d65f21 into s2025 May 16, 2025
3 of 4 checks passed
@krishauser krishauser deleted the Control_Stanley branch May 16, 2025 18:47
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.

6 participants