-
Notifications
You must be signed in to change notification settings - Fork 1
Update number formatting for braf-fusion plots (5/N) #68
Conversation
I continue to get this error related to Docker (I have pulled latest version and ran that):
|
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.
OK - I see the braf changes I mentioned in #67 here.
just noting that once #67 is merged, and merged into this PR, this PR will need a rerun before merge |
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.
Looks good!
Purpose/implementation Section
What scientific question is your analysis addressing?
What was your approach?
What GitHub issue does your pull request address?
Directions for reviewers. Tell potential reviewers what kind of feedback you are
soliciting.
Which areas should receive a particularly close look?
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.