-
Notifications
You must be signed in to change notification settings - Fork 35
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
support extendr_polars #326
Conversation
Merge remote-tracking branch 'origin/main' into extendr_polars # Conflicts: # R/extendr-wrappers.R # src/rust/src/rlib.rs
this update is needed to support the crate extendr_polars
an example R package is found here (test examples) Rust side use example |
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.
@sorhawell I can approve if you want but I have no clue if this is good or if there are mistake. I suppose it works fine since you made some tests repos using it, but maybe @eitsupi would be more comfortable reviewing this?
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.
Sorry, I don't have enough knowledge to review this.......
@etiennebacher arrow-protocol is really hairy ^^ I have looked in the sparse official arrow docs for days to get an ok understanding. I have added a round trip unit test with b38f4d8 and added some notes for what is going on and when to use this. |
Thanks! |
This PR will add support for
extendr_polars
which allows rust extensions to polars in R. Examples