-
-
Notifications
You must be signed in to change notification settings - Fork 16.2k
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
Add section on downloading weights #624
Conversation
@jkschin thanks!! We do have a placeholder bash script there to download weights, but in reality it's not needed as weights are autodownloaded when required. For example if you git clone a fresh copy of the repo and right away run detect.py, it will autodownload yolov5s.pt and put it in your /yolov5 folder. If you ask for a different model, it will download that model as well if it's available: git clone https://github.com/ultralytics/yolov5
cd yolov5
python detect.py |
Ah right - I was using this not for detection but to convert a model into ONNX and thus I only needed the weights. Feel free to close if you think it isn't useful. |
@jkschin well I'm not sure, I'll think about it. It's always good to get user feedback to figure out how to make this work as smoothly as possible. There are also weights available with the release assets, but most people probably don't know about them there either: https://github.com/ultralytics/yolov5/releases/tag/v2.0 |
@jkschin by the way, export.py also autodownloads weights on request if it doesn't find them locally. The main files that do this are train.py, detect.py, test.py and export.py. But like you said most people may not be aware of this. What do you think is the best method to explain this? Lines 25 to 26 in 941cb26
|
Oh interesting. I'll let you know my thought process.
I think in general it isn't too hard to figure this out as is, but it could help if the README had explicit instructions on exporting models. I think there are folks out there who just want to convert models and this repository could be a pretty neat tool. Might help with SEO and bringing more traffic to this repo! |
P.S. For what I was doing at least, an "Export" section in the README could be helpful. I didn't spend too long finding out how to export in any case. |
@jkschin right, those are all good points. Thanks for the explanation. So the main driver is really the export rather than the weights download, but neither are immediately obvious when you scan the README, which is the main eyeball destination of course. Ok I'll think about what to do about this, but I think you are right a new section in the readme makes sense, but then we probably want an 'Export' section instead of a weights download section? |
Yeah an "Export" section makes sense - I can add it in this week if you are open to accepting PRs for this. |
Sure, sounds good! |
Make sure you see https://docs.ultralytics.com/yolov5/tutorials/model_export also if you are in the export market. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
I thought it would be a good idea to add this in.
π οΈ PR Summary
Made with β€οΈ by Ultralytics Actions
π Summary
Added instructions for downloading model weights to README.
π Key Changes
π― Purpose & Impact