-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
This failed to run complaining about ComplexHeatmap. I think you'll just need to load it.
|
also related to docker (I deleted my earlier comment because I didn't have the latest, but now I do and still not in there) - might want to manually push this - actually while we are at it, let's move the docker image to bixu... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 this is good now!
Purpose/implementation Section
What scientific question is your analysis addressing?
Closes #59. This PR adds a script
run_analysis_modules.sh
to generate results from all analysis modules in repo.What was your approach?
What GitHub issue does your pull request address?
#59
Directions for reviewers. Tell potential reviewers what kind of feedback you are
soliciting.
Which areas should receive a particularly close look?
Please review bash script code and ensure it runs successfully. To run from root:
bash scripts/run_analysis_modules.sh
Is there anything that you want to discuss further?
Is the analysis in a mature enough form that the resulting figure(s) and/or table(s) are
ready for review?
Results
What types of results are included (e.g., table, figure)?
What is your summary of the results?
Reproducibility Checklist
project Dockerfile.
Documentation Checklist
README
and it is up to date.analyses/README.md
and the entry is up todate.