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

fix: Change YOLO import default image extension choices to .jpg, .jpeg, .png #353

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EdjeElectronics
Copy link

This simple PR fixes the default image extension choices when using label-studio-converter to import YOLO data.

  • In yolo.py, the function definition for convert_yolo_to_ls uses image_ext=".jpg,.jpeg,.png" as the default image extension options. However, this default choice is overridden by the default --image-ext argument, which is just ".jpg".
  • The PR changes the default choices in the --image-ext argument to match the function default, because it seems like that's what was intended by the original authors of the code.
  • It changes the default --image-ext argument from ".jpg" to ".jpg,.jpeg.png"

Ultimately, this change makes it easier for a new user to use label-studio-converter without having to know about the --image-ext argument, which isn't documented anywhere. The Tutorial: Importing Local YOLO Pre-Annotated Images to Label Studio doesn't discuss the --image_ext argument at all, so new users could potentially be tripped up when they try to import images that aren't .jpg.

@github-actions github-actions bot added the fix label Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant