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.

Sign up for GitHub

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

Jump to bottom

AMP Check fix #9367

Merged
merged 1 commit into from
Sep 11, 2022
Merged

AMP Check fix #9367

merged 1 commit into from
Sep 11, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Sep 11, 2022 •
edited by UltralyticsAssistant
Loading

Resolves #9365

Signed-off-by: Glenn Jocher [email protected]

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Improved model integrity check during automatic mixed precision (AMP) testing in YOLOv5.

📊 Key Changes

  • Added deepcopy import from the Python copy module.
  • Modified AMP comparison function to use a deep copy of the model.

🎯 Purpose & Impact

  • Purpose: To ensure that the original model remains unaltered during the AMP closeness test, thus preserving its state for subsequent operations.
  • Impact: Users can expect more reliable testing of their models with AMP, which can lead to more consistent results when deploying models in different precision settings. 🛡️💡

@glenn-jocher
AMP Check fix
dd7ea87 
Resolves #9365

Signed-off-by: Glenn Jocher <[email protected]>
@glenn-jocher glenn-jocher merged commit 23701ea into master Sep 11, 2022
@glenn-jocher glenn-jocher deleted the update/amp branch September 11, 2022 10:56
@glenn-jocher glenn-jocher self-assigned this Sep 11, 2022
@glenn-jocher glenn-jocher mentioned this pull request Sep 11, 2022
Closed
2 tasks
glenn-jocher referenced this pull request Sep 11, 2022
@glenn-jocher @pre-commit-ci
Update DetectMultiBackend for tuple outputs 2 (#9275)
7aa263c 
* Update DetectMultiBackend for tuple outputs 2

Signed-off-by: Glenn Jocher <[email protected]>

* Update

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Update

* Update

* Update

Signed-off-by: Glenn Jocher <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@Hojland Hojland mentioned this pull request Oct 17, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@glenn-jocher glenn-jocher

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Getting error while training on YOLOv5 colab tutorial
1 participant
@glenn-jocher