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

GV70 2025 Electric #1622

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

GV70 2025 Electric #1622

wants to merge 1 commit into from

Conversation

morrislee
Copy link

Adding fingerprints for 2025 GV70 fingerprint by @phynicle

Adding fingerprints for 2025 GV70 fingerprint by @phynicle
@github-actions github-actions bot added car related to opendbc/car/ hyundai fingerprint labels Jan 12, 2025
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributing to opendbc! In order for us to review your PR as quickly as possible, check the following:

  • Convert your PR to a draft unless it's ready to review
  • Read the contributing docs
  • Before marking as "ready for review", ensure:
    • the goal is clearly stated in the description
    • all the tests are passing
    • include a route or your device' dongle ID if relevant

@morrislee
Copy link
Author

I believe this is all is needed to add support for GV70 2025 described by phynicle

@sshane
Copy link
Contributor

sshane commented Jan 13, 2025

What is the dongle id?

@morrislee
Copy link
Author

What is the dongle id?

waiting for a route, I am getting mixed info. @phynicle says just needs fingerprint, sunny says it needs more. Need a route to confirm

@jyoung8607 jyoung8607 marked this pull request as draft January 13, 2025 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
car related to opendbc/car/ fingerprint hyundai
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants