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

Stegen #44

Merged
merged 61 commits into from
Nov 9, 2018
Merged

Stegen #44

merged 61 commits into from
Nov 9, 2018

Conversation

thibautjombart
Copy link
Contributor

This is not ready to merge yet, but a first complete version of the Stegen practical. Doing a PR to facilitate deployment and keep conversation going here.

@thibautjombart
Copy link
Contributor Author

The code collapsing does not work when the document is embedded in the website, but works when compiled separately using rmarkdown::render() - output attached

practical-stegen.html.zip

zkamvar and others added 11 commits November 2, 2018 17:17
This still needs to be cleaned up and an explanation of
lapply() should be added, but it appears to work for now
:triumph:
In pandoc 2, gfm is favored over markdown_github
This fixes that issue and also changes the Rproj
to a package instead of a website so the Rmd
files don't get re-rendered every time.
We might wants to clarify a few things slightly better to make sure we catch beginning R users better. Especially around setting paths, importing data. Reason from 'classical data import' in excel/STATA
@zkamvar
Copy link
Member

zkamvar commented Nov 6, 2018

I'm at a stopping point. I'm pretty sure things need to be a bit more fleshed out but I am le tired and still have to make my presentations and shit before the end of the week. Take what I added or leave it; I don't particularly care.

@zkamvar
Copy link
Member

zkamvar commented Nov 9, 2018

Imma merge this. It will probably need a bit more polishing, but that can be done in a separate PR

@zkamvar zkamvar merged commit c61206c into master Nov 9, 2018
@zkamvar zkamvar deleted the stegen branch November 9, 2018 12:26
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.

4 participants