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

R basics dev #8

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Conversation

maglet
Copy link
Collaborator

@maglet maglet commented Apr 27, 2021

first draft of a carpentries style lesson of R basics from the library R workshop series

https://docs.google.com/presentation/d/1CLrkOPAPnUC9_xU4abBnEOywHn1JvR30CmgNT0CeP_c/edit#slide=id.g462168d7cd_0_7

@maglet maglet requested a review from sstevens2 April 27, 2021 15:50
Copy link
Collaborator

@sstevens2 sstevens2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't review the lesson content carefully but couple of small formatting things I noted.

Comment on lines +4 to +13
teaching:
exercises:
questions:
-
objectives:
-
keypoints:
-
source: Rmd
---
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure but I think these empty might give jekyll errors. Maybe include FIXME's instead?

Suggested change
teaching:
exercises:
questions:
-
objectives:
-
keypoints:
-
source: Rmd
---
teaching: FIXME
exercises: FIXME
questions:
- FIXME
objectives:
- FIXME
keypoints:
- FIXME
source: Rmd
---

```{r setup, echo = FALSE, purl = FALSE, message = FALSE}
library(knitr)
library(tidyverse)
source("../bin/chunk-options.R")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
source("../bin/chunk-options.R")
source("../bin/chunk-options.R")
knitr_fig_path("01-")

Might need this line, esp if we turn on the github actions rendering.

Comment on lines +633 to +658
# Glossary
* R
* RStudio
* open source
* cross platform
* packages
* reproducibility
* script
* console
* environment
* working directory
* operators
* arithmetic operator
* assignment operator
* functions
* arguments
* data type
* vector
* element
* coersion
* subsetting
* subset operator
* slice operator
* comparison operators
* logical operators
* missing values
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe want to move the glossary in the extras of the lesson and/or link them to glossario. Example in Jekyll lesson

@sstevens2 sstevens2 changed the base branch from gh-pages to main April 30, 2021 20:39
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.

2 participants