Skip to content

Conversation

@zachary-foster
Copy link
Contributor

@zachary-foster zachary-foster commented Mar 15, 2025

  • Name output files by the prefix so that multiple runs of sourmash compare do not overwrite each other when published
  • Updates the conda/docker dependencies for all sourmash tools
  • Adds stub and associated test for sourmash/sketch to fix broken stub test for sourmash/compare
  • Migrates sourmash/gather to nf-test (resolves nf-test migration: sourmash/gather #7698)
  • Migrates sourmash/taxannotate to nf-test (resolves nf-test migration: sourmash/taxannotate #7699)

Copy link
Contributor

@SPPearce SPPearce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you bump the version of sourmash compare too please.
(and ideally the other subtools of sourmash).

@SPPearce
Copy link
Contributor

Oh, looks like the other subtools of sourmash are still on pytest, so probably best not updating in this PR anyway.

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
@zachary-foster
Copy link
Contributor Author

It looks to me like sourmash compare is already on the latest version? I can update the other sourmash tools, but it sounds like we should not do that right now?

@SPPearce
Copy link
Contributor

It looks to me like sourmash compare is already on the latest version? I can update the other sourmash tools, but it sounds like we should not do that right now?

If you could update and swap the other sourmash tools to nf-test at some point in the near future that would be much appreciated, we are trying to finish off all the pytest -> nf-test conversions :)

@SPPearce
Copy link
Contributor

(but doesn't have to be in this PR)

@SPPearce
Copy link
Contributor

Oh, sorry, I meant swap the version of sourmash/sketch, as you are already adjusting the files in this PR anyway.

@zachary-foster
Copy link
Contributor Author

Ok, I will update the docker/conda versions for all the sourmash tools and do the pytest -> nf-test conversions for gather and taxannotate.

Copy link
Contributor

@SPPearce SPPearce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That'd be most appreciated.
You can merge this one and do the updating of versions in a separate PR if you wish.

@zachary-foster zachary-foster requested a review from a team March 17, 2025 21:49
@zachary-foster
Copy link
Contributor Author

Thanks! No problem! I migrated the tests in this PR since I had already started before I saw you last message.

@zachary-foster zachary-foster added this pull request to the merge queue Mar 17, 2025
Merged via the queue into nf-core:master with commit 8501836 Mar 17, 2025
49 checks passed
@zachary-foster zachary-foster deleted the update_sourmash_compare branch March 17, 2025 22:06
zachary-foster added a commit to zachary-foster/modules that referenced this pull request Mar 19, 2025
…ker (nf-core#7808)

* update sourmash/compare so output is named by prefix

* add stub to sourmash/sketch

* Update modules/nf-core/sourmash/sketch/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* move test for sourmash/gather from pytest to nf-test

* migrated sourmash taxannotate to nf-test

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
github-merge-queue bot pushed a commit that referenced this pull request Mar 21, 2025
* add khmer/trimlowabund

* use evironment.yml

* chore(deps): update tj-actions/changed-files action to v46 (#7809)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Added coreograph nf-test (#7812)

coreograph nf-test

* Added ilastik/multicut nftest (#7811)

* main.nf changes

* finished test

* Update `graphtyper/vcfconcatenate`: Add batching to avoid long command line error (#7805)

add batching to vcfconcatenate to avoid long command line error

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* basicpy nf-test (#7772)

* basicpy nf-test

* removed leftover pytest

* separate output files

* updated meta

* separated outputs

* adapted test

* adapted inputs and snapshot

* removed pytest files

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Clair3 (#7765)

* Adds Clair3

* Removes TODOs and changes container urls

* Changes bash command

* Removes useless code lines

* Removes TODO statements

* Update meta.yml

adds more file info to meta.yml

* Update main.nf.test

removes TODO and updates clair3 model directory

* Update meta.yml
fixes doi link

* Add suggestions and changes data pulling

* Fix main.nf.test
adds correct structure to pass data to main

* fix yaml

* fix yaml
final one

* Update test snapshot

* Fix snapshot

* Fix tests

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Change meta description

* Update meta

* Update meta

* update screenshot

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* chore(deps): update github actions (#7796)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Change file name for severus_somatic.vcf (#7814)

* Change file name for severus_somatic.vcf

* Update snapshot

* Update yaml

* Clean up hifiasm module + add bin file input (#7802)

* Tidy hifiasm model + update tests

* Update nf.test

* Fix linting

* Add fasta and PAF outputs to hifiasm; combine all bin files into a single output channel. Update documentation

* Improve descriptiveness of output channel names

* fix samtools/convert stub (#7816)

* add default memory if task.memory is not set

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update `sourmash/compare`: Name output by prefix and update conda/docker (#7808)

* update sourmash/compare so output is named by prefix

* add stub to sourmash/sketch

* Update modules/nf-core/sourmash/sketch/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* move test for sourmash/gather from pytest to nf-test

* migrated sourmash taxannotate to nf-test

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* add Subworkflow linting step to CI (#7835)

add swf linting step

* Add missing swf change detection (#7836)

* add swf linting step

* add missing change detection for subworkflows

* Add bedtools/shuffle module (#7803)

* Add bedtools/shuffle module

* Make linter happy

* Update test snapshots for bedtools/shuffle

* add new ARCHIVE_EXTRACT subworkflow (#7817)

* add new subworkflow

* rename

* fix tags

* add basic meta.yml

* better comments

* Update subworkflows/nf-core/archive_extract/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* APPLYBQSR output line specificity  (#7818)

* updated output line and snap file

* linting fix in meta

* Trying out RunsOn runners (#7840)

* Trying out RunsOn runners

* try older image

* add debugging

* fix action

* pin setup-apptainer

* add gpu test

* fix usage syntax

* switch from docker_self_hosted to docker

* remove test comments

* Fix language server error in salmon/quant (#7843)

* New module: `busco/download` (#7788)

* add busco/download

* add author/maintainer

* update tests snap for new conda version

* revert to older conda version to match container versions

* update tests

* add stub

* removed unneeded code in stub

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* Fix language server errors in arriba/arriba (#7845)

* name output by prefix

* set to process_medium

* check if input and output file are the same

* make sure regex matches only part of string

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* apply changes made to file path handeling to stub

* Update modules/nf-core/khmer/trimlowabund/meta.yml

* Update modules/nf-core/khmer/trimlowabund/meta.yml

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Krešimir Beštak <86408271+kbestak@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
Co-authored-by: robert-a-forsyth <robert.alexander.forsyth@gmail.com>
Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Jim Downie <19718667+prototaxites@users.noreply.github.com>
Co-authored-by: Nicolas Vannieuwkerke <101190534+nvnieuwk@users.noreply.github.com>
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
Co-authored-by: Igor Trujnara <53370556+itrujnara@users.noreply.github.com>
Co-authored-by: Maxime U Garcia <max.u.garcia@gmail.com>
Co-authored-by: John Orgera <65687576+johnoooh@users.noreply.github.com>
bruno-ariano pushed a commit to HTGenomeAnalysisUnit/nf-core-modules that referenced this pull request Mar 24, 2025
* add khmer/trimlowabund

* use evironment.yml

* chore(deps): update tj-actions/changed-files action to v46 (nf-core#7809)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Added coreograph nf-test (nf-core#7812)

coreograph nf-test

* Added ilastik/multicut nftest (nf-core#7811)

* main.nf changes

* finished test

* Update `graphtyper/vcfconcatenate`: Add batching to avoid long command line error (nf-core#7805)

add batching to vcfconcatenate to avoid long command line error

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* basicpy nf-test (nf-core#7772)

* basicpy nf-test

* removed leftover pytest

* separate output files

* updated meta

* separated outputs

* adapted test

* adapted inputs and snapshot

* removed pytest files

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Clair3 (nf-core#7765)

* Adds Clair3

* Removes TODOs and changes container urls

* Changes bash command

* Removes useless code lines

* Removes TODO statements

* Update meta.yml

adds more file info to meta.yml

* Update main.nf.test

removes TODO and updates clair3 model directory

* Update meta.yml
fixes doi link

* Add suggestions and changes data pulling

* Fix main.nf.test
adds correct structure to pass data to main

* fix yaml

* fix yaml
final one

* Update test snapshot

* Fix snapshot

* Fix tests

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Change meta description

* Update meta

* Update meta

* update screenshot

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* chore(deps): update github actions (nf-core#7796)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Change file name for severus_somatic.vcf (nf-core#7814)

* Change file name for severus_somatic.vcf

* Update snapshot

* Update yaml

* Clean up hifiasm module + add bin file input (nf-core#7802)

* Tidy hifiasm model + update tests

* Update nf.test

* Fix linting

* Add fasta and PAF outputs to hifiasm; combine all bin files into a single output channel. Update documentation

* Improve descriptiveness of output channel names

* fix samtools/convert stub (nf-core#7816)

* add default memory if task.memory is not set

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update `sourmash/compare`: Name output by prefix and update conda/docker (nf-core#7808)

* update sourmash/compare so output is named by prefix

* add stub to sourmash/sketch

* Update modules/nf-core/sourmash/sketch/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* move test for sourmash/gather from pytest to nf-test

* migrated sourmash taxannotate to nf-test

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* add Subworkflow linting step to CI (nf-core#7835)

add swf linting step

* Add missing swf change detection (nf-core#7836)

* add swf linting step

* add missing change detection for subworkflows

* Add bedtools/shuffle module (nf-core#7803)

* Add bedtools/shuffle module

* Make linter happy

* Update test snapshots for bedtools/shuffle

* add new ARCHIVE_EXTRACT subworkflow (nf-core#7817)

* add new subworkflow

* rename

* fix tags

* add basic meta.yml

* better comments

* Update subworkflows/nf-core/archive_extract/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* APPLYBQSR output line specificity  (nf-core#7818)

* updated output line and snap file

* linting fix in meta

* Trying out RunsOn runners (nf-core#7840)

* Trying out RunsOn runners

* try older image

* add debugging

* fix action

* pin setup-apptainer

* add gpu test

* fix usage syntax

* switch from docker_self_hosted to docker

* remove test comments

* Fix language server error in salmon/quant (nf-core#7843)

* New module: `busco/download` (nf-core#7788)

* add busco/download

* add author/maintainer

* update tests snap for new conda version

* revert to older conda version to match container versions

* update tests

* add stub

* removed unneeded code in stub

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* Fix language server errors in arriba/arriba (nf-core#7845)

* name output by prefix

* set to process_medium

* check if input and output file are the same

* make sure regex matches only part of string

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* apply changes made to file path handeling to stub

* Update modules/nf-core/khmer/trimlowabund/meta.yml

* Update modules/nf-core/khmer/trimlowabund/meta.yml

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Krešimir Beštak <86408271+kbestak@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
Co-authored-by: robert-a-forsyth <robert.alexander.forsyth@gmail.com>
Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Jim Downie <19718667+prototaxites@users.noreply.github.com>
Co-authored-by: Nicolas Vannieuwkerke <101190534+nvnieuwk@users.noreply.github.com>
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
Co-authored-by: Igor Trujnara <53370556+itrujnara@users.noreply.github.com>
Co-authored-by: Maxime U Garcia <max.u.garcia@gmail.com>
Co-authored-by: John Orgera <65687576+johnoooh@users.noreply.github.com>
github-merge-queue bot pushed a commit that referenced this pull request Mar 26, 2025
* Module for PCA

* Module for PCA

* Module for PCA

* Module for PCA

* Module for PCA

* testing conda

* plinkpca

* Fixed ivar modules (#7868)

* fixed ivar variants module

* updated snapshot ivar variants because versions change

* fixed ivar consensus module

* simplified snapshot for consensus test, same as ivar variants module

* simplified snapshot for trim test, same as ivar variants module

* updated snapshot for ivar trim and consensus modules

* removed empty log file in ivar trim in snapshot

* updated ivar version to last one

* updated version also in conda env

* updated version also in conda env in ivar consensus and variants

* updated container version for ivar consensus and variants

* updated snapshots again

* Update FastK + MerquryFK versions + allow conda, add merquryfk/hapmaker module, port to nf-test (#7847)

* Start updating FastK modules to allow conda

* Migrate all Merquryfk modules to nf-test

* Update to container with explicit R

* Fix container for FastK

* Start fixing fastk modules

* Fix FastK tests, add merquryfk/hapmaker module

* Fix linting for some merquryFK modules

* Fix merquryfk/merquryfk linting

* Fix linting for everything except hapmaker

* Fix snapshots?

* Update tests

* Fix tests pt 1000

* Sort fastk/merge test inputs to try and ensure repeatability

* Rename output channels to avoid linting error

* Fix hapmaker snapshot

* Harshill alignment

* Stub output format determined by args; add details to meta.yml files to disambiguate meta maps

* Remove def again from variables - causes error otherwise

* Fix Fastk/merge stub

* Update snapshot

* Fix rnaseq preprocessing: don't trim with bad linting (#7881)

* Update versions for motus/preplong (#7854)

* Update versions for motus preplong

* Update modules/nf-core/motus/preplong/main.nf

Co-authored-by: Lili Andersson-Li <64467552+LilyAnderssonLee@users.noreply.github.com>

* Versions

* Add tests for downloaddb

* Add snapshot

* Update to the latest motus merge

* Update motus merge

* Remove pytest

* Update pytest

* Update downloaddb

* Update modules/nf-core/motus/downloaddb/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/motus/preplong/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Add stub

* Add stub

---------

Co-authored-by: Lili Andersson-Li <64467552+LilyAnderssonLee@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Fixed conda channel order for fq module (#7871) (#7873)

* Fixed conda channel order for fq module (#7871)

By listing bioconda as the primary channel, the fq package can be installed and used in conda environments.

modified:   modules/nf-core/fq/generate/environment.yml
modified:   modules/nf-core/fq/lint/environment.yml
modified:   modules/nf-core/fq/subsample/environment.yml

* added channel order comment to avoid a regression

---------

Co-authored-by: Angel Pizarro <pizarroa@amazon.com>

* New module:  `khmer/trimlowabund` (#7806)

* add khmer/trimlowabund

* use evironment.yml

* chore(deps): update tj-actions/changed-files action to v46 (#7809)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Added coreograph nf-test (#7812)

coreograph nf-test

* Added ilastik/multicut nftest (#7811)

* main.nf changes

* finished test

* Update `graphtyper/vcfconcatenate`: Add batching to avoid long command line error (#7805)

add batching to vcfconcatenate to avoid long command line error

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* basicpy nf-test (#7772)

* basicpy nf-test

* removed leftover pytest

* separate output files

* updated meta

* separated outputs

* adapted test

* adapted inputs and snapshot

* removed pytest files

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Clair3 (#7765)

* Adds Clair3

* Removes TODOs and changes container urls

* Changes bash command

* Removes useless code lines

* Removes TODO statements

* Update meta.yml

adds more file info to meta.yml

* Update main.nf.test

removes TODO and updates clair3 model directory

* Update meta.yml
fixes doi link

* Add suggestions and changes data pulling

* Fix main.nf.test
adds correct structure to pass data to main

* fix yaml

* fix yaml
final one

* Update test snapshot

* Fix snapshot

* Fix tests

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Change meta description

* Update meta

* Update meta

* update screenshot

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* chore(deps): update github actions (#7796)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Change file name for severus_somatic.vcf (#7814)

* Change file name for severus_somatic.vcf

* Update snapshot

* Update yaml

* Clean up hifiasm module + add bin file input (#7802)

* Tidy hifiasm model + update tests

* Update nf.test

* Fix linting

* Add fasta and PAF outputs to hifiasm; combine all bin files into a single output channel. Update documentation

* Improve descriptiveness of output channel names

* fix samtools/convert stub (#7816)

* add default memory if task.memory is not set

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update `sourmash/compare`: Name output by prefix and update conda/docker (#7808)

* update sourmash/compare so output is named by prefix

* add stub to sourmash/sketch

* Update modules/nf-core/sourmash/sketch/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* move test for sourmash/gather from pytest to nf-test

* migrated sourmash taxannotate to nf-test

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* add Subworkflow linting step to CI (#7835)

add swf linting step

* Add missing swf change detection (#7836)

* add swf linting step

* add missing change detection for subworkflows

* Add bedtools/shuffle module (#7803)

* Add bedtools/shuffle module

* Make linter happy

* Update test snapshots for bedtools/shuffle

* add new ARCHIVE_EXTRACT subworkflow (#7817)

* add new subworkflow

* rename

* fix tags

* add basic meta.yml

* better comments

* Update subworkflows/nf-core/archive_extract/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* APPLYBQSR output line specificity  (#7818)

* updated output line and snap file

* linting fix in meta

* Trying out RunsOn runners (#7840)

* Trying out RunsOn runners

* try older image

* add debugging

* fix action

* pin setup-apptainer

* add gpu test

* fix usage syntax

* switch from docker_self_hosted to docker

* remove test comments

* Fix language server error in salmon/quant (#7843)

* New module: `busco/download` (#7788)

* add busco/download

* add author/maintainer

* update tests snap for new conda version

* revert to older conda version to match container versions

* update tests

* add stub

* removed unneeded code in stub

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* Fix language server errors in arriba/arriba (#7845)

* name output by prefix

* set to process_medium

* check if input and output file are the same

* make sure regex matches only part of string

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* apply changes made to file path handeling to stub

* Update modules/nf-core/khmer/trimlowabund/meta.yml

* Update modules/nf-core/khmer/trimlowabund/meta.yml

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Krešimir Beštak <86408271+kbestak@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
Co-authored-by: robert-a-forsyth <robert.alexander.forsyth@gmail.com>
Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Jim Downie <19718667+prototaxites@users.noreply.github.com>
Co-authored-by: Nicolas Vannieuwkerke <101190534+nvnieuwk@users.noreply.github.com>
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
Co-authored-by: Igor Trujnara <53370556+itrujnara@users.noreply.github.com>
Co-authored-by: Maxime U Garcia <max.u.garcia@gmail.com>
Co-authored-by: John Orgera <65687576+johnoooh@users.noreply.github.com>

* Added cram output to samtools/fixmate (#7820)

* Added cram output to samtools/fixmate

* Fixed tag

* Fixed tag

* Fixed linting

* Updated onts

* Use nft-bam, update output glob

* Fixed meta.yml

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Migrate `ATAQV/MKARV` to nf-test (#7893)

Migrate to nf-test

Co-authored-by: LouisLeNezet <louislenezet@gmaio.com>

* Added repeatmasker/repeatmasker (#7825)

* Added repeatmasker/repeatmasker

* Added onts

* Added mdust (#7823)

* Added mdust

* Responded to review

* Added agat/spfilterbyorfsize (#7822)

* Added agat/spfilterbyorfsize

* Added Gff tests

* Migrate `authentict/deam2cont` to nf-test (#7947)

* Update to nf-test

* Fix samtools version

* Update modules/nf-core/authentict/deam2cont/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/authentict/deam2cont/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/authentict/deam2cont/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Migrate `BAMTOOLS/CONVERT` to nf-test (#7958)

* Migrate  to nf-test

* Update modules/nf-core/bamtools/convert/main.nf

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update modules/nf-core/bamtools/convert/tests/main.nf.test

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update modules/nf-core/bamtools/convert/tests/main.nf.test

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update modules/nf-core/bamtools/convert/tests/main.nf.test

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update variable names

* Fix spacing

---------

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Updated deevariant to avoid failing nf-test (#7960)

Updated the main to avoid failing nf-test

* answer review Edo

* fixing logfile

* fixing variable name bug

* rebasing

* fixing variable name bug

* fixing variable name bug

---------

Co-authored-by: Bruno Ariano <bruno.ariano@fht.org>
Co-authored-by: Sara Monzón <sara.monzon.fdz@gmail.com>
Co-authored-by: Jim Downie <19718667+prototaxites@users.noreply.github.com>
Co-authored-by: Jonathan Manning <jonathan.manning@seqera.io>
Co-authored-by: Sofia Stamouli <91951607+sofstam@users.noreply.github.com>
Co-authored-by: Lili Andersson-Li <64467552+LilyAnderssonLee@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
Co-authored-by: Angel Pizarro <delagoya@gmail.com>
Co-authored-by: Angel Pizarro <pizarroa@amazon.com>
Co-authored-by: Zachary Foster <zacharyfoster1989@gmail.com>
Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Krešimir Beštak <86408271+kbestak@users.noreply.github.com>
Co-authored-by: robert-a-forsyth <robert.alexander.forsyth@gmail.com>
Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Nicolas Vannieuwkerke <101190534+nvnieuwk@users.noreply.github.com>
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
Co-authored-by: Igor Trujnara <53370556+itrujnara@users.noreply.github.com>
Co-authored-by: Maxime U Garcia <max.u.garcia@gmail.com>
Co-authored-by: John Orgera <65687576+johnoooh@users.noreply.github.com>
Co-authored-by: Usman Rashid <usman@smme.edu.pk>
Co-authored-by: Louis Le Nézet <58640615+LouisLeNezet@users.noreply.github.com>
Co-authored-by: LouisLeNezet <louislenezet@gmaio.com>
Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>
Co-authored-by: Solenne Correard <54953390+scorreard@users.noreply.github.com>
zachary-foster added a commit to zachary-foster/modules that referenced this pull request Mar 27, 2025
…ker (nf-core#7808)

* update sourmash/compare so output is named by prefix

* add stub to sourmash/sketch

* Update modules/nf-core/sourmash/sketch/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* move test for sourmash/gather from pytest to nf-test

* migrated sourmash taxannotate to nf-test

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
zachary-foster added a commit to zachary-foster/modules that referenced this pull request Mar 27, 2025
* add khmer/trimlowabund

* use evironment.yml

* chore(deps): update tj-actions/changed-files action to v46 (nf-core#7809)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Added coreograph nf-test (nf-core#7812)

coreograph nf-test

* Added ilastik/multicut nftest (nf-core#7811)

* main.nf changes

* finished test

* Update `graphtyper/vcfconcatenate`: Add batching to avoid long command line error (nf-core#7805)

add batching to vcfconcatenate to avoid long command line error

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* basicpy nf-test (nf-core#7772)

* basicpy nf-test

* removed leftover pytest

* separate output files

* updated meta

* separated outputs

* adapted test

* adapted inputs and snapshot

* removed pytest files

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Clair3 (nf-core#7765)

* Adds Clair3

* Removes TODOs and changes container urls

* Changes bash command

* Removes useless code lines

* Removes TODO statements

* Update meta.yml

adds more file info to meta.yml

* Update main.nf.test

removes TODO and updates clair3 model directory

* Update meta.yml
fixes doi link

* Add suggestions and changes data pulling

* Fix main.nf.test
adds correct structure to pass data to main

* fix yaml

* fix yaml
final one

* Update test snapshot

* Fix snapshot

* Fix tests

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Change meta description

* Update meta

* Update meta

* update screenshot

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* chore(deps): update github actions (nf-core#7796)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Change file name for severus_somatic.vcf (nf-core#7814)

* Change file name for severus_somatic.vcf

* Update snapshot

* Update yaml

* Clean up hifiasm module + add bin file input (nf-core#7802)

* Tidy hifiasm model + update tests

* Update nf.test

* Fix linting

* Add fasta and PAF outputs to hifiasm; combine all bin files into a single output channel. Update documentation

* Improve descriptiveness of output channel names

* fix samtools/convert stub (nf-core#7816)

* add default memory if task.memory is not set

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update `sourmash/compare`: Name output by prefix and update conda/docker (nf-core#7808)

* update sourmash/compare so output is named by prefix

* add stub to sourmash/sketch

* Update modules/nf-core/sourmash/sketch/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* move test for sourmash/gather from pytest to nf-test

* migrated sourmash taxannotate to nf-test

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* add Subworkflow linting step to CI (nf-core#7835)

add swf linting step

* Add missing swf change detection (nf-core#7836)

* add swf linting step

* add missing change detection for subworkflows

* Add bedtools/shuffle module (nf-core#7803)

* Add bedtools/shuffle module

* Make linter happy

* Update test snapshots for bedtools/shuffle

* add new ARCHIVE_EXTRACT subworkflow (nf-core#7817)

* add new subworkflow

* rename

* fix tags

* add basic meta.yml

* better comments

* Update subworkflows/nf-core/archive_extract/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* APPLYBQSR output line specificity  (nf-core#7818)

* updated output line and snap file

* linting fix in meta

* Trying out RunsOn runners (nf-core#7840)

* Trying out RunsOn runners

* try older image

* add debugging

* fix action

* pin setup-apptainer

* add gpu test

* fix usage syntax

* switch from docker_self_hosted to docker

* remove test comments

* Fix language server error in salmon/quant (nf-core#7843)

* New module: `busco/download` (nf-core#7788)

* add busco/download

* add author/maintainer

* update tests snap for new conda version

* revert to older conda version to match container versions

* update tests

* add stub

* removed unneeded code in stub

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* Fix language server errors in arriba/arriba (nf-core#7845)

* name output by prefix

* set to process_medium

* check if input and output file are the same

* make sure regex matches only part of string

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* apply changes made to file path handeling to stub

* Update modules/nf-core/khmer/trimlowabund/meta.yml

* Update modules/nf-core/khmer/trimlowabund/meta.yml

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Krešimir Beštak <86408271+kbestak@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
Co-authored-by: robert-a-forsyth <robert.alexander.forsyth@gmail.com>
Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Jim Downie <19718667+prototaxites@users.noreply.github.com>
Co-authored-by: Nicolas Vannieuwkerke <101190534+nvnieuwk@users.noreply.github.com>
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
Co-authored-by: Igor Trujnara <53370556+itrujnara@users.noreply.github.com>
Co-authored-by: Maxime U Garcia <max.u.garcia@gmail.com>
Co-authored-by: John Orgera <65687576+johnoooh@users.noreply.github.com>
zachary-foster added a commit to zachary-foster/modules that referenced this pull request Mar 27, 2025
* Module for PCA

* Module for PCA

* Module for PCA

* Module for PCA

* Module for PCA

* testing conda

* plinkpca

* Fixed ivar modules (nf-core#7868)

* fixed ivar variants module

* updated snapshot ivar variants because versions change

* fixed ivar consensus module

* simplified snapshot for consensus test, same as ivar variants module

* simplified snapshot for trim test, same as ivar variants module

* updated snapshot for ivar trim and consensus modules

* removed empty log file in ivar trim in snapshot

* updated ivar version to last one

* updated version also in conda env

* updated version also in conda env in ivar consensus and variants

* updated container version for ivar consensus and variants

* updated snapshots again

* Update FastK + MerquryFK versions + allow conda, add merquryfk/hapmaker module, port to nf-test (nf-core#7847)

* Start updating FastK modules to allow conda

* Migrate all Merquryfk modules to nf-test

* Update to container with explicit R

* Fix container for FastK

* Start fixing fastk modules

* Fix FastK tests, add merquryfk/hapmaker module

* Fix linting for some merquryFK modules

* Fix merquryfk/merquryfk linting

* Fix linting for everything except hapmaker

* Fix snapshots?

* Update tests

* Fix tests pt 1000

* Sort fastk/merge test inputs to try and ensure repeatability

* Rename output channels to avoid linting error

* Fix hapmaker snapshot

* Harshill alignment

* Stub output format determined by args; add details to meta.yml files to disambiguate meta maps

* Remove def again from variables - causes error otherwise

* Fix Fastk/merge stub

* Update snapshot

* Fix rnaseq preprocessing: don't trim with bad linting (nf-core#7881)

* Update versions for motus/preplong (nf-core#7854)

* Update versions for motus preplong

* Update modules/nf-core/motus/preplong/main.nf

Co-authored-by: Lili Andersson-Li <64467552+LilyAnderssonLee@users.noreply.github.com>

* Versions

* Add tests for downloaddb

* Add snapshot

* Update to the latest motus merge

* Update motus merge

* Remove pytest

* Update pytest

* Update downloaddb

* Update modules/nf-core/motus/downloaddb/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/motus/preplong/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Add stub

* Add stub

---------

Co-authored-by: Lili Andersson-Li <64467552+LilyAnderssonLee@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Fixed conda channel order for fq module (nf-core#7871) (nf-core#7873)

* Fixed conda channel order for fq module (nf-core#7871)

By listing bioconda as the primary channel, the fq package can be installed and used in conda environments.

modified:   modules/nf-core/fq/generate/environment.yml
modified:   modules/nf-core/fq/lint/environment.yml
modified:   modules/nf-core/fq/subsample/environment.yml

* added channel order comment to avoid a regression

---------

Co-authored-by: Angel Pizarro <pizarroa@amazon.com>

* New module:  `khmer/trimlowabund` (nf-core#7806)

* add khmer/trimlowabund

* use evironment.yml

* chore(deps): update tj-actions/changed-files action to v46 (nf-core#7809)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Added coreograph nf-test (nf-core#7812)

coreograph nf-test

* Added ilastik/multicut nftest (nf-core#7811)

* main.nf changes

* finished test

* Update `graphtyper/vcfconcatenate`: Add batching to avoid long command line error (nf-core#7805)

add batching to vcfconcatenate to avoid long command line error

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* basicpy nf-test (nf-core#7772)

* basicpy nf-test

* removed leftover pytest

* separate output files

* updated meta

* separated outputs

* adapted test

* adapted inputs and snapshot

* removed pytest files

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Clair3 (nf-core#7765)

* Adds Clair3

* Removes TODOs and changes container urls

* Changes bash command

* Removes useless code lines

* Removes TODO statements

* Update meta.yml

adds more file info to meta.yml

* Update main.nf.test

removes TODO and updates clair3 model directory

* Update meta.yml
fixes doi link

* Add suggestions and changes data pulling

* Fix main.nf.test
adds correct structure to pass data to main

* fix yaml

* fix yaml
final one

* Update test snapshot

* Fix snapshot

* Fix tests

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Change meta description

* Update meta

* Update meta

* update screenshot

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* chore(deps): update github actions (nf-core#7796)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Change file name for severus_somatic.vcf (nf-core#7814)

* Change file name for severus_somatic.vcf

* Update snapshot

* Update yaml

* Clean up hifiasm module + add bin file input (nf-core#7802)

* Tidy hifiasm model + update tests

* Update nf.test

* Fix linting

* Add fasta and PAF outputs to hifiasm; combine all bin files into a single output channel. Update documentation

* Improve descriptiveness of output channel names

* fix samtools/convert stub (nf-core#7816)

* add default memory if task.memory is not set

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update `sourmash/compare`: Name output by prefix and update conda/docker (nf-core#7808)

* update sourmash/compare so output is named by prefix

* add stub to sourmash/sketch

* Update modules/nf-core/sourmash/sketch/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* move test for sourmash/gather from pytest to nf-test

* migrated sourmash taxannotate to nf-test

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* add Subworkflow linting step to CI (nf-core#7835)

add swf linting step

* Add missing swf change detection (nf-core#7836)

* add swf linting step

* add missing change detection for subworkflows

* Add bedtools/shuffle module (nf-core#7803)

* Add bedtools/shuffle module

* Make linter happy

* Update test snapshots for bedtools/shuffle

* add new ARCHIVE_EXTRACT subworkflow (nf-core#7817)

* add new subworkflow

* rename

* fix tags

* add basic meta.yml

* better comments

* Update subworkflows/nf-core/archive_extract/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* APPLYBQSR output line specificity  (nf-core#7818)

* updated output line and snap file

* linting fix in meta

* Trying out RunsOn runners (nf-core#7840)

* Trying out RunsOn runners

* try older image

* add debugging

* fix action

* pin setup-apptainer

* add gpu test

* fix usage syntax

* switch from docker_self_hosted to docker

* remove test comments

* Fix language server error in salmon/quant (nf-core#7843)

* New module: `busco/download` (nf-core#7788)

* add busco/download

* add author/maintainer

* update tests snap for new conda version

* revert to older conda version to match container versions

* update tests

* add stub

* removed unneeded code in stub

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* Fix language server errors in arriba/arriba (nf-core#7845)

* name output by prefix

* set to process_medium

* check if input and output file are the same

* make sure regex matches only part of string

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* apply changes made to file path handeling to stub

* Update modules/nf-core/khmer/trimlowabund/meta.yml

* Update modules/nf-core/khmer/trimlowabund/meta.yml

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Krešimir Beštak <86408271+kbestak@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
Co-authored-by: robert-a-forsyth <robert.alexander.forsyth@gmail.com>
Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Jim Downie <19718667+prototaxites@users.noreply.github.com>
Co-authored-by: Nicolas Vannieuwkerke <101190534+nvnieuwk@users.noreply.github.com>
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
Co-authored-by: Igor Trujnara <53370556+itrujnara@users.noreply.github.com>
Co-authored-by: Maxime U Garcia <max.u.garcia@gmail.com>
Co-authored-by: John Orgera <65687576+johnoooh@users.noreply.github.com>

* Added cram output to samtools/fixmate (nf-core#7820)

* Added cram output to samtools/fixmate

* Fixed tag

* Fixed tag

* Fixed linting

* Updated onts

* Use nft-bam, update output glob

* Fixed meta.yml

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Migrate `ATAQV/MKARV` to nf-test (nf-core#7893)

Migrate to nf-test

Co-authored-by: LouisLeNezet <louislenezet@gmaio.com>

* Added repeatmasker/repeatmasker (nf-core#7825)

* Added repeatmasker/repeatmasker

* Added onts

* Added mdust (nf-core#7823)

* Added mdust

* Responded to review

* Added agat/spfilterbyorfsize (nf-core#7822)

* Added agat/spfilterbyorfsize

* Added Gff tests

* Migrate `authentict/deam2cont` to nf-test (nf-core#7947)

* Update to nf-test

* Fix samtools version

* Update modules/nf-core/authentict/deam2cont/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/authentict/deam2cont/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/authentict/deam2cont/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Migrate `BAMTOOLS/CONVERT` to nf-test (nf-core#7958)

* Migrate  to nf-test

* Update modules/nf-core/bamtools/convert/main.nf

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update modules/nf-core/bamtools/convert/tests/main.nf.test

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update modules/nf-core/bamtools/convert/tests/main.nf.test

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update modules/nf-core/bamtools/convert/tests/main.nf.test

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update variable names

* Fix spacing

---------

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Updated deevariant to avoid failing nf-test (nf-core#7960)

Updated the main to avoid failing nf-test

* answer review Edo

* fixing logfile

* fixing variable name bug

* rebasing

* fixing variable name bug

* fixing variable name bug

---------

Co-authored-by: Bruno Ariano <bruno.ariano@fht.org>
Co-authored-by: Sara Monzón <sara.monzon.fdz@gmail.com>
Co-authored-by: Jim Downie <19718667+prototaxites@users.noreply.github.com>
Co-authored-by: Jonathan Manning <jonathan.manning@seqera.io>
Co-authored-by: Sofia Stamouli <91951607+sofstam@users.noreply.github.com>
Co-authored-by: Lili Andersson-Li <64467552+LilyAnderssonLee@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
Co-authored-by: Angel Pizarro <delagoya@gmail.com>
Co-authored-by: Angel Pizarro <pizarroa@amazon.com>
Co-authored-by: Zachary Foster <zacharyfoster1989@gmail.com>
Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Krešimir Beštak <86408271+kbestak@users.noreply.github.com>
Co-authored-by: robert-a-forsyth <robert.alexander.forsyth@gmail.com>
Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Nicolas Vannieuwkerke <101190534+nvnieuwk@users.noreply.github.com>
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
Co-authored-by: Igor Trujnara <53370556+itrujnara@users.noreply.github.com>
Co-authored-by: Maxime U Garcia <max.u.garcia@gmail.com>
Co-authored-by: John Orgera <65687576+johnoooh@users.noreply.github.com>
Co-authored-by: Usman Rashid <usman@smme.edu.pk>
Co-authored-by: Louis Le Nézet <58640615+LouisLeNezet@users.noreply.github.com>
Co-authored-by: LouisLeNezet <louislenezet@gmaio.com>
Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>
Co-authored-by: Solenne Correard <54953390+scorreard@users.noreply.github.com>
Dewey-Wang pushed a commit to Dewey-Wang/modules that referenced this pull request Mar 28, 2025
* Module for PCA

* Module for PCA

* Module for PCA

* Module for PCA

* Module for PCA

* testing conda

* plinkpca

* Fixed ivar modules (nf-core#7868)

* fixed ivar variants module

* updated snapshot ivar variants because versions change

* fixed ivar consensus module

* simplified snapshot for consensus test, same as ivar variants module

* simplified snapshot for trim test, same as ivar variants module

* updated snapshot for ivar trim and consensus modules

* removed empty log file in ivar trim in snapshot

* updated ivar version to last one

* updated version also in conda env

* updated version also in conda env in ivar consensus and variants

* updated container version for ivar consensus and variants

* updated snapshots again

* Update FastK + MerquryFK versions + allow conda, add merquryfk/hapmaker module, port to nf-test (nf-core#7847)

* Start updating FastK modules to allow conda

* Migrate all Merquryfk modules to nf-test

* Update to container with explicit R

* Fix container for FastK

* Start fixing fastk modules

* Fix FastK tests, add merquryfk/hapmaker module

* Fix linting for some merquryFK modules

* Fix merquryfk/merquryfk linting

* Fix linting for everything except hapmaker

* Fix snapshots?

* Update tests

* Fix tests pt 1000

* Sort fastk/merge test inputs to try and ensure repeatability

* Rename output channels to avoid linting error

* Fix hapmaker snapshot

* Harshill alignment

* Stub output format determined by args; add details to meta.yml files to disambiguate meta maps

* Remove def again from variables - causes error otherwise

* Fix Fastk/merge stub

* Update snapshot

* Fix rnaseq preprocessing: don't trim with bad linting (nf-core#7881)

* Update versions for motus/preplong (nf-core#7854)

* Update versions for motus preplong

* Update modules/nf-core/motus/preplong/main.nf

Co-authored-by: Lili Andersson-Li <64467552+LilyAnderssonLee@users.noreply.github.com>

* Versions

* Add tests for downloaddb

* Add snapshot

* Update to the latest motus merge

* Update motus merge

* Remove pytest

* Update pytest

* Update downloaddb

* Update modules/nf-core/motus/downloaddb/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/motus/preplong/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Add stub

* Add stub

---------

Co-authored-by: Lili Andersson-Li <64467552+LilyAnderssonLee@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Fixed conda channel order for fq module (nf-core#7871) (nf-core#7873)

* Fixed conda channel order for fq module (nf-core#7871)

By listing bioconda as the primary channel, the fq package can be installed and used in conda environments.

modified:   modules/nf-core/fq/generate/environment.yml
modified:   modules/nf-core/fq/lint/environment.yml
modified:   modules/nf-core/fq/subsample/environment.yml

* added channel order comment to avoid a regression

---------

Co-authored-by: Angel Pizarro <pizarroa@amazon.com>

* New module:  `khmer/trimlowabund` (nf-core#7806)

* add khmer/trimlowabund

* use evironment.yml

* chore(deps): update tj-actions/changed-files action to v46 (nf-core#7809)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Added coreograph nf-test (nf-core#7812)

coreograph nf-test

* Added ilastik/multicut nftest (nf-core#7811)

* main.nf changes

* finished test

* Update `graphtyper/vcfconcatenate`: Add batching to avoid long command line error (nf-core#7805)

add batching to vcfconcatenate to avoid long command line error

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* basicpy nf-test (nf-core#7772)

* basicpy nf-test

* removed leftover pytest

* separate output files

* updated meta

* separated outputs

* adapted test

* adapted inputs and snapshot

* removed pytest files

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Clair3 (nf-core#7765)

* Adds Clair3

* Removes TODOs and changes container urls

* Changes bash command

* Removes useless code lines

* Removes TODO statements

* Update meta.yml

adds more file info to meta.yml

* Update main.nf.test

removes TODO and updates clair3 model directory

* Update meta.yml
fixes doi link

* Add suggestions and changes data pulling

* Fix main.nf.test
adds correct structure to pass data to main

* fix yaml

* fix yaml
final one

* Update test snapshot

* Fix snapshot

* Fix tests

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Change meta description

* Update meta

* Update meta

* update screenshot

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* chore(deps): update github actions (nf-core#7796)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Change file name for severus_somatic.vcf (nf-core#7814)

* Change file name for severus_somatic.vcf

* Update snapshot

* Update yaml

* Clean up hifiasm module + add bin file input (nf-core#7802)

* Tidy hifiasm model + update tests

* Update nf.test

* Fix linting

* Add fasta and PAF outputs to hifiasm; combine all bin files into a single output channel. Update documentation

* Improve descriptiveness of output channel names

* fix samtools/convert stub (nf-core#7816)

* add default memory if task.memory is not set

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update `sourmash/compare`: Name output by prefix and update conda/docker (nf-core#7808)

* update sourmash/compare so output is named by prefix

* add stub to sourmash/sketch

* Update modules/nf-core/sourmash/sketch/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* move test for sourmash/gather from pytest to nf-test

* migrated sourmash taxannotate to nf-test

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* add Subworkflow linting step to CI (nf-core#7835)

add swf linting step

* Add missing swf change detection (nf-core#7836)

* add swf linting step

* add missing change detection for subworkflows

* Add bedtools/shuffle module (nf-core#7803)

* Add bedtools/shuffle module

* Make linter happy

* Update test snapshots for bedtools/shuffle

* add new ARCHIVE_EXTRACT subworkflow (nf-core#7817)

* add new subworkflow

* rename

* fix tags

* add basic meta.yml

* better comments

* Update subworkflows/nf-core/archive_extract/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* APPLYBQSR output line specificity  (nf-core#7818)

* updated output line and snap file

* linting fix in meta

* Trying out RunsOn runners (nf-core#7840)

* Trying out RunsOn runners

* try older image

* add debugging

* fix action

* pin setup-apptainer

* add gpu test

* fix usage syntax

* switch from docker_self_hosted to docker

* remove test comments

* Fix language server error in salmon/quant (nf-core#7843)

* New module: `busco/download` (nf-core#7788)

* add busco/download

* add author/maintainer

* update tests snap for new conda version

* revert to older conda version to match container versions

* update tests

* add stub

* removed unneeded code in stub

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* Fix language server errors in arriba/arriba (nf-core#7845)

* name output by prefix

* set to process_medium

* check if input and output file are the same

* make sure regex matches only part of string

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* apply changes made to file path handeling to stub

* Update modules/nf-core/khmer/trimlowabund/meta.yml

* Update modules/nf-core/khmer/trimlowabund/meta.yml

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Krešimir Beštak <86408271+kbestak@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
Co-authored-by: robert-a-forsyth <robert.alexander.forsyth@gmail.com>
Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Jim Downie <19718667+prototaxites@users.noreply.github.com>
Co-authored-by: Nicolas Vannieuwkerke <101190534+nvnieuwk@users.noreply.github.com>
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
Co-authored-by: Igor Trujnara <53370556+itrujnara@users.noreply.github.com>
Co-authored-by: Maxime U Garcia <max.u.garcia@gmail.com>
Co-authored-by: John Orgera <65687576+johnoooh@users.noreply.github.com>

* Added cram output to samtools/fixmate (nf-core#7820)

* Added cram output to samtools/fixmate

* Fixed tag

* Fixed tag

* Fixed linting

* Updated onts

* Use nft-bam, update output glob

* Fixed meta.yml

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Migrate `ATAQV/MKARV` to nf-test (nf-core#7893)

Migrate to nf-test

Co-authored-by: LouisLeNezet <louislenezet@gmaio.com>

* Added repeatmasker/repeatmasker (nf-core#7825)

* Added repeatmasker/repeatmasker

* Added onts

* Added mdust (nf-core#7823)

* Added mdust

* Responded to review

* Added agat/spfilterbyorfsize (nf-core#7822)

* Added agat/spfilterbyorfsize

* Added Gff tests

* Migrate `authentict/deam2cont` to nf-test (nf-core#7947)

* Update to nf-test

* Fix samtools version

* Update modules/nf-core/authentict/deam2cont/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/authentict/deam2cont/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/authentict/deam2cont/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Migrate `BAMTOOLS/CONVERT` to nf-test (nf-core#7958)

* Migrate  to nf-test

* Update modules/nf-core/bamtools/convert/main.nf

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update modules/nf-core/bamtools/convert/tests/main.nf.test

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update modules/nf-core/bamtools/convert/tests/main.nf.test

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update modules/nf-core/bamtools/convert/tests/main.nf.test

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update variable names

* Fix spacing

---------

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Updated deevariant to avoid failing nf-test (nf-core#7960)

Updated the main to avoid failing nf-test

* answer review Edo

* fixing logfile

* fixing variable name bug

* rebasing

* fixing variable name bug

* fixing variable name bug

---------

Co-authored-by: Bruno Ariano <bruno.ariano@fht.org>
Co-authored-by: Sara Monzón <sara.monzon.fdz@gmail.com>
Co-authored-by: Jim Downie <19718667+prototaxites@users.noreply.github.com>
Co-authored-by: Jonathan Manning <jonathan.manning@seqera.io>
Co-authored-by: Sofia Stamouli <91951607+sofstam@users.noreply.github.com>
Co-authored-by: Lili Andersson-Li <64467552+LilyAnderssonLee@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
Co-authored-by: Angel Pizarro <delagoya@gmail.com>
Co-authored-by: Angel Pizarro <pizarroa@amazon.com>
Co-authored-by: Zachary Foster <zacharyfoster1989@gmail.com>
Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Krešimir Beštak <86408271+kbestak@users.noreply.github.com>
Co-authored-by: robert-a-forsyth <robert.alexander.forsyth@gmail.com>
Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Nicolas Vannieuwkerke <101190534+nvnieuwk@users.noreply.github.com>
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
Co-authored-by: Igor Trujnara <53370556+itrujnara@users.noreply.github.com>
Co-authored-by: Maxime U Garcia <max.u.garcia@gmail.com>
Co-authored-by: John Orgera <65687576+johnoooh@users.noreply.github.com>
Co-authored-by: Usman Rashid <usman@smme.edu.pk>
Co-authored-by: Louis Le Nézet <58640615+LouisLeNezet@users.noreply.github.com>
Co-authored-by: LouisLeNezet <louislenezet@gmaio.com>
Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>
Co-authored-by: Solenne Correard <54953390+scorreard@users.noreply.github.com>
famosab pushed a commit to famosab/modules that referenced this pull request Jun 3, 2025
…ker (nf-core#7808)

* update sourmash/compare so output is named by prefix

* add stub to sourmash/sketch

* Update modules/nf-core/sourmash/sketch/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* move test for sourmash/gather from pytest to nf-test

* migrated sourmash taxannotate to nf-test

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
famosab added a commit to famosab/modules that referenced this pull request Jun 3, 2025
* add khmer/trimlowabund

* use evironment.yml

* chore(deps): update tj-actions/changed-files action to v46 (nf-core#7809)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Added coreograph nf-test (nf-core#7812)

coreograph nf-test

* Added ilastik/multicut nftest (nf-core#7811)

* main.nf changes

* finished test

* Update `graphtyper/vcfconcatenate`: Add batching to avoid long command line error (nf-core#7805)

add batching to vcfconcatenate to avoid long command line error

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* basicpy nf-test (nf-core#7772)

* basicpy nf-test

* removed leftover pytest

* separate output files

* updated meta

* separated outputs

* adapted test

* adapted inputs and snapshot

* removed pytest files

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Clair3 (nf-core#7765)

* Adds Clair3

* Removes TODOs and changes container urls

* Changes bash command

* Removes useless code lines

* Removes TODO statements

* Update meta.yml

adds more file info to meta.yml

* Update main.nf.test

removes TODO and updates clair3 model directory

* Update meta.yml
fixes doi link

* Add suggestions and changes data pulling

* Fix main.nf.test
adds correct structure to pass data to main

* fix yaml

* fix yaml
final one

* Update test snapshot

* Fix snapshot

* Fix tests

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Change meta description

* Update meta

* Update meta

* update screenshot

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* chore(deps): update github actions (nf-core#7796)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Change file name for severus_somatic.vcf (nf-core#7814)

* Change file name for severus_somatic.vcf

* Update snapshot

* Update yaml

* Clean up hifiasm module + add bin file input (nf-core#7802)

* Tidy hifiasm model + update tests

* Update nf.test

* Fix linting

* Add fasta and PAF outputs to hifiasm; combine all bin files into a single output channel. Update documentation

* Improve descriptiveness of output channel names

* fix samtools/convert stub (nf-core#7816)

* add default memory if task.memory is not set

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update `sourmash/compare`: Name output by prefix and update conda/docker (nf-core#7808)

* update sourmash/compare so output is named by prefix

* add stub to sourmash/sketch

* Update modules/nf-core/sourmash/sketch/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* move test for sourmash/gather from pytest to nf-test

* migrated sourmash taxannotate to nf-test

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* add Subworkflow linting step to CI (nf-core#7835)

add swf linting step

* Add missing swf change detection (nf-core#7836)

* add swf linting step

* add missing change detection for subworkflows

* Add bedtools/shuffle module (nf-core#7803)

* Add bedtools/shuffle module

* Make linter happy

* Update test snapshots for bedtools/shuffle

* add new ARCHIVE_EXTRACT subworkflow (nf-core#7817)

* add new subworkflow

* rename

* fix tags

* add basic meta.yml

* better comments

* Update subworkflows/nf-core/archive_extract/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* APPLYBQSR output line specificity  (nf-core#7818)

* updated output line and snap file

* linting fix in meta

* Trying out RunsOn runners (nf-core#7840)

* Trying out RunsOn runners

* try older image

* add debugging

* fix action

* pin setup-apptainer

* add gpu test

* fix usage syntax

* switch from docker_self_hosted to docker

* remove test comments

* Fix language server error in salmon/quant (nf-core#7843)

* New module: `busco/download` (nf-core#7788)

* add busco/download

* add author/maintainer

* update tests snap for new conda version

* revert to older conda version to match container versions

* update tests

* add stub

* removed unneeded code in stub

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* Fix language server errors in arriba/arriba (nf-core#7845)

* name output by prefix

* set to process_medium

* check if input and output file are the same

* make sure regex matches only part of string

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* apply changes made to file path handeling to stub

* Update modules/nf-core/khmer/trimlowabund/meta.yml

* Update modules/nf-core/khmer/trimlowabund/meta.yml

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Krešimir Beštak <86408271+kbestak@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
Co-authored-by: robert-a-forsyth <robert.alexander.forsyth@gmail.com>
Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Jim Downie <19718667+prototaxites@users.noreply.github.com>
Co-authored-by: Nicolas Vannieuwkerke <101190534+nvnieuwk@users.noreply.github.com>
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
Co-authored-by: Igor Trujnara <53370556+itrujnara@users.noreply.github.com>
Co-authored-by: Maxime U Garcia <max.u.garcia@gmail.com>
Co-authored-by: John Orgera <65687576+johnoooh@users.noreply.github.com>
famosab added a commit to famosab/modules that referenced this pull request Jun 3, 2025
* Module for PCA

* Module for PCA

* Module for PCA

* Module for PCA

* Module for PCA

* testing conda

* plinkpca

* Fixed ivar modules (nf-core#7868)

* fixed ivar variants module

* updated snapshot ivar variants because versions change

* fixed ivar consensus module

* simplified snapshot for consensus test, same as ivar variants module

* simplified snapshot for trim test, same as ivar variants module

* updated snapshot for ivar trim and consensus modules

* removed empty log file in ivar trim in snapshot

* updated ivar version to last one

* updated version also in conda env

* updated version also in conda env in ivar consensus and variants

* updated container version for ivar consensus and variants

* updated snapshots again

* Update FastK + MerquryFK versions + allow conda, add merquryfk/hapmaker module, port to nf-test (nf-core#7847)

* Start updating FastK modules to allow conda

* Migrate all Merquryfk modules to nf-test

* Update to container with explicit R

* Fix container for FastK

* Start fixing fastk modules

* Fix FastK tests, add merquryfk/hapmaker module

* Fix linting for some merquryFK modules

* Fix merquryfk/merquryfk linting

* Fix linting for everything except hapmaker

* Fix snapshots?

* Update tests

* Fix tests pt 1000

* Sort fastk/merge test inputs to try and ensure repeatability

* Rename output channels to avoid linting error

* Fix hapmaker snapshot

* Harshill alignment

* Stub output format determined by args; add details to meta.yml files to disambiguate meta maps

* Remove def again from variables - causes error otherwise

* Fix Fastk/merge stub

* Update snapshot

* Fix rnaseq preprocessing: don't trim with bad linting (nf-core#7881)

* Update versions for motus/preplong (nf-core#7854)

* Update versions for motus preplong

* Update modules/nf-core/motus/preplong/main.nf

Co-authored-by: Lili Andersson-Li <64467552+LilyAnderssonLee@users.noreply.github.com>

* Versions

* Add tests for downloaddb

* Add snapshot

* Update to the latest motus merge

* Update motus merge

* Remove pytest

* Update pytest

* Update downloaddb

* Update modules/nf-core/motus/downloaddb/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/motus/preplong/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Add stub

* Add stub

---------

Co-authored-by: Lili Andersson-Li <64467552+LilyAnderssonLee@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Fixed conda channel order for fq module (nf-core#7871) (nf-core#7873)

* Fixed conda channel order for fq module (nf-core#7871)

By listing bioconda as the primary channel, the fq package can be installed and used in conda environments.

modified:   modules/nf-core/fq/generate/environment.yml
modified:   modules/nf-core/fq/lint/environment.yml
modified:   modules/nf-core/fq/subsample/environment.yml

* added channel order comment to avoid a regression

---------

Co-authored-by: Angel Pizarro <pizarroa@amazon.com>

* New module:  `khmer/trimlowabund` (nf-core#7806)

* add khmer/trimlowabund

* use evironment.yml

* chore(deps): update tj-actions/changed-files action to v46 (nf-core#7809)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Added coreograph nf-test (nf-core#7812)

coreograph nf-test

* Added ilastik/multicut nftest (nf-core#7811)

* main.nf changes

* finished test

* Update `graphtyper/vcfconcatenate`: Add batching to avoid long command line error (nf-core#7805)

add batching to vcfconcatenate to avoid long command line error

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* basicpy nf-test (nf-core#7772)

* basicpy nf-test

* removed leftover pytest

* separate output files

* updated meta

* separated outputs

* adapted test

* adapted inputs and snapshot

* removed pytest files

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Clair3 (nf-core#7765)

* Adds Clair3

* Removes TODOs and changes container urls

* Changes bash command

* Removes useless code lines

* Removes TODO statements

* Update meta.yml

adds more file info to meta.yml

* Update main.nf.test

removes TODO and updates clair3 model directory

* Update meta.yml
fixes doi link

* Add suggestions and changes data pulling

* Fix main.nf.test
adds correct structure to pass data to main

* fix yaml

* fix yaml
final one

* Update test snapshot

* Fix snapshot

* Fix tests

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/clair3/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Change meta description

* Update meta

* Update meta

* update screenshot

* Update modules/nf-core/clair3/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* chore(deps): update github actions (nf-core#7796)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Change file name for severus_somatic.vcf (nf-core#7814)

* Change file name for severus_somatic.vcf

* Update snapshot

* Update yaml

* Clean up hifiasm module + add bin file input (nf-core#7802)

* Tidy hifiasm model + update tests

* Update nf.test

* Fix linting

* Add fasta and PAF outputs to hifiasm; combine all bin files into a single output channel. Update documentation

* Improve descriptiveness of output channel names

* fix samtools/convert stub (nf-core#7816)

* add default memory if task.memory is not set

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/meta.yml

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update `sourmash/compare`: Name output by prefix and update conda/docker (nf-core#7808)

* update sourmash/compare so output is named by prefix

* add stub to sourmash/sketch

* Update modules/nf-core/sourmash/sketch/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* move test for sourmash/gather from pytest to nf-test

* migrated sourmash taxannotate to nf-test

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* add Subworkflow linting step to CI (nf-core#7835)

add swf linting step

* Add missing swf change detection (nf-core#7836)

* add swf linting step

* add missing change detection for subworkflows

* Add bedtools/shuffle module (nf-core#7803)

* Add bedtools/shuffle module

* Make linter happy

* Update test snapshots for bedtools/shuffle

* add new ARCHIVE_EXTRACT subworkflow (nf-core#7817)

* add new subworkflow

* rename

* fix tags

* add basic meta.yml

* better comments

* Update subworkflows/nf-core/archive_extract/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* APPLYBQSR output line specificity  (nf-core#7818)

* updated output line and snap file

* linting fix in meta

* Trying out RunsOn runners (nf-core#7840)

* Trying out RunsOn runners

* try older image

* add debugging

* fix action

* pin setup-apptainer

* add gpu test

* fix usage syntax

* switch from docker_self_hosted to docker

* remove test comments

* Fix language server error in salmon/quant (nf-core#7843)

* New module: `busco/download` (nf-core#7788)

* add busco/download

* add author/maintainer

* update tests snap for new conda version

* revert to older conda version to match container versions

* update tests

* add stub

* removed unneeded code in stub

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>

* Fix language server errors in arriba/arriba (nf-core#7845)

* name output by prefix

* set to process_medium

* check if input and output file are the same

* make sure regex matches only part of string

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/khmer/trimlowabund/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* apply changes made to file path handeling to stub

* Update modules/nf-core/khmer/trimlowabund/meta.yml

* Update modules/nf-core/khmer/trimlowabund/meta.yml

---------

Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Krešimir Beštak <86408271+kbestak@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
Co-authored-by: robert-a-forsyth <robert.alexander.forsyth@gmail.com>
Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Jim Downie <19718667+prototaxites@users.noreply.github.com>
Co-authored-by: Nicolas Vannieuwkerke <101190534+nvnieuwk@users.noreply.github.com>
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
Co-authored-by: Igor Trujnara <53370556+itrujnara@users.noreply.github.com>
Co-authored-by: Maxime U Garcia <max.u.garcia@gmail.com>
Co-authored-by: John Orgera <65687576+johnoooh@users.noreply.github.com>

* Added cram output to samtools/fixmate (nf-core#7820)

* Added cram output to samtools/fixmate

* Fixed tag

* Fixed tag

* Fixed linting

* Updated onts

* Use nft-bam, update output glob

* Fixed meta.yml

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Migrate `ATAQV/MKARV` to nf-test (nf-core#7893)

Migrate to nf-test

Co-authored-by: LouisLeNezet <louislenezet@gmaio.com>

* Added repeatmasker/repeatmasker (nf-core#7825)

* Added repeatmasker/repeatmasker

* Added onts

* Added mdust (nf-core#7823)

* Added mdust

* Responded to review

* Added agat/spfilterbyorfsize (nf-core#7822)

* Added agat/spfilterbyorfsize

* Added Gff tests

* Migrate `authentict/deam2cont` to nf-test (nf-core#7947)

* Update to nf-test

* Fix samtools version

* Update modules/nf-core/authentict/deam2cont/main.nf

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/authentict/deam2cont/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Update modules/nf-core/authentict/deam2cont/tests/main.nf.test

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

---------

Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>

* Migrate `BAMTOOLS/CONVERT` to nf-test (nf-core#7958)

* Migrate  to nf-test

* Update modules/nf-core/bamtools/convert/main.nf

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update modules/nf-core/bamtools/convert/tests/main.nf.test

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update modules/nf-core/bamtools/convert/tests/main.nf.test

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update modules/nf-core/bamtools/convert/tests/main.nf.test

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Update variable names

* Fix spacing

---------

Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>

* Updated deevariant to avoid failing nf-test (nf-core#7960)

Updated the main to avoid failing nf-test

* answer review Edo

* fixing logfile

* fixing variable name bug

* rebasing

* fixing variable name bug

* fixing variable name bug

---------

Co-authored-by: Bruno Ariano <bruno.ariano@fht.org>
Co-authored-by: Sara Monzón <sara.monzon.fdz@gmail.com>
Co-authored-by: Jim Downie <19718667+prototaxites@users.noreply.github.com>
Co-authored-by: Jonathan Manning <jonathan.manning@seqera.io>
Co-authored-by: Sofia Stamouli <91951607+sofstam@users.noreply.github.com>
Co-authored-by: Lili Andersson-Li <64467552+LilyAnderssonLee@users.noreply.github.com>
Co-authored-by: Simon Pearce <24893913+SPPearce@users.noreply.github.com>
Co-authored-by: Angel Pizarro <delagoya@gmail.com>
Co-authored-by: Angel Pizarro <pizarroa@amazon.com>
Co-authored-by: Zachary Foster <zacharyfoster1989@gmail.com>
Co-authored-by: Zachary Foster <fosterz@pop-os.localdomain>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Krešimir Beštak <86408271+kbestak@users.noreply.github.com>
Co-authored-by: robert-a-forsyth <robert.alexander.forsyth@gmail.com>
Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
Co-authored-by: Nicolas Vannieuwkerke <101190534+nvnieuwk@users.noreply.github.com>
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
Co-authored-by: Igor Trujnara <53370556+itrujnara@users.noreply.github.com>
Co-authored-by: Maxime U Garcia <max.u.garcia@gmail.com>
Co-authored-by: John Orgera <65687576+johnoooh@users.noreply.github.com>
Co-authored-by: Usman Rashid <usman@smme.edu.pk>
Co-authored-by: Louis Le Nézet <58640615+LouisLeNezet@users.noreply.github.com>
Co-authored-by: LouisLeNezet <louislenezet@gmaio.com>
Co-authored-by: Felix Lenner <52530259+fellen31@users.noreply.github.com>
Co-authored-by: Solenne Correard <54953390+scorreard@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

nf-test migration: sourmash/taxannotate nf-test migration: sourmash/gather

2 participants