Skip to content

Support polars 1.41.2#95

Open
mccalluc wants to merge 17 commits into
mainfrom
92-extend-polars-range-farther
Open

Support polars 1.41.2#95
mccalluc wants to merge 17 commits into
mainfrom
92-extend-polars-range-farther

Conversation

@mccalluc

@mccalluc mccalluc commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Not needed right now to support OpenDP, but thought it would be easy to address while I'm thinking about it.

@mccalluc mccalluc mentioned this pull request Jun 11, 2026
@cla-bot cla-bot Bot added the cla-signed label Jun 11, 2026
@mccalluc mccalluc marked this pull request as ready for review June 11, 2026 21:19
@mccalluc mccalluc requested a review from eddiestudies June 11, 2026 21:20
Base automatically changed from 92-extend-polars-range to main June 15, 2026 17:35
Comment thread tests/test_misc.py Outdated
@mccalluc mccalluc requested a review from eddiestudies June 15, 2026 18:51
@mccalluc

Copy link
Copy Markdown
Contributor Author

Huh: didn't remember adding anything upstream that would break. Looking into it...

@eddiestudies eddiestudies left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Is this supposed to be removing the recent 1.36.1 changes?

@mccalluc

Copy link
Copy Markdown
Contributor Author

Is this supposed to be removing the recent 1.36.1 changes?

@eddiestudies : No: Can you point at the change that gave you that impression? The biggest deletion block is where it was matching a list of numeric types. The latest polars release adds to that list, and I thought it didn't make sense to add complexity to handle both, and it can just be grabbed in a wildcard... but that might not be what you're looking at.

Comment thread README.md Outdated

- Python versions: Tested against Python 3.10 and 3.13.
- Polars versions: Tested against Polars 1.32.0 and 1.36.1.
- Polars versions: Tested against Polars 1.32.0 and 1.41.2.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This and here made me think we are no longer testing 1.36.1.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sorry for confusion: I meant to just indicate the range being tested, but could just list them all here.

          - '1.32.0' # OpenDP v0.14.1
          - '1.36.1' # OpenDP v0.15.1
          - '1.41.2' # Latest release

I'll make a tweak.

@mccalluc mccalluc requested a review from eddiestudies June 16, 2026 16:01
@mccalluc mccalluc enabled auto-merge (squash) June 16, 2026 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants