-
Notifications
You must be signed in to change notification settings - Fork 26.3k
[quant] Make OP_LIST_TO_FUSER_METHOD public #43286
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: We need to use this in graph mode quantization on fx Test Plan: Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
💊 CI failures summary and remediationsAs of commit 9f405b4 (more details on the Dr. CI page): ✅ None of the CI failures appear to be your fault 💚
🚧 1 fixed upstream failure:These were probably caused by upstream breakages that were already fixed.
Please rebase on the
|
Summary: We need to use this in graph mode quantization on fx Test Plan: Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
Summary: We need to use this in graph mode quantization on fx Test Plan: Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D23221734](https://our.internmc.facebook.com/intern/diff/D23221734) [ghstack-poisoned]
Summary: We need to use this in graph mode quantization on fx Test Plan: Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D23221734](https://our.internmc.facebook.com/intern/diff/D23221734) [ghstack-poisoned]
Summary: We need to use this in graph mode quantization on fx Test Plan: Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D23221734](https://our.internmc.facebook.com/intern/diff/D23221734) [ghstack-poisoned]
vkuzo
left a comment
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. Optional suggeston for future PRs: would it make sense to have a clean split between:
- eager mode
- fx
- shared between eager mode and fx
i.e. the shared code would be in a separate directory or file, to make it clear what's shared and what's not, with ideally the shared part of the codebase growing over time, and eager-only shrinking?
Summary: We need to use this in graph mode quantization on fx Test Plan: Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D23221734](https://our.internmc.facebook.com/intern/diff/D23221734) [ghstack-poisoned]
If you just talk about things under torch/quantization then there is not much that is eager mode only, probably just https://github.com/pytorch/pytorch/blob/master/torch/quantization/quantize.py and https://github.com/pytorch/pytorch/blob/master/torch/quantization/stubs.py I think. There is also https://github.com/pytorch/pytorch/blob/master/torch/nn/quantized/modules/functional_modules.py which is only used in eager mode. we can certainly put them in the same folder, but not sure if it's worth it or not. |
Summary: We need to use this in graph mode quantization on fx Test Plan: Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D23221734](https://our.internmc.facebook.com/intern/diff/D23221734) [ghstack-poisoned]
|
This pull request has been merged in 217ddea. |
Stack from ghstack:
Summary:
We need to use this in graph mode quantization on fx
Test Plan:
Reviewers:
Subscribers:
Tasks:
Tags:
Differential Revision: D23221734