-
Notifications
You must be signed in to change notification settings - Fork 26.3k
[quant][fx] Support conv1d and its fusion variants in QAT #74506
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: This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
🔗 Helpful links
💊 CI failures summary and remediationsAs of commit 3c423a0 (more details on the Dr. CI page):
🕵️ 3 new failures recognized by patternsThe following CI failures do not appear to be due to upstream breakages:
|
Summary: This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
Summary: This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
|
@jerryzh168 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
Summary: This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D35032995](https://our.internmc.facebook.com/intern/diff/D35032995) [ghstack-poisoned]
Summary: This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization TODO: add tests for conv - relu, maybe after we decide whether we want to refactor our tests to be smaller Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Reviewers: Subscribers: Tasks: Tags: ghstack-source-id: fffbb4c Pull Request resolved: #74506
|
@jerryzh168 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
|
what's the plan for Eager mode workflow support? Can we also add it either in this PR or in a follow-up PR? |
Summary: This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D35032995](https://our.internmc.facebook.com/intern/diff/D35032995) [ghstack-poisoned]
Summary: This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D35032995](https://our.internmc.facebook.com/intern/diff/D35032995) [ghstack-poisoned]
Summary: This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D35032995](https://our.internmc.facebook.com/intern/diff/D35032995) [ghstack-poisoned]
Summary: This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D35032995](https://our.internmc.facebook.com/intern/diff/D35032995) [ghstack-poisoned]
we should support it, but not sure if it is P0 or not, the tests in eager mode could be refactored to be more general so that adding test is easier |
Summary: This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D35032995](https://our.internmc.facebook.com/intern/diff/D35032995) [ghstack-poisoned]
|
@jerryzh168 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
Summary: This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D35032995](https://our.internmc.facebook.com/intern/diff/D35032995) [ghstack-poisoned]
Summary: This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization TODO: add tests for conv - relu, maybe after we decide whether we want to refactor our tests to be smaller Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Reviewers: Subscribers: Tasks: Tags: ghstack-source-id: 58e014e Pull Request resolved: #74506
Summary: This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D35032995](https://our.internmc.facebook.com/intern/diff/D35032995) [ghstack-poisoned]
Summary: This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization TODO: add tests for conv - relu, maybe after we decide whether we want to refactor our tests to be smaller Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Reviewers: Subscribers: Tasks: Tags: ghstack-source-id: 2485a7c Pull Request resolved: #74506
|
@jerryzh168 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
I think that it should ideally be added at the same priority, both Eager mode and FX graph mode are first class citizens. |
Summary: Pull Request resolved: #74506 This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Imported from OSS Reviewed By: vkuzo Differential Revision: D35032995 fbshipit-source-id: 645da33f0d893aa44f35ee1384fd1539a9c788e7
Summary: Pull Request resolved: #74506 This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization Test Plan: python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu Imported from OSS Reviewed By: vkuzo Differential Revision: D35032995 fbshipit-source-id: 645da33f0d893aa44f35ee1384fd1539a9c788e7 (cherry picked from commit 6b583ba)
Stack from ghstack (oldest at bottom):
Summary:
This PR supports qat Conv1d, ConvBn1d, ConvBnReLU1d, ConvReLU1d in qat in FX Graph Mode Quantization
Test Plan:
python test/test_quantization.py TestQuantizeFx.test_conv_bn_relu
Reviewers:
Subscribers:
Tasks:
Tags:
Differential Revision: D35032995