Skip to content
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

feat!: bump Rust polars to 0.39.0 #1034

Merged
merged 14 commits into from
Apr 14, 2024
Merged

feat!: bump Rust polars to 0.39.0 #1034

merged 14 commits into from
Apr 14, 2024

Conversation

eitsupi
Copy link
Collaborator

@eitsupi eitsupi commented Apr 14, 2024

@eitsupi eitsupi added this to the 0.16 milestone Apr 14, 2024
@eitsupi eitsupi mentioned this pull request Apr 14, 2024
@eitsupi

This comment was marked as outdated.

@eitsupi eitsupi requested a review from etiennebacher April 14, 2024 10:10
@eitsupi eitsupi changed the title feat!: bump Rust polars to 0.39.0 [skip ci] feat!: bump Rust polars to 0.39.0 Apr 14, 2024
@eitsupi
Copy link
Collaborator Author

eitsupi commented Apr 14, 2024

arr().max() seems buggy.
I created an upstream issue pola-rs/polars#15646.

@eitsupi eitsupi force-pushed the rust-polars-0.39.0 branch from ddac68a to bcf5e2e Compare April 14, 2024 11:45
@eitsupi
Copy link
Collaborator Author

eitsupi commented Apr 14, 2024

This is a pretty tough update.......

I would appreciate it if someone could make the subsequent corrections as I will not be able to spend any more time on this for the next few days.

@etiennebacher
Copy link
Collaborator

Thanks, I'll take a look later

@eitsupi

This comment was marked as resolved.

@eitsupi

This comment was marked as resolved.

@etiennebacher etiennebacher marked this pull request as ready for review April 14, 2024 20:36
@etiennebacher etiennebacher merged commit 7986aaa into main Apr 14, 2024
37 checks passed
@etiennebacher etiennebacher deleted the rust-polars-0.39.0 branch April 14, 2024 20:37
@eitsupi
Copy link
Collaborator Author

eitsupi commented Apr 14, 2024

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants