-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Reduce number of iterations in USB tutorial #2771
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/tutorials/2771
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit da46b62 with merge base 475671e ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
@@ -17,7 +17,7 @@ | |||
This tutorial will walk you through the basics of using the USB lighting | |||
package. | |||
Let's get started by training a ``FreeMatch``/``SoftMatch`` model on | |||
CIFAR-10 using pretrained ViT! | |||
CIFAR-10 using pretrained Vision Transformers (ViT)! |
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.
I wrote out what SSL and ViT stood for (please double check) since they are used throughout the tutorial and initially I didn't know what they meant. Maybe for ML experts they are obvious...
cc: @Hhhhhhao |
Should I be concerned with this warning?
|
Thanks for the initial comments! Hopefully I addressed the main issues, though I'm still not sure why CI is hanging/failing... As a newcomer I'd appreciate any help, thank you! |
I can't seem to add @Hhhhhhao as a reviewer or add labels to the PR 😓 |
How should I address this error? Do we make an exception or should I avoid using this word? |
Co-authored-by: Svetlana Karslioglu <svekars@meta.com>
Co-authored-by: Svetlana Karslioglu <svekars@meta.com>
Co-authored-by: Svetlana Karslioglu <svekars@meta.com>
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.
Thank you for your comments!
@Hhhhhhao Are there any other changes that I should make? |
I think that would be all for now. Thanks for the help! |
Fixes #2711
Description
Reduces the number of training iterations from 4,000 to 500, which now runs for 3 minutes with an accuracy of 87%.
Also fixes the broken image link on the USB framework.
Checklist