Skip to content

Conversation

datumbox
Copy link
Contributor

@datumbox datumbox commented Oct 6, 2021

The bug doesn't affect the accuracy of the technique but it messes up the counter of how many updates we did to the weights. Fixing to cherry-pick on the upcoming release branch.

Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stamping

Copy link
Contributor

@prabhat00155 prabhat00155 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks @datumbox!

@datumbox datumbox merged commit 4b6fc6b into pytorch:main Oct 6, 2021
@datumbox datumbox deleted the bug/ema_n_averaged branch October 6, 2021 09:29
datumbox added a commit to datumbox/vision that referenced this pull request Oct 6, 2021
# Conflicts:
#	references/classification/utils.py
datumbox added a commit that referenced this pull request Oct 6, 2021
# Conflicts:
#	references/classification/utils.py
facebook-github-bot pushed a commit that referenced this pull request Oct 8, 2021
Reviewed By: NicolasHug

Differential Revision: D31505558

fbshipit-source-id: 0119066c61141a03812926d1a874fe47ee85bb3b
mszhanyi pushed a commit to mszhanyi/vision that referenced this pull request Oct 19, 2021
cyyever pushed a commit to cyyever/vision that referenced this pull request Nov 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants