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

Fix FOLLOW mode so it doesn't crash (NOT tuned yet) #11

Closed
wants to merge 2 commits into from

Conversation

bitshop
Copy link
Collaborator

@bitshop bitshop commented Oct 19, 2018

This will OVER accelerate / overshoot target speed - BE CAREFUL RUNNING THIS.

slwiki pushed a commit to slwiki/openpilot that referenced this pull request Dec 3, 2018
@BogGyver BogGyver closed this Jan 11, 2019
@BogGyver BogGyver deleted the tesla_sr branch January 24, 2019 04:32
SippieCup pushed a commit that referenced this pull request Aug 7, 2019
* fixed pathplanner

* fixed bias

* Remove multi-stacking of angle-offset

The existing pathplanner bug results in the previous desired angle being stacked on top of the previous desired angle before the actual angle has time to catch up.  Since the desired angle currently includes the angle offset and bias, these values get stacked onto the desired angle also.  This results in a tsunami wave of torque favoring the offset / bias.

* The same angle / bias should be applied to both

* This item was correct

* Reverted both changes to offset

* Fix timeout in longitudinal test (#772)

* Fix timeout in longitudinal test

* Update hyundaican for Correct Message ID on LKAS11 (#746)

This is the only trace of CF_Lkas_Icon found under /car/hyundai relative to open .dbc

commaai/opendbc#172

* Revert "Update hyundaican for Correct Message ID on LKAS11 (#746)" (#775)

This reverts commit 1f1893a.

* Correct Message ID on LKAS11 under Openpilot .dbc (#747)

commaai/opendbc#172

* Revert "Revert "Update hyundaican for Correct Message ID on LKAS11 (#746)" (#775)"

This reverts commit d5242c5.

* fixed cur_state.delta

* Revert "fixed cur_state.delta"

This reverts commit 227dccd3596594a8d7162a46c9fb771b9f6b6d75.

* fixed cur_state.delta

* added fast angle bias

* fixed merge conflict
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.

2 participants