Skip to content

Features for 1.3.0#100

Open
atrull314 wants to merge 61 commits intomasterfrom
dev
Open

Features for 1.3.0#100
atrull314 wants to merge 61 commits intomasterfrom
dev

Conversation

@atrull314
Copy link
Copy Markdown
Collaborator

This PR contains the following new features for 1.3.0:

  • Added chopper as an alternative to Nanofilt
  • Reduced the amount of storage in teh work directory by minimizing the amount of times FASTQs are modified (i.e. splitting + uncompression)
  • Update template to 3.5.1
  • Update pipeline to use strict syntax
  • Upgraded IsoQuant and updated downstream steps to work with new output format.
  • Due to updates to Isoquant we also removed the splitting and merging of bams done for this step
  • ToulligQC, FastQC, and Nanocomp are now disabled by default due to not scaling well with updated Nanopore chemistries

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/scnanoseq branch on the nf-core/test-datasets repository.
  • Make sure your code lints (nf-core pipelines 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).

nick-youngblut and others added 30 commits May 13, 2025 14:05
gzip compression of temp fastq files and replaced nanofilt with chopper
@nf-core-bot
Copy link
Copy Markdown
Member

Warning

Newer version of the nf-core template is available.

Your pipeline is using an old version of the nf-core template: 3.5.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.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

nf-core pipelines lint overall result: Failed ❌

Posted for pipeline commit cea013b

+| ✅ 204 tests passed       |+
#| ❔   4 tests were ignored |#
#| ❔   1 tests had warnings |#
!| ❗   6 tests had warnings |!
-| ❌   1 tests failed       |-
Details

❌ Test failures:

  • version_consistency - The versioning is not consistent between container, release tag and config. Found manifest.version = 1.2.2, nfcore_yml.version = 1.0.0

❗ Test warnings:

  • pipeline_todos - TODO string in nextflow.config: Update the field with the details of the contributors to your pipeline. New with Nextflow version 24.10.0
  • pipeline_todos - TODO string in nextflow.config: Specify any additional parameters here
  • pipeline_todos - TODO string in main.nf: Optionally add in-text citation tools to this list.
  • pipeline_todos - TODO string in main.nf: Optionally add bibliographic entries to this list.
  • pipeline_todos - TODO string in main.nf: Only uncomment below if logic in toolCitationText/toolBibliographyText has been filled!
  • included_configs - Pipeline config does not include custom configs. Please add the includeConfig line.

❔ Tests ignored:

  • files_unchanged - File ignored due to lint config: docs/images/nf-core-scnanoseq_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-scnanoseq_logo_dark.png
  • files_unchanged - File ignored due to lint config: .gitignore or .prettierignore
  • template_strings - template_strings

❔ Tests fixed:

✅ Tests passed:

Run details

  • nf-core/tools version 3.5.1
  • Run at 2026-05-07 14:13:23

@atrull314 atrull314 assigned atrull314 and unassigned atrull314 May 7, 2026
@atrull314 atrull314 requested a review from lianov May 7, 2026 16:40
@lianov
Copy link
Copy Markdown
Member

lianov commented May 8, 2026

@atrull314 : metro needs to be updated to reflect the change from nanofilt to chopper

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.

4 participants