-
Notifications
You must be signed in to change notification settings - Fork 150
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
Yolov5 docs update #684
Yolov5 docs update #684
Conversation
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.
LGTM @KSGulin will need to update any recipes/model cards in zoomodels as well
integrations/ultralytics-yolov5/tutorials/sparsifying_yolov5_using_recipes.md
Outdated
Show resolved
Hide resolved
integrations/ultralytics-yolov5/tutorials/yolov5_sparse_transfer_learning.md
Outdated
Show resolved
Hide resolved
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.
Approve for now, I will also test those docs soon by following the instruction step-by-step.
@anmarques Good catch Alex. Updated corresponding PRs to use the legacy hyperparameter files |
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.
as discussed in private conversation with @KSGulin , I did run some rudimentary testing on the yolov5 and it works as expected (the base functionalities).
* Initial: yolov5 docs update for v6.1 * Update: move sparseml install outside requirements.txt * Update: reference legacy hyperparam files * Fix: nit * Fix: remove redundant sparseml install * point setup to release branch Co-authored-by: Benjamin Fineran <bfineran@users.noreply.github.com> Co-authored-by: spacemanidol <dcampos3@illinois.edu>
Documentation update in support of Yolov5 update to V6.1 (neuralmagic/yolov5#26)