Skip to content

Conversation

@swolchok
Copy link
Contributor

Plan-then-execute for reductions should avoid repeating prepratory work.

[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
@swolchok
Copy link
Contributor Author

swolchok commented Mar 10, 2025

@pytorch-bot
Copy link

pytorch-bot bot commented Mar 10, 2025

🔗 Helpful Links

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

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

❌ 2 Cancelled Jobs

As of commit 514f907 with merge base f680c44 (image):

CANCELLED JOBS - The following jobs were cancelled. Please retry:

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 Mar 10, 2025
@swolchok swolchok requested review from SS-JIA and kimishpatel March 10, 2025 23:53
[ghstack-poisoned]
@swolchok swolchok requested a review from JacobSzwejbka March 11, 2025 17:50
Copy link
Contributor

@JacobSzwejbka JacobSzwejbka left a comment

Choose a reason for hiding this comment

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

Replacements look fine, Im assuming the OP tests will catch anything I didnt see.

[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
Base automatically changed from gh/swolchok/336/head to main March 11, 2025 22:32
@swolchok swolchok merged commit 6f0ae19 into main Mar 11, 2025
49 of 50 checks passed
@swolchok swolchok deleted the gh/swolchok/337/head branch March 11, 2025 22:34
kedarnath03 pushed a commit to kedarnath03/executorch that referenced this pull request Jun 25, 2025
…in kernels/portable

Plan-then-execute for reductions should avoid repeating prepratory work.

ghstack-source-id: f29f001
ghstack-comment-id: 2712063770
Pull Request resolved: pytorch/executorch#9110
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

4 participants