Run cudf_polars unit tests with RapidsMPF - #21485
Closed
mroeschke wants to merge 63 commits into
Closed
Conversation
mroeschke
added
improvement
|
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. |
Contributor
Author
|
/ok to test 8fb8b75 |
Matt711
reviewed
Feb 19, 2026
github-actions
Bot
added
Python
Contributor
Author
|
/ok to test 8623f22 |
Contributor
Author
|
/ok to test fea7ee9 |
Contributor
Author
|
/ok to test 6816244 |
Contributor
Author
|
/ok to test e6a7f80 |
Contributor
Author
|
/ok to test f10b5f5 |
Contributor
Author
|
/ok to test b29fc5e |
Contributor
Author
|
/ok to test 4997229 |
Contributor
Author
|
/ok to test 26f0fe1 |
Contributor
Author
|
/ok to test 384f878 |
Contributor
Author
|
/ok to test d2095ae |
Contributor
Author
|
/ok to test d74ba8b |
3 tasks
Contributor
Author
|
/ok to test d19b49f |
rapids-bot Bot
pushed a commit
that referenced
this pull request
Mar 16, 2026
Follow up to #21787 Discovered on my cudf_polars unit tests with rapidsmpf branch #21485 Also folds in a change in that PR where a stream wasn't passed to a `plc.Scalar.to_py` call Authors: - Matthew Roeschke (https://github.com/mroeschke) Approvers: - Tom Augspurger (https://github.com/TomAugspurger) URL: #21798
Contributor
Author
|
/ok to test 8f642f3 |
3 tasks
rapids-bot Bot
pushed a commit
that referenced
this pull request
Mar 24, 2026
Broken off from #21485 This PR takes a different approach from the linked PR by modifying an existing invocation in `ci/run_cudf_polars_with_rapidsmpf_pytests.sh` to run all cudf_polars unit tests with rapidsmpf in an existing non-blocking CI job (`wheel-tests-cudf-polars-with-rapidsmpf`) _instead_ of the blocking `wheel-tests-cudf-polars` and `conda-python-other-tests` jobs. We will want to eventually pursue the latter approach once rapidsmpf development is more stable relative to cudf_polars usage. This PR also aggressively xfails and skips tests known to fail in #21467 Authors: - Matthew Roeschke (https://github.com/mroeschke) Approvers: - Bradley Dice (https://github.com/bdice) URL: #21807
Contributor
Author
|
Superseding the remaining work with #22185 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
xref #21467
This PR removes the separate job used that used to run cudf_polars with RapidsMPF and moves it to
ci/run_cudf_polars_pytests.shwhere all the cudf_polars unit tests are run with the future, default configurations described in the parent issueChecklist