Skip to content

[Pallas] Fix failing scratch shapes asserts due to land-time race when #2278 caused scratch shapes to be re-ordered#2302

Merged
AmesingFlank merged 1 commit into
mainfrom
AmesingFlank/stack/44
May 6, 2026
Merged

[Pallas] Fix failing scratch shapes asserts due to land-time race when #2278 caused scratch shapes to be re-ordered#2302
AmesingFlank merged 1 commit into
mainfrom
AmesingFlank/stack/44

Conversation

@AmesingFlank
Copy link
Copy Markdown
Contributor

@AmesingFlank AmesingFlank commented May 6, 2026

The PRs #2278 and #2223 had a land-time race. #2223 introduced new
scratch shape asserts, while #2278 caused scratch shapes to be
re-ordered. No real breakage, but the asserted shapes needs to be
re-ordered.

…#2278 caused scratch shapes to be re-ordered

The PRs #2278 and #2223 had a land-time race. #2223 introduced new
scratch shape asserts, while #2278 caused scratch shapes to be
re-ordered. No real breakage, but the asserted shapes needs to be
re-ordered.

stack-info: PR: #2302, branch: AmesingFlank/stack/44
@AmesingFlank AmesingFlank force-pushed the AmesingFlank/stack/44 branch from 4ab19e2 to 10123e9 Compare May 6, 2026 05:10
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Meta Open Source bot. label May 6, 2026
@AmesingFlank AmesingFlank requested a review from norx1991 May 6, 2026 05:11
@AmesingFlank AmesingFlank merged commit a392490 into main May 6, 2026
22 of 24 checks passed
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 Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants