Skip to content
This repository was archived by the owner on Jun 3, 2025. It is now read-only.

Conversation

@KSGulin
Copy link
Contributor

@KSGulin KSGulin commented Apr 7, 2022

Documentation update in support of Yolov5 update to V6.1 (neuralmagic/yolov5#26)

bfineran
bfineran previously approved these changes Apr 7, 2022
Copy link
Contributor

@bfineran bfineran left a 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

@bfineran bfineran requested review from a team, dbogunowicz, markurtz and mgoin and removed request for a team April 7, 2022 17:30
dbogunowicz
dbogunowicz previously approved these changes Apr 8, 2022
Copy link
Contributor

@dbogunowicz dbogunowicz left a 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.

@dbogunowicz dbogunowicz added the 0.12 release Pull request pending for 0.12 release. label Apr 8, 2022
@KSGulin KSGulin dismissed stale reviews from dbogunowicz and bfineran via 8374837 April 8, 2022 11:22
@KSGulin
Copy link
Contributor Author

KSGulin commented Apr 8, 2022

@anmarques Good catch Alex. Updated corresponding PRs to use the legacy hyperparameter files

dbogunowicz
dbogunowicz previously approved these changes Apr 8, 2022
Copy link
Contributor

@dbogunowicz dbogunowicz left a 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).

bfineran
bfineran previously approved these changes Apr 8, 2022
@spacemanidol spacemanidol merged commit 054bb61 into main Apr 8, 2022
@spacemanidol spacemanidol deleted the yolov5-docs-update branch April 8, 2022 18:58
dbogunowicz pushed a commit that referenced this pull request Apr 11, 2022
* 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]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

0.12 release Pull request pending for 0.12 release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants