-
Notifications
You must be signed in to change notification settings - Fork 133
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
Allow to import Open Images annotations without images #463
Conversation
@@ -269,7 +269,7 @@ def load_from(annotation_name): | |||
if _RE_INVALID_PATH_COMPONENT.fullmatch(subset): | |||
raise UnsupportedSubsetNameError(item_id=image_id, subset=subset) | |||
|
|||
image_path = image_paths_by_id.get(image_id) | |||
image_path = image_paths_by_id.get(image_id, image_id) | |||
|
|||
image = None | |||
if image_path is not None: |
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.
Now, this condition is never False, and then image.path
is set to an incorrect value.
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.
Changed this solution
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.
Will you add a test for this? There was a test for the image meta, maybe it needs to be changed?
Added test. |
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.
I don't see there is image metainfo extractor parameter, probably you will need to add it for CVAT integration.
Summary and Motivation
Open Images requires you to know the size of the image.
Current format implementation does not add
Image
toItem
if the image has no path.It is not possible to load a dataset without images, but with meta information about them.
How to test
Checklist
develop
branchLicense
Feel free to contact the maintainers if that's a concern.