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

Call to .split_data() in .label() causing confusion #49

Open
AlainPilon opened this issue Sep 5, 2023 · 1 comment
Open

Call to .split_data() in .label() causing confusion #49

AlainPilon opened this issue Sep 5, 2023 · 1 comment
Assignees

Comments

@AlainPilon
Copy link

The call to split_data is causing some confusion IMHO.

It should either:

  • be a flag we can toggle on/off.
  • copy the files to the YOLO train/val folders instead of moving them.
  • have some output in the command line to tell the user about the file being moved.

Right now, someone calling label() would see the YOLO folder structure appears with an empty annotations folder and waste a few minutes trying to figure out why.

To make it even more confusing, calling label() a second time would crash on the image copy (because it does not want to overwrite existing file) and keep all the annotations in the annotations folder.

@capjamesg
Copy link
Member

@AlainPilon Thank you for opening the Issue! I think label() should save annotations unless no objects of interest were found in all images. Can you confirm that Autodistill was able to label at least one image in your dataset? Also, can you share the folder path for the empty annotations folder?

@capjamesg capjamesg self-assigned this Sep 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants