Skip to content

Conversation

@SaoirseARM
Copy link
Collaborator

@SaoirseARM SaoirseARM commented Jan 20, 2026

Summary

When using groups, we need to update the count for grouped Conv2D layers.

cc @freddan80 @per @zingo @oscarandersson8218 @digantdesai

- When using groups, we need to update the count layers.

Signed-off-by: Saoirse Stewart <saoirse.stewart@arm.com>
@SaoirseARM SaoirseARM requested a review from zingo January 20, 2026 10:56
@pytorch-bot
Copy link

pytorch-bot bot commented Jan 20, 2026

🔗 Helpful Links

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

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

✅ No Failures

As of commit f29aded with merge base 176c162 (image):
💚 Looks good so far! There are no failures yet. 💚

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 Jan 20, 2026
@SaoirseARM
Copy link
Collaborator Author

@pytorchbot label "release notes: none"

@pytorch-bot pytorch-bot bot added the release notes: none Do not include this in the release notes label Jan 20, 2026
@SaoirseARM
Copy link
Collaborator Author

@pytorchbot label ciflow/trunk

@SaoirseARM
Copy link
Collaborator Author

@pytorchbot label 'partner: arm'

@pytorch-bot pytorch-bot bot added the partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm label Jan 20, 2026
Copy link
Collaborator

@zingo zingo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK to merge is Arm tests pass!

@zingo
Copy link
Collaborator

zingo commented Jan 22, 2026

Rebased to re-test after a cortex-m fix was merged

@SaoirseARM SaoirseARM merged commit 59aa69d into pytorch:main Jan 22, 2026
300 checks passed
@SaoirseARM SaoirseARM deleted the toupstream/group_conv branch January 22, 2026 11:10
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 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