-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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: longitudinal support for all CAN-FD EV and Hybrids #26345
Conversation
With the HKG GV70 PR now committed, I have added two commits excluding cars flagged as CANFD_CAMERA_SCC from having radar disabled. I will upload updated route shortly. |
With latest change. |
Thanks, but this is blocked on refactoring the panda safety address checks. Once that's done, it'll be easy to support HDA1 long. |
Looks like only ICE CAN-FD will be blocked on that refactor. We can merge this in the meantime. |
@adeebshihadeh just curious, is ICE CAN-FD blocking on these PRs: #26090 commaai/panda#1107 ? or is this a separate issue? Just trying to see if I can contribute, but want to make sure i'm looking in the right direction. For ICE CAN-FD cars using radar cruise, it would be the same as the hybrid / evs correct? |
No, we don't support long for any CAN-FD models yet. Unfortunately, there's not much to contribute in this case. We just need to do the refactor ourselves. |
Description
OP long (and e2e) is currently enabled for HDA2 Can-fd HKG cars but not HDA1.
I have picked the modifications from #25434 (thank you Sunnyhaibin!) that only relate to enabling OP Long. This code should not affect HDA2 variants of cars that have both HDA1 and HDA2 variants because the flag for labelling a car as "CANFD_CAMERA_SCC" is placed after an "Else" statement following cars flagged as HDA2. Therefore, an HDA2 car can never receive the "CANFD_CAMERA_SCC" flag. I have added EV6 and Ioniq 5 to the CAMERA_SCC_CAR car list because I can verify this with my HDA1 EV6, but the referenced PR shows that other Can-fd cars have this tested as well.11/8/22 update:
Based on latest code, all that is changed is 1) set OP long to true for can-fd cars and 2) and "if hda2" in front of sending adrv messages, 3) two commits excluding cars flagged as CANFD_CAMERA_SCC from having radar disabled.
Prerequisite
This requires a Panda update that does the following:
see: commaai/panda#1128
Verification
HDA1 EV6
Route: 68d6a96e703c00c9|2022-11-03--13-35-26