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

Cuts to remaining episodes #34

Merged
merged 2 commits into from
Jul 10, 2018
Merged

Cuts to remaining episodes #34

merged 2 commits into from
Jul 10, 2018

Conversation

ErinBecker
Copy link
Contributor

Makes suggested cuts as in #22 to:

  • writing good software
  • manipulating dataframes with dplyr
  • creating publication-quality graphics with ggplot

Also rebuilds site to show these changes.

Replaces:

@lwasser
Copy link
Member

lwasser commented Jun 13, 2018

this looks good to me too. i'd like to see what others think. my only question -t he last episode on writing good software seems important even if it's not explicitly taught. i wonder about leaving as an optional review lesson. there isnt time for all of these lessons however in 2 days with the geospatial stuff regardless :) it's just optional for folks. thoughts everyone?

@ErinBecker
Copy link
Contributor Author

ErinBecker commented Jun 13, 2018

@lwasser - thanks for the thoughts! My recommendation is to avoid "optional lessons" whenever possible, as we want to provide a clear pathway for what is the "canonical workshop" for this curriculum. The writing good software episode was written with the perspective that people in the audience would be doing just that (writing their own software packages), but I don't think that's the expectation we have of these learners (if it is please correct me!).

From experience with other lessons (particularly some of the older SWC lessons), allowing many "optional episodes" in the lesson leads to lesson bloat and makes it very difficult for new Instructors to decide on what they should actually teach. Often this leads to them trying to teach everything, which makes the lessons go way to fast and is demotivating. This is feedback we've gotten a lot from Instructors on discussion sessions and also from observing workshops.

The main idea with cutting down these R lessons so heavily is that they will be able to be taught in full in a two day workshop, along with the raster vector lesson. I think we only want to include information in this intro R lesson that is absolutely required in order for learners to be able to do the raster vector lesson, which is why I would recommend making this cut.

I'll of course leave this up to the Maintainers, but that is my recommendation.

@fmichonneau
Copy link
Member

@ErinBecker for this repository, Travis takes care of building and deploying the website, so no need to commit changes associated with rebuilding the lesson.

@lwasser
Copy link
Member

lwasser commented Jun 13, 2018 via email

@ErinBecker
Copy link
Contributor Author

Given the approval from @lwasser and no objections from others, I'll go ahead and merge this.

@ErinBecker ErinBecker merged commit b54e1d9 into datacarpentry:master Jul 10, 2018
@ErinBecker ErinBecker deleted the lesson-cuts-dplyr branch July 10, 2018 17:51
zkamvar pushed a commit that referenced this pull request May 4, 2023
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.

None yet

3 participants