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

[Suggestion] Replace the read.csv(data/gapminder.csv) with library(gapminder) #481

Open
anuj2054 opened this issue Mar 6, 2019 · 2 comments

Comments

Projects
None yet
3 participants
@anuj2054
Copy link

commented Mar 6, 2019

Since gapminder dataset also exist as R package, it might be easier to ask students to install that package and use the dataset just like we would use an iris dataset.

Students have already had experience creating and reading in CSV files in episode 4 with the cats and feline data, so I dont think showing them how to read in CSV files is the point here.

So using this package removes the confusion of having to find the data in the SWC website etc for the students.

install.packages("gapminder")
library(gapminder)
View(gapminder)
@naupaka

This comment has been minimized.

Copy link
Member

commented Mar 7, 2019

Worth considering! In general I think learners are more likely to have to read in data from csv files than getting their data from packages, and more practice doing so is therefore beneficial, but I could potentially be convinced otherwise. Thoughts, @mawds @jcoliver ?

@jcoliver

This comment has been minimized.

Copy link
Collaborator

commented Mar 7, 2019

I agree with @naupaka . While loading in the data from the gapminder package is easier, I do think the reinforcing read.csv through multiple examples is important for our learners, since they will, for the most part, not be dealing with pre-loaded data sets in their own work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.