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.

Sign up for GitHub

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

Jump to bottom

Support arbitrary image extensions in formats #166

Closed
15 tasks done
zhiltsov-max opened this issue Mar 16, 2021 · 2 comments · Fixed by #173
Closed
15 tasks done

Support arbitrary image extensions in formats #166

zhiltsov-max opened this issue Mar 16, 2021 · 2 comments · Fixed by #173
Assignees
@zhiltsov-max
Labels
BUG Something isn't working data formats PR is related to dataset formats ENHANCE Enhancement of existing features

Comments

@zhiltsov-max
Copy link
Contributor

zhiltsov-max commented Mar 16, 2021 •
edited
Loading

Most of formats do not specify the image format they use. Some do, but it is very useful for the framework to support arbitrary image extensions to avoid useless reconverting between formats on exporting and to relax requirements on importing datasets.

Formats:

  • COCO
  • PASCAL VOC
  • YOLO
  • TF Detection API
  • WIDER Face
  • VGGFace2
  • MOT sequences
  • MOTS PNG
  • ImageNet
  • CamVid
  • CVAT
  • LabelMe
  • ICDAR13/15
  • Market-1501
  • LFW
@zhiltsov-max zhiltsov-max added BUG Something isn't working ENHANCE Enhancement of existing features data formats PR is related to dataset formats labels Mar 16, 2021
@zhiltsov-max zhiltsov-max self-assigned this Mar 16, 2021
This was referenced Mar 17, 2021
Merged
Merged
Merged
Merged
Merged
This was referenced Apr 9, 2021
Closed
Merged
@mrd
Copy link

mrd commented Apr 29, 2021

I found that PNG files were still hardwired into the CVAT converter. To fix this I used glob.glob. Before I submit a pull request can I make sure that import glob is acceptable in the cvat plugin, or would you rather it be done another way?

changes: develop...AdaptiveCity:arbitrary-image-format-cvat-converter

@zhiltsov-max
Copy link
Contributor Author

zhiltsov-max commented Apr 29, 2021 •
edited
Loading

@mrd, good observation. I'm not convinced it is reasonable to use lossy formats for video frames, but it definitely should be allowed. Globbing is allowed, but I'd suggest to change your implementation to use the approach from other formats to avoid repeated OS calls. You're welcome to open a PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zhiltsov-max zhiltsov-max

Labels
BUG Something isn't working data formats PR is related to dataset formats ENHANCE Enhancement of existing features
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Support arbitrary image extensions in formats (changelog) openvinotoolkit/datumaro
2 participants
@mrd @zhiltsov-max