Skip to content
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

Impute below all #195

Merged
merged 2 commits into from
Aug 12, 2018
Merged

Impute below all #195

merged 2 commits into from
Aug 12, 2018

Conversation

njtierney
Copy link
Owner

Description

This PR changes impute_below to impute_below_all - since it operates on all columns. It also makes impute_below an alias to shadow_shift. More discussion on shadow_shift's deprecation at #193

Related Issue

This fixes #159

Example

library(naniar)

impute_below(c(1,22,3,NA,0.1))
#> [1]  1.000000 22.000000  3.000000 -2.516131  0.100000
impute_below_all(airquality) %>% head()
#>       Ozone   Solar.R Wind Temp Month Day
#> 1  41.00000 190.00000  7.4   67     5   1
#> 2  36.00000 118.00000  8.0   72     5   2
#> 3  12.00000 149.00000 12.6   74     5   3
#> 4  18.00000 313.00000 11.5   62     5   4
#> 5 -19.72321 -33.57778 14.3   56     5   5
#> 6  28.00000 -33.07810 14.9   66     5   6

Created on 2018-08-12 by the reprex package (v0.2.0).

Tests

Yes, tests have been included!

NEWS + DESCRIPTION

Yup!

@njtierney njtierney merged commit 2e4a8f8 into master Aug 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rename impute_below to impute_below_all
1 participant