Skip to content

[Pallas] Update xfail reasons for examples to reflect current errors#2074

Draft
norx1991 wants to merge 1 commit intomainfrom
yifeixu/pallas-xfail-reason-updates
Draft

[Pallas] Update xfail reasons for examples to reflect current errors#2074
norx1991 wants to merge 1 commit intomainfrom
yifeixu/pallas-xfail-reason-updates

Conversation

@norx1991
Copy link
Copy Markdown
Contributor

@norx1991 norx1991 commented Apr 22, 2026

No description provided.

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Meta Open Source bot. label Apr 22, 2026
The stored xfail reasons on many examples were stale — most dated back
to when the Pallas backend first xfailed them, but the underlying error
has since moved (or was always something different). Ran the full
xfailed list against current origin/main and updated each reason to
match the actual error observed today.

The updates also surface some new clusters worth tracking:

- "int tensor indexing not yet lowered for TPU" (5 tests:
  embedding_pointers/block_ptr, concat, cross_entropy, gather_gemv) —
  all hit lowering.py:1699
- "Pallas rejects int64 inputs (jagged offsets)" (6 tests: all jagged
  kernels) — blocked in _pallas_check_dtypes
- "broadcasting shape mismatch in codegen" (5 tests:
  matmul_layernorm_dynamic_shapes, concat_block_ptr, se_bwd_dx/da,
  grpo_loss_bwd) — TypeError from jax.numpy broadcast
- "Mosaic compile error" (2 tests: attention_dynamic, layernorm_bwd)

No behavior change; xfail markers still apply, only the `reason=` string
is updated.
@norx1991 norx1991 force-pushed the yifeixu/pallas-xfail-reason-updates branch from a4ff6ff to efe0fba Compare April 22, 2026 18:20
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