-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
[quant][fx][be] Remove _input_output_observed from backend_config #92589
Conversation
Summary: This is no longer needed, we can use dtype to decide whether an observer is needed or not Test Plan: python test/test_quantization.py TestQuantizeFx python test/test_quantization.py TestQuantizeFxOps Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/92589
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit c170fae: This comment was automatically generated by Dr. CI and updates every 15 minutes. |
Summary: This is no longer needed, we can use dtype to decide whether an observer is needed or not Test Plan: python test/test_quantization.py TestQuantizeFx python test/test_quantization.py TestQuantizeFxOps Reviewers: Subscribers: Tasks: Tags: ghstack-source-id: 2a9ab99ee7a824954ff9af1a19655cc781d6c3b2 Pull Request resolved: #92589
…_config" Summary: This is no longer needed, we can use dtype to decide whether an observer is needed or not Test Plan: python test/test_quantization.py TestQuantizeFx python test/test_quantization.py TestQuantizeFxOps Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
Summary: This is no longer needed, we can use dtype to decide whether an observer is needed or not Test Plan: python test/test_quantization.py TestQuantizeFx python test/test_quantization.py TestQuantizeFxOps Reviewers: Subscribers: Tasks: Tags: ghstack-source-id: de0818e7bff1ae077257f270563a83f5fb82b29a Pull Request resolved: #92589
…_config" Summary: This is no longer needed, we can use dtype to decide whether an observer is needed or not Test Plan: python test/test_quantization.py TestQuantizeFx python test/test_quantization.py TestQuantizeFxOps Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
Summary: This is no longer needed, we can use dtype to decide whether an observer is needed or not Test Plan: python test/test_quantization.py TestQuantizeFx python test/test_quantization.py TestQuantizeFxOps Reviewers: Subscribers: Tasks: Tags: ghstack-source-id: a0ec0524eb9f06f3ee8b9967e416fdb7a5db857d Pull Request resolved: #92589
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.
lgtm
…_config" Summary: This is no longer needed, we can use dtype to decide whether an observer is needed or not Test Plan: python test/test_quantization.py TestQuantizeFx python test/test_quantization.py TestQuantizeFxOps Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
…_config" Summary: This is no longer needed, we can use dtype to decide whether an observer is needed or not Test Plan: python test/test_quantization.py TestQuantizeFx python test/test_quantization.py TestQuantizeFxOps Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
@pytorchbot rebase |
@pytorchbot successfully started a rebase job. Check the current status here |
…_config" Summary: This is no longer needed, we can use dtype to decide whether an observer is needed or not Test Plan: python test/test_quantization.py TestQuantizeFx python test/test_quantization.py TestQuantizeFxOps Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
Successfully rebased |
Summary: This is no longer needed, we can use dtype to decide whether an observer is needed or not Test Plan: python test/test_quantization.py TestQuantizeFx python test/test_quantization.py TestQuantizeFxOps Reviewers: Subscribers: Tasks: Tags: ghstack-source-id: 6a2f6bc89bed2beb502543e6365c189cf430df2a Pull Request resolved: #92589
…_config" Summary: This is no longer needed, we can use dtype to decide whether an observer is needed or not Test Plan: python test/test_quantization.py TestQuantizeFx python test/test_quantization.py TestQuantizeFxOps Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
…_config" Summary: This is no longer needed, we can use dtype to decide whether an observer is needed or not Test Plan: python test/test_quantization.py TestQuantizeFx python test/test_quantization.py TestQuantizeFxOps Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
…_config" Summary: This is no longer needed, we can use dtype to decide whether an observer is needed or not Test Plan: python test/test_quantization.py TestQuantizeFx python test/test_quantization.py TestQuantizeFxOps Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Stack from ghstack (oldest at bottom):
Summary:
This is no longer needed, we can use dtype to decide whether an observer is needed or not
Test Plan:
python test/test_quantization.py TestQuantizeFx
python test/test_quantization.py TestQuantizeFxOps
Reviewers:
Subscribers:
Tasks:
Tags: