Skip to content

Revert "Revert "Arm backend: Run adaptive_avg_pool2d before quantization""#17831

Open
rascani wants to merge 1 commit intomainfrom
revert-17595-revert-17494-fix_mv2_channels_last_export
Open

Revert "Revert "Arm backend: Run adaptive_avg_pool2d before quantization""#17831
rascani wants to merge 1 commit intomainfrom
revert-17595-revert-17494-fix_mv2_channels_last_export

Conversation

@rascani
Copy link
Contributor

@rascani rascani commented Mar 3, 2026

Reverts #17595 to essentially reland #17494. Original summary from @gggekov here:

To run mobilenet_v2 with good performance on Ethos-U55, we need to export the model in channels_last. If we export in channels_first (default behaviour), we pay a hefty performance penalty because the Ethos-U55 hardware is not efficient at doing Transpose
(see details in #17157). The adaptive_avg_pool2d operator, part of mv2,
is traced differently by ExecuTorch
depending on whether it was exported in
channels-first(operator not decomposed) or
exported in channels-last
(operator is decomposed by ExecuTorch in to_edge). To work around that, we add adaptive_avg_pool2d
to the transform_for_annotation
pipeline in order to decompose the
operator before quantization.

@rascani rascani requested a review from digantdesai as a code owner March 3, 2026 19:46
@pytorch-bot pytorch-bot bot added the ci-no-td label Mar 3, 2026
@pytorch-bot
Copy link

pytorch-bot bot commented Mar 3, 2026

🔗 Helpful Links

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

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

❌ 16 New Failures, 13 Unrelated Failures

As of commit 9432f16 with merge base 23b2382 (image):

NEW FAILURES - The following jobs have failed:

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

BROKEN TRUNK - The following jobs failed but were 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.

@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 Mar 3, 2026
@github-actions
Copy link

github-actions bot commented Mar 3, 2026

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

@meta-codesync
Copy link
Contributor

meta-codesync bot commented Mar 3, 2026

@rascani has imported this pull request. If you are a Meta employee, you can view this in D95098035.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-no-td ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants