-
Notifications
You must be signed in to change notification settings - Fork 25.6k
[quant][graphmode] Run RemoveRedundantDequantize in the end #39923
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
Summary: Test Plan: Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
💊 CI failures summary and remediationsAs of commit b17af0e (more details on the Dr. CI page): ✅ None of the CI failures appear to be your fault 💚
❄️ 1 failure tentatively classified as flakybut reruns have not yet been triggered to confirm:
|
Summary: Test Plan: Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D22014712](https://our.internmc.facebook.com/intern/diff/D22014712) [ghstack-poisoned]
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.
lg, would be awesome if we also had some context on the "why" (for new team members to learn)
sure, updated summary |
Summary: `propagateQuantizationOps` generate new `dequantize` ops, so this pass should run after that. Test Plan: Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D22014712](https://our.internmc.facebook.com/intern/diff/D22014712) [ghstack-poisoned]
This pull request has been merged in b372000. |
…39923) Summary: Pull Request resolved: pytorch#39923 Test Plan: Imported from OSS Differential Revision: D22014712 fbshipit-source-id: a681bbab5d90ac8ebeff03cde5277501e86ee1d4
Stack from ghstack:
repeat
#39925 [quant][graphmode] Support quantizingrepeat
Summary:
propagateQuantizationOps
generate newdequantize
ops, so this pass should run after that.Test Plan:
Reviewers:
Subscribers:
Tasks:
Tags:
Differential Revision: D22014712