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

docs(python): warn that "by" argument must be sorted for results to be correct in rolling_* functions #11013

Merged
merged 1 commit into from
Sep 10, 2023

Conversation

MarcoGorelli
Copy link
Collaborator

@MarcoGorelli MarcoGorelli commented Sep 8, 2023

reference: #10991

I'm short on time in the coming weeks so won't have time to work on the suggested solution in #10991 (comment) before the next release

Does anyone have time to implement it for 0.19.3? I'm travelling and so don't have time for anything non-trivial at the moment I'm afraid

If not, then I'd appreciate it if either this or #11002 could make it in for 0.19.3

Polars downloads are exploding and I'm worried that someone might get wrong results and lose faith in the library

@github-actions github-actions bot added documentation Improvements or additions to documentation python Related to Python Polars labels Sep 8, 2023
Copy link
Member

@stinodego stinodego left a comment

Choose a reason for hiding this comment

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

Sure, good to have this warning in there for now! Can revisit later.

@stinodego stinodego merged commit 0f37edf into pola-rs:main Sep 10, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation python Related to Python Polars
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants