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.

Sign up for GitHub

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

Jump to bottom

Tssp prius torque control #24669

Merged
merged 5 commits into from
May 27, 2022
Merged

Tssp prius torque control #24669

merged 5 commits into from
May 27, 2022

Conversation

haraschax
Copy link
Contributor

No description provided.

@haraschax haraschax merged commit d708a13 into master May 27, 2022
@haraschax haraschax deleted the tssp_prius_torque_control branch May 27, 2022 17:21
krkeegan pushed a commit to krkeegan/openpilot that referenced this pull request May 27, 2022
@haraschax @krkeegan
Tssp prius torque control (commaai#24669)
cb91f7c 
* use llk

* use steering sensor at low speed stil

* Try more simple

* rm prius tune

* updated ref
cydia2020 added a commit to cydia2020/dodgypilot that referenced this pull request May 28, 2022
@cydia2020
Tssp prius torque control commaai#24669
9c3b385 
and TSS-P Corolla: use torque lateral control (commaai#24291)
spektor56 pushed a commit to spektor56/ghostpilot that referenced this pull request Jun 1, 2022
@haraschax @spektor56
Tssp prius torque control (commaai#24669)
6bf3289 
* use llk

* use steering sensor at low speed stil

* Try more simple

* rm prius tune

* updated ref
spektor56 pushed a commit to spektor56/ghostpilot that referenced this pull request Jun 2, 2022
@haraschax @spektor56
Tssp prius torque control (commaai#24669)
153d892 
* use llk

* use steering sensor at low speed stil

* Try more simple

* rm prius tune

* updated ref
krkeegan added a commit to krkeegan/openpilot that referenced this pull request Jun 6, 2022
@krkeegan
↔️ Use Torque for Lateral Controller
e4e2f93 
Pulled commit from future Comma work.

The KIA commit was pulled just to make my life easier.

commit 0eae5b3
Author: Shane Smiskol <shane@smiskol.com>
Date:   Thu Jun 2 01:06:14 2022 -0700

    Move some Toyotas to the torque tune for testing

commit cb91f7c
Author: HaraldSchafer <harald.the.engineer@gmail.com>
Date:   Fri May 27 10:21:02 2022 -0700

    Tssp prius torque control (commaai#24669)

    * use llk

    * use steering sensor at low speed stil

    * Try more simple

    * rm prius tune

    * updated ref

commit baaa47f
Author: HaraldSchafer <harald.the.engineer@gmail.com>
Date:   Tue May 24 17:52:33 2022 -0700

    Rerevert torque control (commaai#24649)

    * Revert "Revert torque control (commaai#24565)"

    This reverts commit 9f8b037.

    * Move tune out of car specific stuff

    * Update ref commit

commit 7c69036
Author: Adeeb Shihadeh <adeebshihadeh@gmail.com>
Date:   Mon May 23 19:16:17 2022 -0700

    Kia EV6 (commaai#24485)

    * ev6 squash

    * test route

    * cleanup

    * remove those

    * resume spam

    * setme

    * more reliable button pressing

    * new steer signals

    * for now

    * update docs
krkeegan added a commit to krkeegan/openpilot that referenced this pull request Jun 6, 2022
@krkeegan
↔️ Use Torque for Lateral Controller
25127bb 
Pulled commit from future Comma work.

The KIA commit was pulled just to make my life easier.

commit 0eae5b3
Author: Shane Smiskol <shane@smiskol.com>
Date:   Thu Jun 2 01:06:14 2022 -0700

    Move some Toyotas to the torque tune for testing

commit cb91f7c
Author: HaraldSchafer <harald.the.engineer@gmail.com>
Date:   Fri May 27 10:21:02 2022 -0700

    Tssp prius torque control (commaai#24669)

    * use llk

    * use steering sensor at low speed stil

    * Try more simple

    * rm prius tune

    * updated ref

commit baaa47f
Author: HaraldSchafer <harald.the.engineer@gmail.com>
Date:   Tue May 24 17:52:33 2022 -0700

    Rerevert torque control (commaai#24649)

    * Revert "Revert torque control (commaai#24565)"

    This reverts commit 9f8b037.

    * Move tune out of car specific stuff

    * Update ref commit

commit 7c69036
Author: Adeeb Shihadeh <adeebshihadeh@gmail.com>
Date:   Mon May 23 19:16:17 2022 -0700

    Kia EV6 (commaai#24485)

    * ev6 squash

    * test route

    * cleanup

    * remove those

    * resume spam

    * setme

    * more reliable button pressing

    * new steer signals

    * for now

    * update docs
rjsmith1999 pushed a commit to rjsmith1999/openpilot that referenced this pull request Jun 11, 2022
@haraschax @rjsmith1999
Tssp prius torque control (commaai#24669)
a9ab2d7 
* use llk

* use steering sensor at low speed stil

* Try more simple

* rm prius tune

* updated ref

(cherry picked from commit d708a13)
bensonkwek pushed a commit to bensonkwek/openpilot that referenced this pull request Jun 11, 2022
@krkeegan @bensonkwek
↔️ Use Torque for Lateral Controller
7751ea7 
Pulled commit from future Comma work.

The KIA commit was pulled just to make my life easier.

commit 0eae5b3
Author: Shane Smiskol <shane@smiskol.com>
Date:   Thu Jun 2 01:06:14 2022 -0700

    Move some Toyotas to the torque tune for testing

commit cb91f7c
Author: HaraldSchafer <harald.the.engineer@gmail.com>
Date:   Fri May 27 10:21:02 2022 -0700

    Tssp prius torque control (commaai#24669)

    * use llk

    * use steering sensor at low speed stil

    * Try more simple

    * rm prius tune

    * updated ref

commit baaa47f
Author: HaraldSchafer <harald.the.engineer@gmail.com>
Date:   Tue May 24 17:52:33 2022 -0700

    Rerevert torque control (commaai#24649)

    * Revert "Revert torque control (commaai#24565)"

    This reverts commit 9f8b037.

    * Move tune out of car specific stuff

    * Update ref commit

commit 7c69036
Author: Adeeb Shihadeh <adeebshihadeh@gmail.com>
Date:   Mon May 23 19:16:17 2022 -0700

    Kia EV6 (commaai#24485)

    * ev6 squash

    * test route

    * cleanup

    * remove those

    * resume spam

    * setme

    * more reliable button pressing

    * new steer signals

    * for now

    * update docs
martinl pushed a commit to martinl/openpilot that referenced this pull request Jun 12, 2022
@haraschax @martinl
Tssp prius torque control (commaai#24669)
19ab336 
* use llk

* use steering sensor at low speed stil

* Try more simple

* rm prius tune

* updated ref
ntegan1 pushed a commit to ntegan1/apilot that referenced this pull request Jun 16, 2022
@haraschax @ntegan1
Tssp prius torque control (commaai#24669)
6e39247 
* use llk

* use steering sensor at low speed stil

* Try more simple

* rm prius tune

* updated ref
rjsmith1999 pushed a commit to rjsmith1999/openpilot that referenced this pull request Jun 16, 2022
@haraschax @rjsmith1999
Tssp prius torque control (commaai#24669)
b70770c 
* use llk

* use steering sensor at low speed stil

* Try more simple

* rm prius tune

* updated ref

(cherry picked from commit d708a13)
(cherry picked from commit a9ab2d7)
rjsmith1999 pushed a commit to rjsmith1999/openpilot that referenced this pull request Jun 16, 2022
@haraschax @rjsmith1999
Tssp prius torque control (commaai#24669)
e19f9c9 
* use llk

* use steering sensor at low speed stil

* Try more simple

* rm prius tune

* updated ref

(cherry picked from commit d708a13)
(cherry picked from commit a9ab2d7)
rjsmith1999 pushed a commit to rjsmith1999/openpilot that referenced this pull request Jun 17, 2022
@haraschax @rjsmith1999
Tssp prius torque control (commaai#24669)
c7868a0 
* use llk

* use steering sensor at low speed stil

* Try more simple

* rm prius tune

* updated ref

(cherry picked from commit d708a13)
(cherry picked from commit a9ab2d7)
rjsmith1999 pushed a commit to rjsmith1999/openpilot that referenced this pull request Jun 17, 2022
@haraschax @rjsmith1999
Tssp prius torque control (commaai#24669)
4971b25 
* use llk

* use steering sensor at low speed stil

* Try more simple

* rm prius tune

* updated ref

(cherry picked from commit d708a13)
(cherry picked from commit a9ab2d7)
rjsmith1999 pushed a commit to rjsmith1999/openpilot that referenced this pull request Jun 27, 2022
@haraschax @rjsmith1999
Tssp prius torque control (commaai#24669)
cb82478 
* use llk

* use steering sensor at low speed stil

* Try more simple

* rm prius tune

* updated ref
rjsmith1999 added a commit to rjsmith1999/openpilot that referenced this pull request Jun 27, 2022
@rjsmith1999
[controls] Squashed commit of the following:
d95a6a9 
commit d066702
Author: ClockeNessMnstr <locke.dftc@gmail.com>
Date:   Tue Jun 7 12:41:03 2022 -0400

    update DH names + notes for MPC output curvatures (commaai#24701)

    * update names + notes for MPC outputs

    "current_curvature" is not the correct description of what the MPC is outputting in it's curvature_ego state.
    The MPC is integrating it's free variable, curvature_rate, such that curvature[0] is actually the desired_curvature before any delay.
    inversely: the curvature_rate_desired is the desired rate of change to the setpoint and not the actual curvature rate.

    If we were to set the initial curvature = measured curvature in the MPC initiation these names would be correct.
    This was possibly how it was initially set up but the nomenclature here is now confusing.

    * more notes

    * match

    * Clarify commaai#1

commit cb82478
Author: HaraldSchafer <harald.the.engineer@gmail.com>
Date:   Fri May 27 10:21:02 2022 -0700

    Tssp prius torque control (commaai#24669)

    * use llk

    * use steering sensor at low speed stil

    * Try more simple

    * rm prius tune

    * updated ref

commit f1f7a0b
Author: HaraldSchafer <harald.the.engineer@gmail.com>
Date:   Tue May 24 17:52:33 2022 -0700

    Rerevert torque control (commaai#24649)

    * Revert "Revert torque control (commaai#24565)"

    This reverts commit 9f8b037.

    * Move tune out of car specific stuff

    * Update ref commit

commit 041c255
Author: HaraldSchafer <harald.the.engineer@gmail.com>
Date:   Tue May 17 19:00:57 2022 -0700

    Revert torque control (commaai#24565)

    * torque reversal start

    * Fix carmodel tests

    * Update ref

    * update ref

    * Elif is better than if

commit 14f5dab
Author: HaraldSchafer <harald.the.engineer@gmail.com>
Date:   Fri May 20 13:50:47 2022 -0700

    Latcontrol torque: integrator need not be reset (commaai#24606)

    slow integrators need not be reset

commit 9221f3f
Author: ClockeNessMnstr <locke.dftc@gmail.com>
Date:   Fri May 13 19:52:20 2022 -0400

    LatControlTorque: clean up class variable (commaai#24526)

    * move to super

    * no class variable

    * there's CP

    * whitespace

    * drop CI from latcontrol super

    * Revert "drop CI from latcontrol super"

    This reverts commit 9218273.

    Co-authored-by: Shane Smiskol <shane@smiskol.com>

commit efb4dbd
Author: HaraldSchafer <harald.the.engineer@gmail.com>
Date:   Fri May 6 22:17:21 2022 -0700

    Latcontrol torque: fix integrator induced ping pong (commaai#24458)

    * Latcontrol torque: fix integrator induced ping pong

    * Reset on disengage since unwind resets anywayh

    * Might be overkill

    * rm whitespace

    * update ref

commit d105ae2
Author: HaraldSchafer <harald.the.engineer@gmail.com>
Date:   Mon May 2 14:42:18 2022 -0700

    Latcontrol torque: update tuning (commaai#24357)

    * Little more chill

    * Update ref

    * Update refs

commit 0ce2b58
Author: Shane Smiskol <shane@smiskol.com>
Date:   Thu Apr 28 00:42:52 2022 -0700

    Clean up controllers (commaai#24340)

    * clean up lat controllers

    * pass CP once

    * sort

commit 47b162c
Author: Shane Smiskol <shane@smiskol.com>
Date:   Wed Apr 27 01:18:05 2022 -0700

    LatControlTorque: fix deadzone and missing steer saturated warning (commaai#24294)

    * fix steer saturated alert and deadzone

    * fix and formatting

    * update refs

    * update refs

    * not needed

    * remove

commit 51a7ef2
Author: HaraldSchafer <harald.the.engineer@gmail.com>
Date:   Wed Apr 20 11:26:24 2022 -0700

    Latcontrol torque: max_torque rename (commaai#24265)

commit 33e00f6
Author: HaraldSchafer <harald.the.engineer@gmail.com>
Date:   Tue Apr 19 19:34:31 2022 -0700

    Lateral torque-based control with roll on TSS2 corolla and TSSP rav4 (commaai#24260)

    * Initial commit

    * Fix bugs

    * Need more torque rate

    * Cleanup cray cray control

    * Write nicely

    * Chiiil

    * Not relevant for cray cray control

    * Do some logging

    * Seems like it has more torque than I thought

    * Bit more feedforward

    * Tune change

    * Retune

    * Retune

    * Little more chill

    * Add coroll

    * Add corolla

    * Give craycray a good name

    * Update to proper logging

    * D to the PI

    * Should be in radians

    * Add d

    * Start oscillations

    * Add D term

    * Only change torque rate limits for new tune

    * Add d logging

    * Should be enough

    * Wrong sign in D

    * Downtune a little

    * Needed to prevent faults

    * Add lqr rav4 to tune

    * Try derivative again

    * Data based retune

    * Data based retune

    * add friction compensation

    * Doesnt need too much P with friction comp

    * remove lqr

    * Remove kd

    * Fix tests

    * fix tests

    * Too much error

    * Get roll induced error under 1cm/deg

    * Too much jitter

    * Do roll comp

    * Add ki

    * Final update

    * Update refs

    * Cleanup latcontrol_torque a little more
ntegan1 pushed a commit to ntegan1/apilot that referenced this pull request Jul 12, 2022
@haraschax @ntegan1
Tssp prius torque control (commaai#24669)
6f857aa 
* use llk

* use steering sensor at low speed stil

* Try more simple

* rm prius tune

* updated ref
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@haraschax