-
Notifications
You must be signed in to change notification settings - Fork 37
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
Rename $apply()
and $map()
to $map_elements()
and $map_batches()
#534
Conversation
Feel free to drop any example from map/apply you don't see necessary. Some of these examples I think take a little too long to run CI-wise. |
Yes but they are still useful I think, maybe I'll move them in a "tips and tricks" vignette with your examples for |
@eitsupi sorry I forgot again, should I bump r-polars version in cargo.toml to 0.36.0 or 0.35.1? (I did 0.36.0 for now) |
Actually, we are only looking at different version numbers here, so it should not be the same number, but 0.35.1 is a good choice to match the minor version of rust-polars. r-polars/dev/generate-lib-sums.R Line 32 in 2fbf67d
|
…lars into deprecate-apply-map
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.
Thanks!
Close #512
We haven't implemented
$apply()
for DataFrame or Groupby yet. I also cleaned the docs of both methods.