-
Notifications
You must be signed in to change notification settings - Fork 933
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump polars version to <1.24 #18076
Bump polars version to <1.24 #18076
Conversation
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. |
/ok to test |
@@ -197,7 +197,6 @@ def pytest_configure(config: pytest.Config) -> None: | |||
"tests/unit/io/test_multiscan.py::test_include_file_paths[scan_csv-write_csv]": "Need to expose include_file_paths xref: cudf#18012", | |||
"tests/unit/streaming/test_streaming_io.py::test_parquet_eq_statistics[False]": "Debug output on stderr doesn't match", | |||
# Maybe flaky, order-dependent? | |||
"tests/unit/test_projections.py::test_schema_full_outer_join_projection_pd_13287": "Order-specific result check, query is correct but in different order", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: Polars doesn't check the row order, so this passes. This is not dependent on #17698.
…imp/polars/bump-polars-version
/ok to test |
Tests are failing because polars-cloud (a dependency of polars) currently only supports 1.22. So polars==1.22 is being installed in the CI job instead of polars 1.23 |
/ok to test |
/ok to test |
/ok to test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixing one typo then approving.
/ok to test |
/merge |
b8ec71a
into
rapidsai:branch-25.04
Description
The PR upgrades the Polars version to 1.23.
Checklist