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

longitudinal personality: display in Toyota instrument cluster #31760

Merged
merged 27 commits into from
Mar 9, 2024

Conversation

sshane
Copy link
Contributor

@sshane sshane commented Mar 7, 2024

Starting with Toyota. Requires #31728. Part of #31735

@sshane sshane changed the title longitudinal personality: display in car instrument panel/HUD longitudinal personality: display in instrument panel Mar 7, 2024
@sshane sshane added the toyota label Mar 7, 2024
@sshane sshane linked an issue Mar 7, 2024 that may be closed by this pull request
@sshane sshane marked this pull request as draft March 7, 2024 10:35
@sshane sshane changed the title longitudinal personality: display in instrument panel longitudinal personality: display in instrument cluster Mar 7, 2024
@sshane
Copy link
Contributor Author

sshane commented Mar 7, 2024

https://youtu.be/_GAfTWFPov0

@sshane sshane changed the title longitudinal personality: display in instrument cluster longitudinal personality: display in Toyota instrument cluster Mar 8, 2024
@sshane sshane marked this pull request as ready for review March 8, 2024 03:37
incognitojam added a commit to incognitojam/openpilot that referenced this pull request Mar 8, 2024
Adjust the time gap to 2, 3 or 4 bars for openpilot long.

TODO: set AccTGap_D_Dsply when value changes so that the popup appears
with the new time gap

Depends on commaai#31760.
@sshane
Copy link
Contributor Author

sshane commented Mar 9, 2024

Right now CarController will press the distance button when we're trying to cancel without OP long. Checking CP.openpilotLongitudinalControl should catch everything.

Don't want to press button while canceling when no openpilotLongitudinalControl.

- Unsupported DSU:
  - Sending button: checking OP long works (no OP long, can't press buttons)

- DSU:
  - Sending button: checking OP long works (no OP long, can't press buttons)

- Smart DSU:
  - Sending button:
    - DSU car: checking SDSU works (always OP long)
    - NODSU car: checking OP long works (toggle, don't want to press while canceling)
    - TSS2 radar car: ^^^

- Unplugged DSU:
  - Sending button: OP works (no buttons but can still press button)

- TSS2:
  - Sending button:
    - Camera-ACC: checking OP long works (always OP long)
    - Radar-ACC: checking OP long works (no buttons but can still press button)

@sshane sshane force-pushed the disp-personality branch from 8ccc233 to d4ad7c3 Compare March 9, 2024 07:16
@sshane sshane merged commit 2c353a2 into master Mar 9, 2024
27 of 28 checks passed
@sshane sshane deleted the disp-personality branch March 9, 2024 07:26
incognitojam added a commit to incognitojam/openpilot that referenced this pull request Mar 11, 2024
Adjust the time gap to 2, 3 or 4 bars for openpilot long.

TODO: set AccTGap_D_Dsply when value changes so that the popup appears
with the new time gap

Depends on commaai#31760.
sshane pushed a commit that referenced this pull request Mar 12, 2024
* Ford: show longitudinal personality in IPC

Adjust the time gap to 2, 3 or 4 bars for openpilot long.

TODO: set AccTGap_D_Dsply when value changes so that the popup appears
with the new time gap

Depends on #31760.

* ACC UI: show time gap popup when distance changes

* Revert "ACC UI: show time gap popup when distance changes"

This reverts commit c4e8e10.

* ACC UI: send on distance bars change

* don't need this check - just send on first frame
cydia2020 pushed a commit to cydia2020/dodgypilot that referenced this pull request May 14, 2024
…ai#31760)

* start at param

* start by sending personality

* change to personality

* POC: button changes personality

* what's wrong with this?

* fix

* not really possible but fuzzy test catches this

* there's always a typo

* dang, we're dropping messages

* clean up

* no comment

* bump

* rename

* revert longplan

* revert this

* Fix check

* more appropriate up here

* consistenet

* Update selfdrive/car/toyota/carstate.py

* Update ref_commit
cydia2020 pushed a commit to cydia2020/dodgypilot that referenced this pull request May 14, 2024
* Ford: show longitudinal personality in IPC

Adjust the time gap to 2, 3 or 4 bars for openpilot long.

TODO: set AccTGap_D_Dsply when value changes so that the popup appears
with the new time gap

Depends on commaai#31760.

* ACC UI: show time gap popup when distance changes

* Revert "ACC UI: show time gap popup when distance changes"

This reverts commit c4e8e10.

* ACC UI: send on distance bars change

* don't need this check - just send on first frame
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants