-
Notifications
You must be signed in to change notification settings - Fork 251
In tutorials/quantize_vit, extract common methods to util.py #238
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
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/ao/238
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit b7d6811 with merge base adfe570 ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
Not sure how this test failure relates to the code change.... |
It's unrelated, we'll merge the fix then your change so no worries! |
@lancerts can you rebase to main? The error should go away |
The sync from your end seems still same failure https://github.com/pytorch/ao/actions/runs/9120697036/job/25078538641 So did a rebase on my end. TY |
ce7699d
to
b226a7e
Compare
Co-authored-by: Mark Saroufim <[email protected]>
Co-authored-by: Mark Saroufim <[email protected]>
…#238) * Extract common methods to util.py * Update tutorials/quantize_vit/run_vit_b.py Co-authored-by: Mark Saroufim <[email protected]> * Update tutorials/quantize_vit/run_vit_b_quant.py Co-authored-by: Mark Saroufim <[email protected]> * amend * amend * Include the torchao utils --------- Co-authored-by: Mark Saroufim <[email protected]> Co-authored-by: Mark Saroufim <[email protected]>
In tutorials/quantize_vit, extract common methods to util.py
Local execution tested via ./run.sh
cc @msaroufim