+
Skip to content

Conversation

luan-n-nguyen
Copy link
Contributor

PR checklist

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the pipeline conventions in the contribution docs
  • If necessary, also make a PR on the nf-core/oncoanalyser branch on the nf-core/test-datasets repository.
  • Make sure your code lints (nf-core lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

luan-n-nguyen and others added 30 commits September 25, 2024 11:07
@nf-core-bot
Copy link
Member

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.
Please update your pipeline to the latest version.

For more documentation on how to update your pipeline, please see the nf-core documentation and Synchronisation documentation.

@luan-n-nguyen
Copy link
Contributor Author

luan-n-nguyen commented Mar 25, 2025

Hi all, i've now completed the documentation for OA v2.0.0.

For this PR, we only need to consider the files in docs/. Changes in any other files can be ignored.

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`:
Copy link
Contributor

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 |
Copy link
Contributor

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 ?

Copy link
Contributor Author

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`
Copy link
Contributor

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?

Copy link
Contributor Author

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
Copy link
Contributor

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

@scwatts
Copy link
Member

scwatts commented Apr 9, 2025

Moving documentation changes over into #181 to avoid having to merge these conflicts

@scwatts scwatts closed this Apr 9, 2025
@scwatts scwatts deleted the update-docs-v2.0 branch April 13, 2025 23:26
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.

5 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载