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

Hyundai: dashcam FCEV #1973

Merged
merged 4 commits into from
Mar 11, 2025
Merged

Hyundai: dashcam FCEV #1973

merged 4 commits into from
Mar 11, 2025

Conversation

sshane
Copy link
Contributor

@sshane sshane commented Mar 11, 2025

Dashcamming the car from #933 as it's missing an rx check

Bug is in the safety tests in #1903. @sunnyhaibin can you fix?

@github-actions github-actions bot added car related to opendbc/car/ hyundai car safety vehicle-specific safety code labels Mar 11, 2025
@sshane sshane merged commit 6254215 into master Mar 11, 2025
8 checks passed
@sshane sshane deleted the disable-hyundai-fcev branch March 11, 2025 04:09
sunnyhaibin added a commit to sunnypilot/opendbc that referenced this pull request Mar 11, 2025
sshane added a commit that referenced this pull request Mar 12, 2025
* Revert "Hyundai: dashcam FCEV (#1973)"

This reverts commit 6254215.

* Hyundai: Fix FCEV RX check and move out of dashcam

* less duplication

* line

* same

* add missing test

---------

Co-authored-by: Shane Smiskol <shane@smiskol.com>
chrispypatt pushed a commit to chrispypatt/opendbc that referenced this pull request Mar 12, 2025
* Disable FCEV

* coverage

* fixme

* misra

misra
chrispypatt pushed a commit to chrispypatt/opendbc that referenced this pull request Mar 12, 2025
* Revert "Hyundai: dashcam FCEV (commaai#1973)"

This reverts commit 6254215.

* Hyundai: Fix FCEV RX check and move out of dashcam

* less duplication

* line

* same

* add missing test

---------

Co-authored-by: Shane Smiskol <shane@smiskol.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
car safety vehicle-specific safety code car related to opendbc/car/ hyundai
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant