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

Update model #210

Merged
merged 5 commits into from
Jun 21, 2024
Merged

Update model #210

merged 5 commits into from
Jun 21, 2024

Conversation

MateoLostanlen
Copy link
Member

Our previous model was trained on images of size 1024. With the number of predictions that need to be made increasing on stations equipped with ptz cameras (smaller angle of view, therefore more images) and the reduction to 3 cpu, the pi doesn't have (almost) enough time to analyze everything. I'm also afraid of overuse. I therefore propose to switch to our best model in 640, reference 019aeef76fde49588d24fb167ba881a9.

We go from

On test test :

F1 : 0.916750 P : 0.934415 R : 0.899740 (threshold 0.1)

to

F1 : 0.890486 P : 0.933955 R : 0.850883 (threshold 0.1)

On test test :

F1 : 0.824006 P : 0.803797 R : 0.845258 (threshold 0.15)

to

F1 : 0.801754 P : 0.849442 R : 0.759136 (threshold 0.15)

I will re more training soon with 640 models only

@MateoLostanlen MateoLostanlen requested a review from a team June 21, 2024 16:39
Copy link

codecov bot commented Jun 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.38%. Comparing base (16fa010) to head (cac7c3a).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #210   +/-   ##
========================================
  Coverage    92.38%   92.38%           
========================================
  Files            6        6           
  Lines          486      486           
========================================
  Hits           449      449           
  Misses          37       37           
Flag Coverage Δ
unittests 92.38% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MateoLostanlen MateoLostanlen merged commit 4c7aebf into develop Jun 21, 2024
14 checks passed
@MateoLostanlen MateoLostanlen deleted the update_model branch June 21, 2024 17:08
@MateoLostanlen MateoLostanlen mentioned this pull request Jul 15, 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