-
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
implement <Series>$n_chunks()
method
#999
Conversation
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 for your contribution!
I think they should return an integer, but they may be set to return a double since there is no int64 type in R.
Please complete the roxygen comment and generate the Rd file.
See https://github.com/pola-rs/r-polars/blob/main/DEVELOPMENT.md
n_chunks
method<Series>$n_chunks()
method
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.
Looks good.
Could you update the NEWS?
Sorry, I missed this message :) . Great appreciate for your review |
One issue is the returned type, I have seen
Python
use integer, but I observed$len()
method return a numeric. Is there any reason to return numeric? For R function length, they all have a integer mode