Skip to content

Conversation

@jerryzh168
Copy link
Contributor

#40596)

Summary:
Pull Request resolved: #40596

Previously the fusion patterns for {add/mul}_scalar is inconsistent since the op pattern
produces a non-quantized tensor and the op replacement graph produces a quantized tensor

Test Plan: Imported from OSS

Differential Revision: D22251072

fbshipit-source-id: e16eb92cf6611578cca1ed8ebde961f8d0610137

…40596)

Summary:
Pull Request resolved: #40596

Previously the fusion patterns for {add/mul}_scalar is inconsistent since the op pattern
produces a non-quantized tensor and the op replacement graph produces a quantized tensor

Test Plan: Imported from OSS

Differential Revision: D22251072

fbshipit-source-id: e16eb92cf6611578cca1ed8ebde961f8d0610137
@jerryzh168 jerryzh168 requested a review from apaszke as a code owner July 2, 2020 18:58
@facebook-github-bot facebook-github-bot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Jul 2, 2020
@jerryzh168 jerryzh168 added this to the 1.6.0 milestone Jul 2, 2020
Summary:
Pull Request resolved: #40743

`aten::append` modifies input inplace and the output is ignored, these ops are not
supported right now, so we'll need to first make `aten::append` non-inplace
by change
```
ignored = aten::append(list, x)
```
to
```
x_list = aten::ListConstruct(x)
result = aten::add(list, x_list)
```
and then quantize the aten::add instead.

Test Plan:
TestQuantizeJitOps.test_general_shape_ops

Imported from OSS

Differential Revision: D22302151

fbshipit-source-id: 931000388e7501e9dd17bec2fad8a96b71a5efc5
@jerryzh168 jerryzh168 changed the title [quant][graphmode][fix] dequantize propagation for {add/mul}_scalar (… [quant][graphmode][fix] dequantize propagation for {add/mul}_scalar + aten::repeat Jul 3, 2020
@malfet malfet merged commit 6220cc4 into release/1.6 Jul 7, 2020
@facebook-github-bot facebook-github-bot deleted the 1.6.fix1 branch January 27, 2021 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

oncall: jit Add this issue/PR to JIT oncall triage queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants