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

loggerd: temporary workaround for synchronization issue #33120

Closed
wants to merge 1 commit into from

Conversation

deanlee
Copy link
Contributor

@deanlee deanlee commented Jul 29, 2024

Implements a temporary workaround to resolve synchronization problems documented in the following issues:

Copy link
Contributor

github-actions bot commented Jul 29, 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

@deanlee deanlee changed the title loggerd: Implement temporary workaround for synchronization issue loggerd: temporary workaround for synchronization issue Jul 29, 2024
@deanlee deanlee force-pushed the loggerd_quick_fix_sync_issue branch from 2f031d6 to 8b7ba81 Compare July 29, 2024 18:26
@deanlee
Copy link
Contributor Author

deanlee commented Aug 7, 2024

closed, submitted a solid solution: #33219

@deanlee deanlee closed this Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant