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

Update Polars to v0.36 #804

Merged
merged 5 commits into from
Jan 8, 2024
Merged

Update Polars to v0.36 #804

merged 5 commits into from
Jan 8, 2024

Conversation

philss
Copy link
Contributor

@philss philss commented Jan 8, 2024

This PR includes the tasks listed in #797

philss and others added 5 commits January 3, 2024 21:06
* change counts to count

* fix nan comparision

* fix window_median & quantile
* fix describe

* review changes

* implement describe in elixir

* Update lib/explorer/data_frame.ex

Co-authored-by: José Valim <jose.valim@gmail.com>

---------

Co-authored-by: Philip Sampaio <philip.sampaio@gmail.com>
Co-authored-by: José Valim <jose.valim@gmail.com>
@philss
Copy link
Contributor Author

philss commented Jan 8, 2024

Since we reviewed must of it in other PRs, I think it's safe to merge.
LMK if you find something and I can fix later in the main.

@josevalim
Copy link
Member

Awesome work @lkarthee !!!!

@philss philss merged commit 1285dc6 into main Jan 8, 2024
4 checks passed
@philss philss deleted the ps-bump-polars-to-v0.36 branch January 8, 2024 21:11
@philss
Copy link
Contributor Author

philss commented Jan 8, 2024

Yeaah, thank you for the work here, @lkarthee!! :D

@lkarthee
Copy link
Member

lkarthee commented Jan 9, 2024

Thank you @josevalim @philss @billylanchantin for helping me. It was a good learning experience.

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.

3 participants