-
Notifications
You must be signed in to change notification settings - Fork 504
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
Update openimages script #1277
Update openimages script #1277
Conversation
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅ |
584eb48
to
27223ff
Compare
LGTM. New preprocessing script is working as expected. |
Hi @pgmpablo157321 Thanks for the very useful change. Can we have an option to download a subset of dataset (say 500 images)? I was trying to add a Github test for retinanet and downloading and preprocessing the dataset takes about an hour (on a 48 core machine). |
27223ff
to
72725e0
Compare
@arjunsuresh I just added a flag to do that. You can now run the script: |
Thanks a lot @pgmpablo157321 This change can save hours of effort for the submitters :) |
72725e0
to
15e3c2a
Compare
15e3c2a
to
35ea828
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @pgmpablo157321 for the fix.
Fixes #1257
Remove the
fiftyone
package dependency and update theopenimages.py
andopenimages_calibration.py
scripts