Skip to content

Conversation

@oscarandersson8218
Copy link
Collaborator

@oscarandersson8218 oscarandersson8218 commented Jan 30, 2025

Summary

The decomposition logic of batchnorm is better suited for a pass than a node visitor.

Change-Id: I859912dc6ed437aa96a3bcdd819531dc8e02230e

cc @digantdesai @freddan80 @per @zingo

@pytorch-bot
Copy link

pytorch-bot bot commented Jan 30, 2025

🔗 Helpful Links

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

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

⏳ No Failures, 1 Pending

As of commit 6c70e74 with merge base 9d4e1ee (image):
💚 Looks good so far! There are no failures yet. 💚

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

@facebook-github-bot facebook-github-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 Jan 30, 2025
@oscarandersson8218 oscarandersson8218 added partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm ciflow/trunk topic: not user facing labels Jan 30, 2025
@zingo
Copy link
Collaborator

zingo commented Jan 31, 2025

Please rebase now that 5f849ca is merged.

The decomposition logic of batchnorm is better suited for a pass than a
node visitor.

Signed-off-by: Oscar Andersson <oscar.andersson@arm.com>
Change-Id: I859912dc6ed437aa96a3bcdd819531dc8e02230e
@zingo zingo merged commit 0966d99 into pytorch:main Jan 31, 2025
106 checks passed
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. partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm topic: not user facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants