Repository navigation
Add Hyundai Grandeur Diesel 2019 support - #320
Conversation
|
Thank you for your PR! If you're not already in the FrogPilot Discord, feel free to join and let me know you've opened a PR! |
4596112 to
f8fb066
Compare
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
3bba1f1 to
2414467
Compare
|
I pushed a commit to your PR that lines the port up with the other IG cars. The platform is now "HYUNDAI_AZERA_DIESEL_6TH_GEN", the docs name is "Hyundai Azera Diesel 2019" so the year parses, and it's in the lane-line HUD list with the other IG cars. I also dropped "MANDO_RADAR" since the route you drove on your openpilot fork ran without it. If your car does output radar points on "0x500" on "bus 1", let me know and we can add it back. Could you pull the latest version of your branch and do a drive? Engaging, steering, pressing the gas and brake, and checking the lane lines on the cluster would cover it. If you had the car picked manually in "Car Model", you'll need to pick "Hyundai Azera Diesel 2019" again since the platform name changed. |
|
That crash comes from the branch name. "MAKE-PRS-HERE" is blocked from running on a car, and that snapshot is missing "sentry.capture_message", so it crashes instead of showing the block message. Even without the crash, that branch name would force the car to "MOCK". Could you create a new branch on your fork from "MAKE-PRS-HERE" with any other name, for example "azera-diesel", and install it with "installer.comma.ai/atawi44/azera-diesel"? It's the same commit, so the port will be identical, and the same drive will cover it. |
|
I re-enabled "MANDO_RADAR" in a new commit. I checked your route too. "0x500" through "0x53F" are on "bus 1", the tracks decode with the Mando DBC, and the radar interface turns them into points. Thanks for testing! |



No description provided.