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

Use UI speed for vEgoCluster #11

Closed

Conversation

carleeno
Copy link

@carleeno carleeno commented Jul 21, 2024

Description

This fixes the display bug that the on-screen speedometer doesn't always agree with the OEM speedometer.

This is because the OEM speedometer has some hysteresis added to prevent it from flickering.

Verification

(To re-verify after changes)

Route

Route: (need new route after changes)

@github-actions github-actions bot added the car label Jul 21, 2024
Copy link

github-actions bot commented Jul 21, 2024

Thanks for contributing to openpilot! 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
    • the change is something we merge
    • include a route or your device' dongle ID if relevant

@carleeno carleeno marked this pull request as draft July 21, 2024 04:46
@carleeno
Copy link
Author

carleeno commented Jul 21, 2024

Draft until re-verified

@carleeno carleeno force-pushed the use-UI-speed-for-vEgoCluster branch from 23d5470 to 20e6145 Compare August 3, 2024 16:32
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.

2 participants