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

Consider using read_csv() rather than using read.csv() #480

Closed
Br-Johnson opened this issue Oct 19, 2018 · 2 comments
Closed

Consider using read_csv() rather than using read.csv() #480

Br-Johnson opened this issue Oct 19, 2018 · 2 comments
Labels
status:refer to cac Curriculum Advisory Committee input needed type:discussion Discussion or feedback about the lesson

Comments

@Br-Johnson
Copy link

In the 'starting-with-r' lesson, is there a reason not to use read_csv() as opposes to read.csv()? I suppose it comes down to whether or not the lessons want to introduce the readr package, and the tidyverse early on. read_csv() is much faster for large datasets compared to read.csv(), read_csv() also treats strings more consistently. Using read_csv() you can also point directly to urls, perhaps obviating the need for download.file().

@anacost anacost added status:refer to cac Curriculum Advisory Committee input needed type:discussion Discussion or feedback about the lesson labels Oct 22, 2018
@anacost
Copy link
Contributor

anacost commented Oct 22, 2018

Hi @Br-Johnson, thank you for your comment.
I disagree about treating strings more consistently. read.csv() has default to read strings as factors, which is more valuable for classification of data.
In read.csv() you can also point directly to URL (https://stat.ethz.ch/R-manual/R-devel/library/utils/html/read.table.html).

As mentioned in (swcarpentry/r-novice-gapminder#442), this should be a discussion for the Advisory Committee.

@mondorescue
Copy link
Contributor

@Br-Johnson Your comment is duly noted. We are working on changing over to using tidyverse hence using read_csv() consistently throughout the lesson (see issue #335).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:refer to cac Curriculum Advisory Committee input needed type:discussion Discussion or feedback about the lesson
Projects
None yet
Development

No branches or pull requests

3 participants