-
Notifications
You must be signed in to change notification settings - Fork 7k
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
Adding resnext101 64x4d model #5935
Merged
YosuaMichael
merged 15 commits into
pytorch:main
from
YosuaMichael:models/resnext101_64x4d
May 9, 2022
Merged
Adding resnext101 64x4d model #5935
YosuaMichael
merged 15 commits into
pytorch:main
from
YosuaMichael:models/resnext101_64x4d
May 9, 2022
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
datumbox
approved these changes
May 9, 2022
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.
@YosuaMichael LGTM, thanks! Just one minor nit, see below.
The failing test seems to not related to this PR, see #5971 |
24 tasks
37 tasks
facebook-github-bot
pushed a commit
that referenced
this pull request
May 11, 2022
Summary: * Add resnext101_64x4d model definition * Add test for resnext101_64x4d * Add resnext101_64x4d weight * Update checkpoint to use EMA weigth * Add quantization model signature for resnext101_64x4d * Fix class name and update accuracy using 1 gpu and batch_size=1 * Apply ufmt * Update the quantized weight and accuracy that we still keep the training log * Add quantized expect file * Update docs and fix acc1 * Add recipe for quantized to PR * Update models.rst Reviewed By: YosuaMichael Differential Revision: D36281598 fbshipit-source-id: 300bd36343b8ad8b185a246b794e078bdf67f5c8
16 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolve #3485
Training and Validation script
We train the model using the following script:
From the training, we found the best result is on epoch 455 with the EMA model.
We validate with the 1 gpu and batch_size==1 using the following script (with final output):
Quantized model
We do quantize the model by using the following script:
And from the result we validate it again using 1 gpu and batch_size=1 with the following script: