-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
feat(cameras): add a route to update the last known image of a camera #371
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #371 +/- ##
==========================================
+ Coverage 86.19% 86.39% +0.20%
==========================================
Files 34 34
Lines 920 941 +21
==========================================
+ Hits 793 813 +20
- Misses 127 128 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Worth noting: @MateoLostanlen we'll have to update the engine. We now have 2 potential high frequency interactions: heartbeat & update image. I suggest reversing the equations to set the adequate frequency (& decorrelate it from the prediction frequency on raspberry pi):
What do you think? |
This PR introduces the following modifications:
Here is the updated UML
(live link: https://dbdiagram.io/d/Pyronear-UML-665a15d0b65d933879357b58)