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

PT Export: Fix negative numbers issue #430

Closed
dari1495 opened this issue Dec 23, 2020 · 2 comments · Fixed by #434
Closed

PT Export: Fix negative numbers issue #430

dari1495 opened this issue Dec 23, 2020 · 2 comments · Fixed by #434
Assignees
Labels
backend Low Priority PoseTrack This issue is mostly needed for PoseTrack2018

Comments

@dari1495
Copy link
Collaborator

Make negative numbers 0

@dari1495 dari1495 added backend PoseTrack This issue is mostly needed for PoseTrack2018 Low Priority labels Dec 23, 2020
@dari1495 dari1495 self-assigned this Dec 23, 2020
@anDoer
Copy link
Collaborator

anDoer commented Dec 30, 2020

Please clamp all coordinates outside of the frame, i.e. set

x = max(min(width - 1, x), 0)
y = max(min(height - 1, y), 0)

@anDoer
Copy link
Collaborator

anDoer commented Jan 15, 2021

We have a special case with head bounding boxes:
as head bounding boxes are only required for evaluation (the length of the head bounding boxe's diagonal determines the person height), clamping negative coordinates would be harmful.

For that reason we should allow negative numbers for head bounding boxes.
This makes it necessary to change to export format of head bounding boxes from

[x1, y1, width, height] to [x1, y1, x2, y2]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Low Priority PoseTrack This issue is mostly needed for PoseTrack2018
Projects
None yet
2 participants