Skip to content

Find tests that are skipped in every configuration#1969

Draft
mdboom wants to merge 6 commits intoNVIDIA:mainfrom
mdboom:skipped-test-analysis
Draft

Find tests that are skipped in every configuration#1969
mdboom wants to merge 6 commits intoNVIDIA:mainfrom
mdboom:skipped-test-analysis

Conversation

@mdboom
Copy link
Copy Markdown
Contributor

@mdboom mdboom commented Apr 23, 2026

See #1903

@copy-pr-bot
Copy link
Copy Markdown
Contributor

copy-pr-bot Bot commented Apr 23, 2026

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions github-actions Bot added the CI/CD CI/CD infrastructure label Apr 23, 2026
@mdboom
Copy link
Copy Markdown
Contributor Author

mdboom commented Apr 23, 2026

/ok to test

@github-actions
Copy link
Copy Markdown

@leofang
Copy link
Copy Markdown
Member

leofang commented Apr 23, 2026

Universally-skipped tests

No test run logs found.

@mdboom
Copy link
Copy Markdown
Contributor Author

mdboom commented Apr 23, 2026

/ok to test

@mdboom
Copy link
Copy Markdown
Contributor Author

mdboom commented Apr 23, 2026

Universally-skipped tests

No test run logs found.

Yep. So much for cowboy coding.

@github-actions github-actions Bot added the cuda.bindings Everything related to the cuda.bindings module label Apr 23, 2026
@mdboom
Copy link
Copy Markdown
Contributor Author

mdboom commented Apr 24, 2026

/ok to test

@mdboom
Copy link
Copy Markdown
Contributor Author

mdboom commented Apr 24, 2026

/ok to test

@mdboom
Copy link
Copy Markdown
Contributor Author

mdboom commented Apr 24, 2026

I added a test that always skips. It's not getting reported. So this still isn't working.

@mdboom
Copy link
Copy Markdown
Contributor Author

mdboom commented Apr 24, 2026

/ok to test

@rparolin
Copy link
Copy Markdown
Collaborator

This is cool.

Comment thread .github/workflows/ci.yml Outdated
fi

# Start intersection from the first wheel configuration, then narrow.
cp "${WORKDIR}/configs/${configs[0]}/skipped.txt" "${WORKDIR}/intersection.txt"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

My $0.02: At this complexity, I'd looking at shelling out to python instead of trying to do this much data wrangling in the shell.

Copy link
Copy Markdown
Contributor Author

@mdboom mdboom Apr 27, 2026

Choose a reason for hiding this comment

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

Yeah - I totally agree. This is vibe-coded -- I think my next ask of it is to convert all this to Python. It also isn't doing the right thing. I think Python will make it easier to understand why -- and maybe the agent as well.

@mdboom
Copy link
Copy Markdown
Contributor Author

mdboom commented Apr 27, 2026

/ok to test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD CI/CD infrastructure cuda.bindings Everything related to the cuda.bindings module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants