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

Cm107 #5

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Cm107 #5

wants to merge 5 commits into from

Conversation

cm107
Copy link
Collaborator

@cm107 cm107 commented Apr 27, 2020

I confirmed that the problem regarding the bounding boxes has been fixed.
FYI: I think it would be a good idea to reorganize test/train.py.
There are magic numbers inside of your functions, which is not good.
Someone who is looking for a usage example would have to read through all of your functions to know what needs to be modified. 💦

cm107 added 5 commits April 27, 2020 14:29
…tation is doing.

I confirmed that the keypoint and segmentation rotation augmentation is working fine, but the size of the bounding box is not being preserved when there is no segmentation.
…hat result from non-polygon shapes being generated.
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