-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
ARROW-11136: [R] Bindings for is.nan #9167
Conversation
r/tests/testthat/test-dataset.R
Outdated
expect_equivalent( | ||
ds %>% | ||
select(part, dbl) %>% | ||
filter(!is.nan(dbl), part == 1) %>% |
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.
I'm not sure this is a great test because the NaN you added looks like it is in part == 2
@@ -111,7 +111,8 @@ cast_array_expression <- function(x, to_type, safe = TRUE, ...) { | |||
|
|||
.unary_function_map <- list( | |||
"!" = "invert", | |||
"is.na" = "is_null" | |||
"is.na" = "is_null", | |||
"is.nan" = "is_nan" |
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.
It's beautiful that this is the only change needed to support this in the dplyr code.
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.
Yeah, absolutely! There's more testing in this PR than operational code :)
Co-authored-by: Neal Richardson <neal.p.richardson@gmail.com>
Closes apache#9167 from jonkeane/ARROW-11136-is.nan Authored-by: Jonathan Keane <jkeane@gmail.com> Signed-off-by: Neal Richardson <neal.p.richardson@gmail.com>
No description provided.