[DO NOT MERGE - only for CI/artifacts] cudf-polars testing #16492
Conversation
This file contains 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
Use new polars engine config object in cudf-polars callback
github-actions
bot
added
Python
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## branch-24.08 #16492 +/- ##
================================================
- Coverage 83.28% 83.28% -0.01%
================================================
Files 184 184
Lines 28318 28318
================================================
- Hits 23586 23585 -1
- Misses 4732 4733 +1 ☔ View full report in Codecov by Sentry. |
## Description <!-- Provide a standalone description of changes in this PR. --> <!-- Reference any issues closed by this PR with "closes #1234". --> <!-- Note: The pull request title will be included in the CHANGELOG. --> Adapts to IR changes in polars 1.4 and handles nrows/skiprows a little more correctly. ## Checklist - [ ] I am familiar with the [Contributing Guidelines](https://github.com/rapidsai/cudf/blob/HEAD/CONTRIBUTING.md). - [ ] New or existing tests cover these changes. - [ ] The documentation is up to date with these changes. --------- Co-authored-by: Lawrence Mitchell <lmitchell@nvidia.com>
Add support for ``pl.col.str.replace`` and ``pl.col.str.replace_many`` Authors: - Thomas Li (https://github.com/lithomas1) Approvers: None URL: #16039
|
This PR exists solely to produce builds of the latest version of cudf-polars until the release because we do not have branch builds set up on the cudf-polars feature branch. This PR can be closed after the release and should never be merged. |
lithomas1
changed the title
…sts (#16441) (#16605) Authors: - Thomas Li (https://github.com/lithomas1) - https://github.com/brandon-b-miller Approvers: - Vyas Ramasubramani (https://github.com/vyasr) URL: #16605
contributes to #16478 This implements "cum_min", "cum_max", "cum_prod", "cum_sum" "cum_count" is not implemented for now, since there's no exact libcudf match (I imagine the non-grouped case is also not used that much but haven't checked). I suppose we could implement it by creating a column of 1s and copying the null mask over, and doing a cum_sum on that. Let me know if you want to try that. Authors: - Thomas Li (https://github.com/lithomas1) Approvers: - https://github.com/brandon-b-miller URL: #16509
Use a key column rather than a placeholder for count agg
## Description - Closes #16477
|
Closing since the CI auto-trigger doesn't work for me anymore :(. Someone else should do a new pull request. cc @vyasr |
|
Thanks for the ping! The new PR is #16680 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
DO NOT MERGE, just to produce CI artifacts.
Checklist