yolo import - getting image list from image_info if possible#111
yolo import - getting image list from image_info if possible#111
Conversation
There was a problem hiding this comment.
The current approach with fake image name crafting doesn't look right to me. It also keeps these fake image paths in the media paths of produced dataset items. It looks like it would be better to change _get_lazy_subset_items to generate a list of (image path, annotation path) instead.
There was a problem hiding this comment.
Do you suggest to create DatasetItems with empty media in such cases?
There was a problem hiding this comment.
No, creating a list of tuples with 1 or 2 paths filled is probably enough.
There was a problem hiding this comment.
I mean, to create DatasetItem with media we need a path for that media. And unless we pass the correct extension from somewhere, it will be a faked path
There was a problem hiding this comment.
Ah, I forgot we removed a way to create meta-only Image with just size. Ok, let's keep it as it is.
There was a problem hiding this comment.
As an option, we could try to change the image info passed to include extensions.
|



Summary
Ultralytics yolo subset can be specified as a folder in data.yaml. In that case, if there are only annotations (no images) in the dataset folder, importer finds no images, creates no items, and therefore does not read any annotations.
Using image_info to get items.
How to test
Checklist
License
Feel free to contact the maintainers if that's a concern.