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

Does the Micro mode support multiple aspect ratios? #14

Closed
zakinp opened this issue Nov 25, 2022 · 1 comment
Closed

Does the Micro mode support multiple aspect ratios? #14

zakinp opened this issue Nov 25, 2022 · 1 comment

Comments

@zakinp
Copy link

zakinp commented Nov 25, 2022

The question is the same as title. Because I found that the tedd's dataset is all composed of square images.

@victorchall
Copy link
Owner

Yes, it's a universal feature, HOWEVER when you mix aspect ratios you need a lot more images as each "bucket" of each aspect ratio must fill the batch_size.

I.e. if you have 13 images at 16:9 and your batch size is 4, 13 does not fit into 4 evenly, thus one image will be dropped.

If you have many hundreds of images the dropped images are not going to be a big deal. If you only have 20 images you're better off making them all the same aspect ratio. If all images are the same aspect ratio it will not drop anything regardless of batch size.

So, for people just doing fast micro stuff you may just be better off doing your own cropping to make them all the same aspect ratio. Also, try to keep them all larger if you can. 540x540, 700x700, etc so crop jitter can do its magic.

Ted was created before I added multiaspect and crop jitter features.

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

No branches or pull requests

2 participants