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

Fix centroid imported from VIA file into bboxes dataset #302

Closed
sfmig opened this issue Sep 11, 2024 · 0 comments · Fixed by #303
Closed

Fix centroid imported from VIA file into bboxes dataset #302

sfmig opened this issue Sep 11, 2024 · 0 comments · Fixed by #303
Labels
bug Something isn't working

Comments

@sfmig
Copy link
Contributor

sfmig commented Sep 11, 2024

Is your feature request related to a problem? Please describe.
We talk about the centroid position of the bboxes in the docs, but what we are importing is actually the top left corner of the bounding box.

Describe the solution you'd like
Transform the imported data to store the actual bbox centroid in the position data array

centroid = top_left + shape / 2

Describe alternatives you've considered
We could change the documentation instead but storing the centroid seems more coherent with the rest of the tools we deal with, and is likely less jittery than the corners.

Additional context
\

@sfmig sfmig added enhancement New optional feature bug Something isn't working and removed enhancement New optional feature labels Sep 11, 2024
@sfmig sfmig mentioned this issue Sep 12, 2024
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

Successfully merging a pull request may close this issue.

1 participant