ARROW-12184: [R] Bindings for na.fail, na.omit, na.exclude, na.pass - #10056

Closed

thisisnic wants to merge 26 commits into

apache:masterfrom

thisisnic:ARROW-12184-na.fail

Commits on Apr 21, 2021

Import na.* functions from stats
thisisnic
committed
46775b4
Import na.* functions from stats in namespace
thisisnic
committed
f826a24
Add tests for na.omit, na.exclude, na.fail, and na.pass for ChunkedArray objects
thisisnic
committed
88de5c5
Add na.omit, na.exclude, na.pass, and na.fail functions for ArrowDatum objects
thisisnic
committed
6fc0d5d
Add expect_vector_error and expect_vector_equivalent functions for testing
thisisnic
committed
c180cc8
Remove individual tests for na.* functions in favour of using expect_vector_equal etc
thisisnic
committed
ec75960
Add tests for Table object and na.* functions
thisisnic
committed
74fb6f7
Remove duplicate identical function and remove na.pass
thisisnic
committed
275e19d
Implement na.fail, na.omit, and na.exclude for ArrowTabular objects
thisisnic
committed
fb22de4
Remove redundant na.pass tests, and swap out expect_data_frame for expect_equivalent as failing due to attribute mismatch
thisisnic
committed
c42a91e
Update NAMESPACE
thisisnic
committed
f0716bd
Update function arguments to match generics
thisisnic
committed
3786af4
Add in test for na.fail on Scalar
thisisnic
committed
1b5f4f5
Update function arguments to match generics
thisisnic
committed
5c25e51
Fix spacing in arrow-datum
thisisnic
committed
9ad23dc
Remove unnecessary spacing
thisisnic
committed
c0d9f30
More spacing
thisisnic
committed
ffa9932
Update expect_vector_equal have have ignore_attr argument and expect_vector to use this arg
thisisnic
committed
5b8d300
Reorder vector with NA at end
thisisnic
committed
6c0a699
Refactor na.omit to exclude dplyr use
thisisnic
committed
351bded
Remove unnecessary whitespace
thisisnic
committed
09844a6
Update na.omit to use purrr::map and purrr::reduce instead of doing freaky evaluation things
thisisnic
committed
2712b30
Rename expect_vector to expect_as_vector
thisisnic
committed
8451d3d
Refactor NA count to for loop for efficiency

thisisnic
and
nealrichardson
committed
9d6ada9
Update na.omit to use a single array expression

thisisnic
and
nealrichardson
committed
f113284
Update r/R/arrow-datum.R
nealrichardson
authored
ed52365

Navigation Menu

Uh oh!

ARROW-12184: [R] Bindings for na.fail, na.omit, na.exclude, na.pass - #10056

ARROW-12184: [R] Bindings for na.fail, na.omit, na.exclude, na.pass#10056
thisisnic wants to merge 26 commits into
apache:masterfrom
thisisnic:ARROW-12184-na.fail

Commits on Apr 21, 2021

Import na.* functions from stats

Import na.* functions from stats in namespace

Add tests for na.omit, na.exclude, na.fail, and na.pass for ChunkedArray objects

Add na.omit, na.exclude, na.pass, and na.fail functions for ArrowDatum objects

Add expect_vector_error and expect_vector_equivalent functions for testing

Remove individual tests for na.* functions in favour of using expect_vector_equal etc

Add tests for Table object and na.* functions

Remove duplicate identical function and remove na.pass

Implement na.fail, na.omit, and na.exclude for ArrowTabular objects

Remove redundant na.pass tests, and swap out expect_data_frame for expect_equivalent as failing due to attribute mismatch

Update NAMESPACE

Update function arguments to match generics

Add in test for na.fail on Scalar

Update function arguments to match generics

Fix spacing in arrow-datum

Remove unnecessary spacing

More spacing

Update expect_vector_equal have have ignore_attr argument and expect_vector to use this arg

Reorder vector with NA at end

Refactor na.omit to exclude dplyr use

Remove unnecessary whitespace

Update na.omit to use purrr::map and purrr::reduce instead of doing freaky evaluation things

Rename expect_vector to expect_as_vector

Refactor NA count to for loop for efficiency

Update na.omit to use a single array expression

Update r/R/arrow-datum.R