Skip to content

Commit a5a9621

Browse files
authored
Relax XNNPACK quantized convolution test tolerances (#19479)
### Summary XNNPACK quantized convolution tests are failing sporadically. When tests run multiple convolutions back to back, the output can differ from eager by more than one quant value. This is reasonable, so bump the tolerance to 2 times the output quant step.
1 parent 23a91d5 commit a5a9621

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

backends/xnnpack/test/ops/test_conv2d.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -245,7 +245,7 @@ def _test(
245245
.check_count({"torch.ops.higher_order.executorch_call_delegate": 1})
246246
.to_executorch()
247247
.serialize()
248-
.run_method_and_compare_outputs(qtol=1)
248+
.run_method_and_compare_outputs(qtol=2)
249249
)
250250

251251
def _test_dq(
@@ -276,7 +276,7 @@ def _test_dq(
276276
tester.check_not(["executorch_exir_dialects_edge__ops_aten_conv2d_default"])
277277
tester.to_executorch()
278278
tester.serialize()
279-
tester.run_method_and_compare_outputs(qtol=1)
279+
tester.run_method_and_compare_outputs(qtol=2)
280280

281281
def test_fp16_conv2d(self) -> None:
282282
for transpose in (True, False):

0 commit comments

Comments
 (0)