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

Complete code for training with zero annotations (Issue #43) #221

Closed
wants to merge 5 commits into from

Conversation

jrbastien
Copy link

Hi,

I have incorporated NiklasWilson code and also corrected utils.py in order to be able to be able to use the lines with no bounding boxes in data_train.txt.

There was a message "Ignoreable error: Folder/File could not be accessed, may not exist ->" that I thought could mislead the user so I improved it to say "Sometimes it is desirable that your training dataset include images with non-labeled objects that you do not want to detect (i.e. negative samples without bounding box). Non-annotated images can be placed in -> /home/username/Src/TrainYourOwnYOLO/Data/Source_Images/no_object. However, this is not mandatory, you can now continue with the training."

I think that it is complete and ready to use. Feel free to merge into the master if you like.

@AntonMu
Copy link
Owner

AntonMu commented Apr 16, 2021

Hi @jrbastien

Thank you for all your work! I appreciate it.

I would like to merge it at some point but I would need to review it carefully. It contains a lot of code changes that need to be merged safely.

To get things moving, please have a look at the contributing doc. It contains a list of instructions to make a successful pull request.

If your PR contains independent changes please consider splitting them up into smaller PRs.

Thank you!

@AntonMu
Copy link
Owner

AntonMu commented Sep 26, 2021

Closing this due to inactivity.

@AntonMu AntonMu closed this Sep 26, 2021
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

Successfully merging this pull request may close these issues.

None yet

2 participants