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

Add Genesis G80 to special panda safety mode #2602

Merged
merged 1 commit into from
Nov 22, 2020

Conversation

slppo
Copy link
Contributor

@slppo slppo commented Nov 22, 2020

Description Upon activating OP in 2018 Genesis G80, screen immediately shows "Take Control Immediately, Controls Mismatch" and disengages. This PR is to add the G80 to special panda safety mode to resolve this issue.

Verification I was able to verify by driving my G80 on my fork and engage OP without encountering the Controls Mismatch screen. I am unable to provide the exact route because I am unable to connect my device with Comma Connect (scanning QR code doesn't do anything), so I don't have access to useradmin.

Route
Route: 6b301bf83f10aa90|2020-11-22--16-45-07

@adeebshihadeh
Copy link
Contributor

Can you provide a route from this branch that we can use for our tests? Currently, there is no test route for the Genesis G80.

@slppo
Copy link
Contributor Author

slppo commented Nov 22, 2020

Sure -- is there a way to get the route directly from the device without using https://my.comma.ai?

@adeebshihadeh
Copy link
Contributor

No, the route will need to be fully uploaded. You can check the upload status in your my.comma.ai/useradmin page.

@slppo
Copy link
Contributor Author

slppo commented Nov 22, 2020

Yeah so I wasn't able to connect my device to my account (scanning the QR code in the app doesn't do anything) so my useradmin page doesn't have any devices, not sure if there's another way to connect them?

Might be related issue on the Comma connect side commaai/connect#2

@slppo
Copy link
Contributor Author

slppo commented Nov 22, 2020

Was able to get account linked. Here's the route ID 6b301bf83f10aa90|2020-11-22--16-45-07

@adeebshihadeh adeebshihadeh merged commit 9c93612 into commaai:master Nov 22, 2020
@pd0wm pd0wm added bugfix car vehicle-specific and removed car bug fix labels Jun 2, 2022
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.

3 participants