-
Notifications
You must be signed in to change notification settings - Fork 27
Update documentation for v2.0.0 #166
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
Conversation
…crash in tumor only mode
…iguration' heading/section
… sectio 'container images'
…ction 'why lilac crashes on panel sample', add section 'automatically increases compute resources'
Warning Newer version of the nf-core template is available. Your pipeline is using an old version of the nf-core template: 2.14.1. For more documentation on how to update your pipeline, please see the nf-core documentation and Synchronisation documentation. |
Hi all, i've now completed the documentation for OA v2.0.0. For this PR, we only need to consider the files in |
only be used for [tuning process resource | ||
specifications](https://nf-co.re/docs/usage/configuration#tuning-workflow-resources), other infrastructural tweaks (such | ||
as output directories), or module arguments (args). | ||
To run from CRAM, simply provide the CRAM and/or CRAM index to `bam`/`bai` ibn column `filetype`: |
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.
small typo ibn
equivalence of these two pieces of software. The functional/feature parity with regards to version releases are detailed | ||
in the below table. | ||
|
||
| oncoanalyser | hmftools | |
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.
Shouldn't this stay in ?
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.
We agreed hartwig pipeline / hmftools updates will be tied to an oncoanalyser version. So no need for section 'release parity'
|
||
The default settings of `oncoanalyser` will accommodate typical sequencing depths for sample inputs and each individual | ||
tool is generally sequencing depth agnostic. However, variant calling is optimised for 100x tumor and 40x normal when | ||
invoked in `wgts` mode and expects sparse high-depth read data characteristic of panel sequencing when run in `targeted` |
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.
I think this could benefit from staying in?
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.
Restored this note
|
||
::: | ||
|
||
### Running `oncoanalyser` offline |
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.
Actually, the way to do this is using the nf-core download
(using the nf-core python package) command which will also download all the container images etc and you can copy that file in your HPC and run offline from there
Moving documentation changes over into #181 to avoid having to merge these conflicts |
PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).nextflow run . -profile debug,test,docker --outdir <OUTDIR>
).docs/usage.md
is updated.docs/output.md
is updated.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors).