[quant][graphmode][fx] Separate handling Copy operator to a helper function #54644

jerryzh168 · 2021-03-25T00:33:45Z

Stack from ghstack:

[quant][graphmode][fx][refactor] Quantize by Use of a Tensor instead of Tensor #54928 [quant][graphmode][fx][refactor] Quantize by Use of a Tensor instead of Tensor
[quant][graphmode][fx] Produce torch.cat instead of torch.ops.quantized.cat #54924 [quant][graphmode][fx] Produce torch.cat instead of torch.ops.quantized.cat
[quant][graphmode][fx] Optimize cat #54813 [quant][graphmode][fx] Optimize cat
[nn] Add remove_duplicate option for named_modules #54812 [nn] Add allow_duplicate option for named_modules
[quant][graphmode][fx][refactor] Factor out insert_observers_for_model to a separate function #54733 [quant][graphmode][fx][refactor] Factor out insert_observers_for_model to a separate function
[quant][graphmode][fx] Separate handling Copy operator to a helper function #54644 [quant][graphmode][fx] Separate handling Copy operator to a helper function
[quant][fx][graphmode][refactor] Change activation_post_process_map to track the observer name instead #54643 [quant][fx][graphmode][refactor] Change activation_post_process_map to track the observer name instead
[quant][graphmode][refactor] Remove reduandent code #54073 [quant][graphmode][refactor] Remove reduandent code
[quant][graphmode][fx] Add support for one value being quantized with different qconfigs #53586 [quant][graphmode][fx] Add support for one value being quantized with different qconfigs

Summary:
Previously we special case copy operator in normal insert observer code, this PR tries to split the
special case logic to a separate function and keep the rest of the code clean.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: D27314678

…nction Summary: Previously we special case copy operator in normal insert observer code, this PR tries to split the special case logic to a separate function and keep the rest of the code clean. Test Plan: Reviewers: Subscribers: Tasks: Tags: [ghstack-poisoned]

facebook-github-bot · 2021-03-25T00:33:59Z

💊 CI failures summary and remediations

As of commit 6ee1560 (more details on the Dr. CI page):

1/1 failures possibly* introduced in this PR
- 1/1 non-scanned failure(s)

This comment was automatically generated by Dr. CI (expand for details).

Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

…a helper function" Summary: Previously we special case copy operator in normal insert observer code, this PR tries to split the special case logic to a separate function and keep the rest of the code clean. Test Plan: Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D27314678](https://our.internmc.facebook.com/intern/diff/D27314678) [ghstack-poisoned]

…nction Summary: Previously we special case copy operator in normal insert observer code, this PR tries to split the special case logic to a separate function and keep the rest of the code clean. Test Plan: Reviewers: Subscribers: Tasks: Tags: ghstack-source-id: ccccaca Pull Request resolved: #54644

torch/quantization/fx/quantize.py

torch/quantization/fx/utils.py

torch/quantization/fx/quantize.py

…a helper function" Summary: Previously we special case copy operator in normal insert observer code, this PR tries to split the special case logic to a separate function and keep the rest of the code clean. Test Plan: Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D27314678](https://our.internmc.facebook.com/intern/diff/D27314678) [ghstack-poisoned]

…nction Summary: Previously we special case copy operator in normal insert observer code, this PR tries to split the special case logic to a separate function and keep the rest of the code clean. Test Plan: Reviewers: Subscribers: Tasks: Tags: ghstack-source-id: 2bb5658 Pull Request resolved: #54644

…a helper function" Summary: Previously we special case copy operator in normal insert observer code, this PR tries to split the special case logic to a separate function and keep the rest of the code clean. Test Plan: Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D27314678](https://our.internmc.facebook.com/intern/diff/D27314678) [ghstack-poisoned]

facebook-github-bot · 2021-04-01T00:50:56Z

This pull request has been merged in c57541c.

…nction (#54644) Summary: Previously we special case copy operator in normal insert observer code, this PR tries to split the special case logic to a separate function and keep the rest of the code clean. Test Plan: Imported from OSS Reviewed By: vkuzo [ghstack-poisoned]

…nction (#54644) Summary: Previously we special case copy operator in normal insert observer code, this PR tries to split the special case logic to a separate function and keep the rest of the code clean. Test Plan: Imported from OSS Reviewed By: vkuzo ghstack-source-id: 48f3ea0 Pull Request resolved: #55429

…opy operator to a helper function (#54644)" Summary: Previously we special case copy operator in normal insert observer code, this PR tries to split the special case logic to a separate function and keep the rest of the code clean. Test Plan: Imported from OSS Reviewed By: vkuzo Differential Revision: [D27609972](https://our.internmc.facebook.com/intern/diff/D27609972) [ghstack-poisoned]

…a helper function (#54644)" Summary: Previously we special case copy operator in normal insert observer code, this PR tries to split the special case logic to a separate function and keep the rest of the code clean. Test Plan: Imported from OSS Reviewed By: vkuzo Differential Revision: [D27609972](https://our.internmc.facebook.com/intern/diff/D27609972) [ghstack-poisoned]