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.

Sign up for GitHub

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

Jump to bottom

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

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

wants to merge 2 commits into from

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
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@rht