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

Ford: parse distance button #31733

Merged
merged 1 commit into from
Mar 7, 2024
Merged

Conversation

incognitojam
Copy link
Member

@incognitojam incognitojam commented Mar 6, 2024

Use the ACC gap toggle button signal from the SCCM. There are two other signals for "increase" and "decrease" gap buttons, but no supported car has these buttons.

e886087f430e7fe7/2024-03-03--13-00-04/25 contains some presses

Use the ACC gap toggle button signal from the SCCM. There are two other
signals for "increase" and "decrease" gap buttons, but no supported car
has these buttons.
Copy link
Contributor

github-actions bot commented Mar 6, 2024

Thanks for contributing to openpilot! In order for us to review your PR as quickly as possible, check the following:

  • Convert your PR to a draft unless it's ready to review
  • Read the contributing docs
  • Before marking as "ready for review", ensure:
    • the goal is clearly stated in the description
    • all the tests are passing
    • the change is something we merge
    • include a route or your device' dongle ID if relevant

@github-actions github-actions bot added car vehicle-specific ford labels Mar 6, 2024
@sshane sshane merged commit 98a491b into commaai:master Mar 7, 2024
31 checks passed
@incognitojam incognitojam deleted the ford-distance-button branch March 7, 2024 11:03
cydia2020 pushed a commit to cydia2020/dodgypilot that referenced this pull request May 14, 2024
Ford: parse ACC gap toggle button

Use the ACC gap toggle button signal from the SCCM. There are two other
signals for "increase" and "decrease" gap buttons, but no supported car
has these buttons.
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