-
Notifications
You must be signed in to change notification settings - Fork 46
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
Polars: add make_with_columns
#824
base: main
Are you sure you want to change the base?
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.
Exciting to have our first context constructor!
|
||
use super::*; | ||
|
||
fn get_3_row_test_data() -> (LazyFrameDomain, LazyFrame, ExprDomain) { |
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.
Similar comment as in the other PR, make this return a Fallible
so you can replace the .unwrap_test()
s with ?
.
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.
Do you think these test data helper functions could be reused? If so, move them out into a #[cfg(test)]
module in rust/src/transformations/polars/mod.rs
(and mark the functions as pub
so you can use it from these tests).
5ed95ed
to
bb405ea
Compare
bb405ea
to
9cdfeaf
Compare
9cdfeaf
to
8e05b11
Compare
8e05b11
to
995af02
Compare
995af02
to
92c3c0a
Compare
716c969
to
18c7eb4
Compare
92c3c0a
to
af414d8
Compare
bb423fc
to
f83e71e
Compare
9e375ab
to
5200178
Compare
99fab60
to
807cad5
Compare
72e0a4f
to
9aed08e
Compare
No description provided.