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

WEBP image support for training YoloV5 #2166

Closed
Transigent opened this issue Feb 9, 2021 · 1 comment · Fixed by #2174
Closed

WEBP image support for training YoloV5 #2166

Transigent opened this issue Feb 9, 2021 · 1 comment · Fixed by #2174
Labels
question Further information is requested

Comments

@Transigent
Copy link
Contributor

❔Question

I noted in this YoloV3 post that cv2 supports a range of image formats including WEBP.

I assumed this would apply to YoloV% also, so I went ahead and regenerated my dataset as WEBP files (they are a lot smaller for the same quality) and tried to run on Colab in the usual way but it failed saying 'no images found'.

I then modified line 28 of utils/dataset.py
img_formats = ['bmp', 'jpg', 'jpeg', 'png', 'tif', 'tiff', 'dng'] # acceptable image suffixes
to add webp as a suffix and it appears to be working fine.

Is there a reason why WEBP files are not enabled here by default?

Thanks!
And thanks for the superb YoloV5 library!

Additional context

@Transigent Transigent added the question Further information is requested label Feb 9, 2021
@glenn-jocher
Copy link
Member

@Transigent hi, great discovery! No reason not to include webp if it works. Can you please submit a PR with the extension addition? Thank you!

@glenn-jocher glenn-jocher added TODO and removed TODO labels Feb 9, 2021
@glenn-jocher glenn-jocher linked a pull request Feb 10, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants