Skip to content

Conversation

@rascani
Copy link
Contributor

@rascani rascani commented Feb 11, 2026

Summary

The test expected 1 dequantize/quantize_per_tensor op each after transforms, but the graph actually produces 2. The extra pair is a requantization boundary between avg_pool2d and linear caused by an unannotated clone_dim_order node blocking FoldAndAnnotateQParamsPass from folding through the view_copy -> clone_dim_order -> linear chain.

This PR was authored with Claude.

Test plan

pytest "backends/cortex_m/test/models/test_mobilenet_v2.py::test_dialect_mv2[mobilenet_v2]"

The test expected 1 dequantize/quantize_per_tensor op each after
transforms, but the graph actually produces 2. The extra pair is a
requantization boundary between avg_pool2d and linear caused by an
unannotated clone_dim_order node blocking FoldAndAnnotateQParamsPass
from folding through the view_copy -> clone_dim_order -> linear chain.

This PR was authored with Claude.
@rascani rascani requested a review from psiddh February 11, 2026 23:28
@pytorch-bot
Copy link

pytorch-bot bot commented Feb 11, 2026

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/17405

Note: Links to docs will display an error until the docs builds have been completed.

❌ 2 New Failures, 16 Pending, 1 Unrelated Failure

As of commit f5428f4 with merge base f08db65 (image):

NEW FAILURES - The following jobs have failed:

FLAKY - The following job failed but was likely due to flakiness present on trunk:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. release notes: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants