Skip to content

Port the analysis scripts .R->.ipynb#11

Open
rht wants to merge 2 commits intoFutrell:masterfrom
rht:jupyter
Open

Port the analysis scripts .R->.ipynb#11
rht wants to merge 2 commits intoFutrell:masterfrom
rht:jupyter

Conversation

@rht
Copy link
Contributor

@rht rht commented Mar 20, 2017

This PR is WIP:

  • The line plots are not showing (due to some interpolation error with geom_smooth)
  • Brief "literate-programming" explanation before each plots

Should mindep_plots and mindep_hist be combined?

@rht rht changed the title Port the analysis scripts to .R->.ipynb Port the analysis scripts .R->.ipynb Mar 20, 2017
@rht
Copy link
Contributor Author

rht commented Mar 21, 2017

...I figured the source code of the jupyter notebooks are hard to audit/inline-comment. I wonder if it is simpler with just using literate R or heavily commented code. Then it would be much easier to wire the script to generate the plots and html/pdf reports to travis.

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.

1 participant