Fix quantized max_pool2d output observer for tuple-returning ops (#19259)#19259
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/19259
Note: Links to docs will display an error until the docs builds have been completed. ❌ 3 New Failures, 1 Cancelled Job, 3 Unrelated FailuresAs of commit 1317bf1 with merge base 8464b47 ( NEW FAILURES - The following jobs have failed:
CANCELLED JOB - The following job was cancelled. Please retry:
FLAKY - The following job failed but was likely due to flakiness present on trunk:
BROKEN TRUNK - The following jobs failed but was present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
@abeakkas has exported this pull request. If you are a Meta employee, you can view the originating Diff in D103436172. |
This PR needs a
|
…orch#19259) Summary: prepare_pt2e skips the output observer for max_pool2d_with_indices because its output is a tuple, not a single tensor. This caused the quantized_max_pool2d fusion to silently fail. Fix: annotate getitem[0] (single tensor) instead of the tuple-returning op. Also bail out if indices are consumed or the graph structure is unexpected. Reviewed By: mcremon-meta Differential Revision: D103436172
d69567f to
d7184cb
Compare
…orch#19259) Summary: prepare_pt2e skips the output observer for max_pool2d_with_indices because its output is a tuple, not a single tensor. This caused the quantized_max_pool2d fusion to silently fail. Fix: annotate getitem[0] (single tensor) instead of the tuple-returning op. Also bail out if indices are consumed or the graph structure is unexpected. Reviewed By: mcremon-meta Differential Revision: D103436172
d7184cb to
08c7ef6
Compare
…orch#19259) Summary: prepare_pt2e skips the output observer for max_pool2d_with_indices because its output is a tuple, not a single tensor. This caused the quantized_max_pool2d fusion to silently fail. Fix: annotate getitem[0] (single tensor) instead of the tuple-returning op. Also bail out if indices are consumed or the graph structure is unexpected. Reviewed By: mcremon-meta Differential Revision: D103436172
08c7ef6 to
e4cde86
Compare
…orch#19259) Summary: prepare_pt2e skips the output observer for max_pool2d_with_indices because its output is a tuple, not a single tensor. This caused the quantized_max_pool2d fusion to silently fail. Fix: annotate getitem[0] (single tensor) instead of the tuple-returning op. Also bail out if indices are consumed or the graph structure is unexpected. Reviewed By: mcremon-meta Differential Revision: D103436172
e4cde86 to
debb30d
Compare
…orch#19259) Summary: prepare_pt2e skips the output observer for max_pool2d_with_indices because its output is a tuple, not a single tensor. This caused the quantized_max_pool2d fusion to silently fail. Fix: annotate getitem[0] (single tensor) instead of the tuple-returning op. Also bail out if indices are consumed or the graph structure is unexpected. Reviewed By: mcremon-meta Differential Revision: D103436172
debb30d to
1317bf1
Compare
Summary:
prepare_pt2e skips the output observer for max_pool2d_with_indices because its output is a tuple, not a single tensor. This caused the quantized_max_pool2d fusion to silently fail.
Fix: annotate getitem[0] (single tensor) instead of the tuple-returning op. Also bail out if indices are consumed or the graph structure is unexpected.
Reviewed By: mcremon-meta
Differential Revision: D103436172