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

Warning mentions vroom() instead of read_csv() #1477

Open
presnell opened this issue Mar 2, 2023 · 1 comment
Open

Warning mentions vroom() instead of read_csv() #1477

presnell opened this issue Mar 2, 2023 · 1 comment
Labels
bug an unexpected problem or unintended behavior read 📖

Comments

@presnell
Copy link

presnell commented Mar 2, 2023

Unless I explained it to them beforehand, I suspect that my students might be confused by the warning produced by the code chunk below (taken directly from section 8.3.2 of r4ds2e). Isn't the appearance of vroom(dat) a surprise, given that the innocent user of readr::read_csv() has probably never heard of vroom?

csv <- "
  x
  10
  .
  20
  30"
df <- read_csv(csv, col_types = list(x = col_double()))
@hadley
Copy link
Member

hadley commented Jul 31, 2023

Actual reprex:

library(readr)

csv <- "
  x
  10
  .
  20
  30"
df <- read_csv(csv, col_types = list(x = col_double()))
#> Warning: One or more parsing issues, call `problems()` on your data frame for details,
#> e.g.:
#>   dat <- vroom(...)
#>   problems(dat)

Created on 2023-07-31 with reprex v2.0.2

@hadley hadley changed the title Confusing warning? Warning mentions vroom() instead of read_csv() Jul 31, 2023
@hadley hadley added bug an unexpected problem or unintended behavior read 📖 labels Jul 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug an unexpected problem or unintended behavior read 📖
Projects
None yet
Development

No branches or pull requests

2 participants