-
Notifications
You must be signed in to change notification settings - Fork 157
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 <[email protected]> Co-authored-by: spacemanidol <[email protected]>
Documentation update in support of Yolov5 update to V6.1 (neuralmagic/yolov5#26)