Skip to content

[executorch][cuda] Test CudaExecutor on every step width up to 65 - #24011

Merged
Gasoonjia merged 1 commit into
gh/Gasoonjia/58/origfrom
gh/Gasoonjia/59/orig
Oct 20, 2026
Merged

Gasoonjia merged 1 commit into
gh/Gasoonjia/58/origfrom
gh/Gasoonjia/59/orig

Conversation

@Gasoonjia

Copy link
Copy Markdown
Contributor

This PR was created by the merge bot to help merge the original PR into the main branch.
ghstack PR number: #23819 by @Gasoonjia
^ Please use this as the source of truth for the PR details, comments, and reviews
ghstack PR base: https://github.com/pytorch/executorch/tree/gh/Gasoonjia/59/base
ghstack PR head: https://github.com/pytorch/executorch/tree/gh/Gasoonjia/59/head
Merge bot PR base: https://github.com/pytorch/executorch/tree/gh/Gasoonjia/58/orig
Merge bot PR head: https://github.com/pytorch/executorch/tree/gh/Gasoonjia/59/orig
Differential Revision: D125110873
@diff-train-skip-merge

Pull Request resolved: #23819

CudaExecutor chooses the narrowest static forward_{N} that holds a step, pads unused rows into the KV scratch cell without returning their logits, and uses forward_others with slicing for wider steps. Extend the toy artifact to the Muse-sized method set forward_{1,2,4,...,64} plus forward_others, with eight independently referenced prompts and continuations.

EveryStepWidthUpTo65MatchesEagerGreedy covers every width 1..65 plus 96 and 97, shuffled repeatedly, with CUDA graphs both on and off. Mix sequence owners and chunk lengths, teacher-force eager reference histories, and vary which chunks request output. Assert the selected method sequence, exact output ownership/count and eager greedy token at each requested position. This is correctness coverage of static padding, dynamic widths and slicing, without serving performance reporting.
ghstack-source-id: 451855103
@exported-using-ghexport

Differential Revision: [D125110873](https://our.internmc.facebook.com/intern/diff/D125110873/)
@pytorch-bot

pytorch-bot Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

🔗 Helpful Links

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

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

⏳ No Failures, 191 Pending

As of commit 0be3c22 with merge base 897d28f (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 Oct 20, 2026
@Gasoonjia
Gasoonjia added this pull request to stack #24012 October 20, 2026 03:42
@Gasoonjia
Gasoonjia merged commit b1beef3 into main Oct 20, 2026
260 of 274 checks passed
@Gasoonjia
Gasoonjia deleted the gh/Gasoonjia/59/orig branch October 20, 2026 03:43
@Gasoonjia
Gasoonjia deployed to upload-benchmark-results October 20, 2026 09:46 — with GitHub Actions Active

This branch was successfully deployed

2 active deployments
upload-benchmark-results — 0be3c227 Deployed Oct 20, 2026 by Gasoonjia via upload-benchmark-results #21108
cadence — 0be3c227 Deployed Oct 20, 2026 by Gasoonjia via hifi-op-test / hifi4 #34143
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.

2 participants