-
-
Notifications
You must be signed in to change notification settings - Fork 121
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
control_mpc: add MPC control law #333
control_mpc: add MPC control law #333
Conversation
edit: PR https://github.com/DangerKlippers/danger-klipper/pull/334 Now Open @dalegaard I am working on a doc [MPC.md] update from my own branch. It should be ready for PR this weekend. Let me know if you specifically have any areas which need to be added, changed or improved. Rough TODO: |
ed70bb1
to
0aac830
Compare
thanks @YanceyA. I added a bit of doc to the config reference. Let me know if you want me to correct or add anything else there |
0aac830
to
de482df
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Roger, a few small comments on the config. Otherwise it all looks great.
de482df
to
d9a1ea6
Compare
thanks for the suggestions |
d9a1ea6
to
27b0e74
Compare
27b0e74
to
069caa6
Compare
This adds MPC control law to main branch.
Has been extensively tested on
feature/mpc_experimental
branch but would like to get to main so we can get wider testing. Would appreciate final code review.Let me know if we need to add any extra docs.
Best regards,
Lasse