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

ev6 w/o HDAII doesn't display FCA/AEB warning when in OP long #1157

Open
bdimcheff opened this issue Sep 16, 2023 · 5 comments
Open

ev6 w/o HDAII doesn't display FCA/AEB warning when in OP long #1157

bdimcheff opened this issue Sep 16, 2023 · 5 comments
Labels
bug car related to opendbc/car/ hyundai

Comments

@bdimcheff
Copy link

Describe the bug

It's my understanding (based on the warning message about OP long being alpha) that on an ev6 with HDA I, enabling OP longitudinal control will disable stock AEB and FCA. The indicator on the dashboard for my ev6 definitely does not have any warning lights illuminated, however. I don't know if this means that stock AEB/FCA is still enabled, or if it is disabled but the warning lights aren't properly illuminating.

Which car does this affect?

Kia EV6 2023 Wind AWD (HDA I)

Provide a route where the issue occurs

fab807792f63fea6|2023-09-15--18-57-23--0

openpilot version

master-ci

Additional info

No response

@bdimcheff bdimcheff added bug car related to opendbc/car/ labels Sep 16, 2023
@sshane
Copy link
Contributor

sshane commented Sep 18, 2023

It indeed should be showing for HDA1 cars, I think we forgot about this when merging commaai/openpilot#26345.

Can you get two routes: one with AEB disabled in the settings so it shows this icon, and one with AEB enabled as normal?

@bdimcheff
Copy link
Author

@sshane yeah sure. I assume this means to disable it via the car's control settings and just drive for a minute. Do I need to actually activate openpilot, or just route logging while I'm driving is enough?

@sshane
Copy link
Contributor

sshane commented Sep 20, 2023

Just 1 minute parked should be enough.

@bdimcheff
Copy link
Author

aeb enabled/default settings on fab807792f63fea6|2023-09-22--19-37-45--0.

aeb disabled for the latter part of fab807792f63fea6|2023-09-22--19-41-46--0. It seems like you can't disable it permanently, as it toggles back on when the car is power cycled.

@sshane
Copy link
Contributor

sshane commented Sep 23, 2023

Can you try this? commaai/openpilot#30021

@sshane sshane added the hyundai label Aug 20, 2024
@sshane sshane transferred this issue from commaai/openpilot Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug car related to opendbc/car/ hyundai
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants