Skip to content

[CI] Parity: exclude inductor.test_cpu_repro from comparison#3363

Merged
jithunnair-amd merged 2 commits into
developfrom
ethanwee/parity-exclude-cpu-repro
Jun 24, 2026
Merged

[CI] Parity: exclude inductor.test_cpu_repro from comparison#3363
jithunnair-amd merged 2 commits into
developfrom
ethanwee/parity-exclude-cpu-repro

Conversation

@ethanwee1

Copy link
Copy Markdown

Split out of #3210 (2 of 3) for easier review.

inductor.test_cpu_repro is blocklisted on ROCm upstream, so it never runs there and would otherwise show up as falsely MISSED against the CUDA baseline (~791 phantom MISSED rows on a recent mi350 run). Add it to EXCLUDED_TEST_SUITES so it is dropped from the ROCm-vs-CUDA comparison.

Verified against a real mi350 run: removes all inductor.test_cpu_repro rows; MISSED gap drops ~1,294 -> ~503 and DEFAULT disagreement 3.03% -> 2.77% (SKIPPED unchanged).

Made with Cursor

test_cpu_repro is blocklisted on ROCm upstream, so it never runs there.
The parity summary was therefore counting it as MISSED against the CUDA
baseline, which is misleading. Add it to EXCLUDED_TEST_SUITES so it is
dropped from the ROCm-vs-CUDA comparison.
@rocm-repo-management-api

Copy link
Copy Markdown

Jenkins build for d16e30ed777040c3a320dd6b1d33b330f597e8e6 commit is in progress
Links: Pipeline Overview / Build artifacts

@jithunnair-amd jithunnair-amd merged commit 93a1a68 into develop Jun 24, 2026
@jithunnair-amd jithunnair-amd deleted the ethanwee/parity-exclude-cpu-repro branch June 24, 2026 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants