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

Double warning for all-unique, not-ignored columns #1222

Open
michaellevy opened this issue Aug 13, 2018 · 1 comment
Open

Double warning for all-unique, not-ignored columns #1222

michaellevy opened this issue Aug 13, 2018 · 1 comment

Comments

@michaellevy
Copy link
Contributor

Only the warning from prep_data should be surfaced.

suppressPackageStartupMessages(library(healthcareai))
data.frame(x = letters, y = 1:26) %>% 
  prep_data()
#> Warning in find_columns_to_ignore(d, c(rlang::quo_name(outcome), ignored)):
#> The following column(s) have a unique value for every row so will be
#> ignored: x
#> Warning in prep_data(.): The following variable(s) look a lot like
#> identifiers: They are character-type and have a unique value on every row.
#> They will be ignored: x
#> Training new data prep recipe...
#> , with no outcome variable specified
#> healthcareai-prepped data. Recipe used to prepare data:
#> Data Recipe
#> 
#> Inputs:
#> 
#>       role #variables
#>  predictor          1
#> 
#> Training data contained 26 data points and no missing data.
#> 
#> Operations:
#> 
#> Sparse, unbalanced variable filter removed no terms [trained]
#> Mean Imputation for y [trained]
#> Current data:
#> # A tibble: 26 x 2
#>    x         y
#>    <fct> <int>
#>  1 a         1
#>  2 b         2
#>  3 c         3
#>  4 d         4
#>  5 e         5
#>  6 f         6
#>  7 g         7
#>  8 h         8
#>  9 i         9
#> 10 j        10
#> # ... with 16 more rows

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

@jeremykohn
Copy link

@michaellevy
Hi, I'm new to the project and looking for a good first issue to work on. Is this issue available?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants