Implemented a better way to handle group multiple detections than a simple pruning
I have implemented a better way to incorporate several detections using detect_all_faces
.
When a single face should be detected, it merges all overlapping detected bounding boxes that are over threshold into one single bounding box. In the old way, when multiple faces should be detected, for overlapping detected bounding boxes, only the one with the highest detection value was returned.
In the new version, also for multiple faces, overlapping bounding boxes are merged into one, similar to the single face case.
@andre.anjos, @tiago.pereira, @amohammadi : Could you please check this?
Merge request reports
Activity
Thanks for the PR @mguenther
I will have a look tomorrow if you don't mind.
Added 1 commit:
- 9e449582 - Fixed issue with averaging detections when there was nothing to average
Added 1 commit:
- 3713eee8 - Removed superfluous return
Hey @mguenther I've seen that you are still pushing code in this branch.
I will rename the as WIP if you don't mind.
Added 19 commits:
-
3713eee8...043a57de - 16 commits from branch
master
- 2e466166 - Implemented a better way to handle group multiple detections than a simple pruni…
- f18d05b5 - Fixed issue with averaging detections when there was nothing to average
- 85ad030f - Removed superfluous return
Toggle commit list-
3713eee8...043a57de - 16 commits from branch
Just rebased with the master.
I will merge tomorrow, because we are having a weird issue with the CI and I just catched the issue.
If look in the pipelines (https://gitlab.idiap.ch/bob/bob.ip.facedetect/pipelines) all stages are complete, but the status is still running.
Mentioned in commit 65d22ad7
Mentioned in issue bob.bio.face#11 (closed)
Mentioned in merge request bob.bio.face!16 (merged)
Mentioned in issue bob.bio.video#4 (closed)
Mentioned in merge request bob.bio.video!8 (merged)
Mentioned in commit bob.bio.video@a17dfeb4