Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Patch to dev #565

Merged
merged 14 commits into from
Feb 18, 2025
Merged

Patch to dev #565

merged 14 commits into from
Feb 18, 2025

Conversation

fellen31
Copy link
Collaborator

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
  • 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).

@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 17, 2025 •
edited
Loading

nf-core pipelines lint overall result: Passed ✅ ⚠️

Posted for pipeline commit cccb55f

+| ✅ 182 tests passed       |+
#| ❔  23 tests were ignored |#
!| ❗  19 tests had warnings |!

❗ Test warnings:

  • nextflow_config - Config variable not found: validation.help.beforeText
  • nextflow_config - Config variable not found: validation.help.afterText
  • nextflow_config - Config variable not found: validation.summary.beforeText
  • nextflow_config - Config variable not found: validation.summary.afterText
  • 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
  • local_component_structure - add_most_severe_consequence.nf in modules/local should be moved to a TOOL/SUBTOOL/main.nf structure
  • local_component_structure - merge_json.nf in modules/local should be moved to a TOOL/SUBTOOL/main.nf structure
  • local_component_structure - cramino.nf in modules/local should be moved to a TOOL/SUBTOOL/main.nf structure
  • local_component_structure - add_most_severe_pli.nf in modules/local should be moved to a TOOL/SUBTOOL/main.nf structure
  • local_component_structure - yak.nf in modules/local should be moved to a TOOL/SUBTOOL/main.nf structure
  • local_component_structure - genome_assembly.nf in subworkflows/local should be moved to a SUBWORKFLOW_NAME/main.nf structure
  • local_component_structure - assembly_variant_calling.nf in subworkflows/local should be moved to a SUBWORKFLOW_NAME/main.nf structure
  • local_component_structure - prepare_genome.nf in subworkflows/local should be moved to a SUBWORKFLOW_NAME/main.nf structure
  • local_component_structure - bam_infer_sex.nf in subworkflows/local should be moved to a SUBWORKFLOW_NAME/main.nf structure
  • local_component_structure - annotate_consequence_pli.nf in subworkflows/local should be moved to a SUBWORKFLOW_NAME/main.nf structure
  • local_component_structure - call_paralogs.nf in subworkflows/local should be moved to a SUBWORKFLOW_NAME/main.nf structure
  • local_component_structure - qc_aligned_reads.nf in subworkflows/local should be moved to a SUBWORKFLOW_NAME/main.nf structure
  • local_component_structure - phasing.nf in subworkflows/local should be moved to a SUBWORKFLOW_NAME/main.nf structure
  • local_component_structure - convert_input_files.nf in subworkflows/local should be moved to a SUBWORKFLOW_NAME/main.nf structure

❔ Tests ignored:

  • files_exist - File is ignored: CITATIONS.md
  • files_exist - File is ignored: CODE_OF_CONDUCT.md
  • files_exist - File is ignored: assets/nf-core-nallo_logo_light.png
  • files_exist - File is ignored: docs/README.md
  • files_exist - File is ignored: docs/images/nf-core-nallo_logo_light.png
  • files_exist - File is ignored: docs/images/nf-core-nallo_logo_dark.png
  • files_exist - File is ignored: .github/ISSUE_TEMPLATE/config.yml
  • files_exist - File is ignored: .github/workflows/awstest.yml
  • files_exist - File is ignored: .github/workflows/awsfulltest.yml
  • files_exist - File is ignored: conf/modules.config
  • files_exist - File is ignored: conf/igenomes_ignored.config
  • files_exist - File is ignored: conf/test_full.config
  • nextflow_config - Config variable ignored: manifest.name
  • nextflow_config - Config variable ignored: manifest.homePage
  • files_unchanged - File ignored due to lint config: CODE_OF_CONDUCT.md
  • files_unchanged - File ignored due to lint config: .github/CONTRIBUTING.md
  • files_unchanged - File ignored due to lint config: .github/ISSUE_TEMPLATE/bug_report.yml
  • files_unchanged - File ignored due to lint config: assets/nf-core-nallo_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-nallo_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-nallo_logo_dark.png
  • files_unchanged - File ignored due to lint config: docs/README.md
  • files_unchanged - File ignored due to lint config: .gitignore or .prettierignore
  • modules_config - modules_config

✅ Tests passed:

Run details

  • nf-core/tools version 3.2.0
  • Run at 2025-02-17 12:54:42

@fellen31 fellen31 marked this pull request as ready for review February 17, 2025 13:18
@fellen31 fellen31 requested a review from a team as a code owner February 17, 2025 13:18
jemten
jemten approved these changes Feb 18, 2025
View reviewed changes
Copy link
Collaborator

@jemten jemten left a comment

Choose a reason for hiding this comment

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

👍

@fellen31 fellen31 merged commit 9d926b1 into dev Feb 18, 2025
24 checks passed
@fellen31 fellen31 deleted the patch branch February 18, 2025 12:43
fellen31 added a commit that referenced this pull request Mar 3, 2025
@fellen31 @padraicc
@jemten @rannick
Release 0.5.0 (#581)
74ff9f3 
* bump version (#530)

* Add credits (#531)

* Add missing credits

* update changelog

* Assemble ONT genomes (#516)

* Assemble ONT genomes

* hifiasm 0.24.0

* update tests

* update patch

* Template update for nf-core/tools version 3.1.1 (#532)

* Initial template commit

* Template update for nf-core/tools version 3.1.2.dev0

* Template update for nf-core/tools version 3.1.1

* changelog

* merge mistake

* merge mistake

* merge mistake

* merge mistakes

* fix lint

* update ci

* fix version

* fix tests

* fix RO crate

* remove nf-core reference

* bump deepvariant version (#535)

* Output per-base depth in d4 format (#537)

* output per-base depth in d4 format

* Changelog

* update nf-core module

* Downgrade to Sniffles1 (#536)

* Downgrade to Sniffles1

* Update CHANGELOG.md

* bump deepvariant version (#535)

* fix readme

* Add skip for SV calling

* Template sync 3.1.2 (#541)

* Initial template commit

* Template update for nf-core/tools version 3.1.2.dev0

* Template update for nf-core/tools version 3.1.2.dev0

* Template update for nf-core/tools version 3.1.2.dev0

* Template update for nf-core/tools version 3.1.1

* Template update for nf-core/tools version 3.1.1

* use ro-crate command

* changelog

* update workflow dependencies

* make sv filtering conditional on skip_sv_calling param

* style: fix text alignment

* style: fix linting issue

* update changelog

* fix linting issues in CHANGELOG.md param table

* Run CI tests with latest-stable version of Nextflow (#545)

* Run CI tests with latest-stable version of nextflow

* Changelog

* Fix naming mistakes in output docs (#546)

* Fix naming mistakes in output docs

* Changelog and prettier

* Update docs/output.md

Co-authored-by: Anders Jemt <jemten@users.noreply.github.com>

* prettier

---------

Co-authored-by: Anders Jemt <jemten@users.noreply.github.com>

* change vcf headers, merge by sample, merge by case, first draft for merging paraphase json

* add script to merge json files

* update changelog

* fixes

* add arg for bcftools query

* add prefix to CREATE_SAMPLES_FILE

* use arg2

* modify create_sample_files to add the haplotype in the sample name

* haplotype specific sample names, force single

* Fix naming for SNVs and pipeline not running with (#556)

* create empty channels to use

* update paraphase

* updated snapshots

* update snapshots

* Update CHANGELOG.md

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

* Update modules/local/merge_json.nf

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

* Update modules/local/merge_json.nf

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

* Update conf/modules/call_paralogs.config

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

* Update conf/modules/call_paralogs.config

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

* Update modules/local/merge_json.nf

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

* Apply suggestions from code review

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

* case to family and aligned bracket

* multi-argument formatting

* use groupTuple

* add merge json

* add executing permissions to script

* update script

* update script

* update script

* add to software references

* fix config issue

* remove snapshots

* update snaphshots

* exclude vcf and tbi unpublished

* update snaps

* remove snap

* add publish dir

* update snap

* revert change

* paralogs false default publish, remove publishing for all but family merged vcf

* use family_id directory

* prettier

* updated modules added

* update output.md

* publish family json

* update snapshots

* use sample and family directories, update mapping for family merging

* update snapshots

* Update docs/output.md

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

* prettier

* Limit VEP to 1 cpu while bugged (#558)

* Bump Severus to 1.3 (#557)

* Bump Severus version

* Update CHANGELOG.md

* Always run all SV callers (#542)

* Downgrade to Sniffles1

* Update CHANGELOG.md

* bump deepvariant version (#535)

* Always run multiple SV-callers

* changelog

* changelog

* update tests and ci

* Also output CNV-calls

* CHANGELOG

* update conf

* wip

* update docs+tests

* remove rouge versions.yml

* Template update to v3.2.0 (#560)

* Initial template commit

* Template update for nf-core/tools version 3.1.2.dev0

* Template update for nf-core/tools version 3.1.1

* Template update for nf-core/tools version None

* Template update for nf-core/tools version None

* CHANGELOG

* Patch to dev (#565)

* Fix pipeline can't run without

* CHANGELOG

* docs

* docs

* update snaps

* fix CI

* Update CHANGELOG.md

* changelog date

* Add vep_plugin_files to files check

* CHANGELOG

* Update CHANGELOG.md

* always failing CI because of nf-core update

* Replace dipcall (#566)

* wip

* Replace dipcall

* Replace dipcall

* description

* Update CHANGELOG

* ignore bam md5

* CHANGELOG

* Update main.nf

* Update workflows/nallo.nf

Co-authored-by: Anders Jemt <jemten@users.noreply.github.com>

---------

Co-authored-by: Anders Jemt <jemten@users.noreply.github.com>

* Fix filter SNVs tries to run even with --skip_snv_calling (#570)

* Fix filter SNVs tries to run even with --skip_snv_calling

* CHANGELOG

* Update CHANGELOG.md

* Fix unsorted sniffles1 variants (#572)

* Fix unsorted sniffles1 variants

* update tests

* Update metro map (#573)

* CHANGELOG

* Add contributors to `nextflow.config` (#574)

* Add contributors to config

* Add missing

* Fix a few language server issues (#533)

* start fixing language server issues

* revert some changes, remove encode

* fix some more

* Update main.nf

* update methylation tests

* Fix nextflow.config issues (#578)

* Fix sniffles1 (#580)

* Fix sniffles1

* fix lint

* Add missing nf-test triggers

* update snaps

* remove missed dipcall references (#577)

* Merge master into dev (#576)

* Fix pipeline not running without `--vep_cache` (#553)

* Fix pipeline can't run without

* CHANGELOG

* docs

* docs

* update snaps

* fix CI

* Update CHANGELOG.md

* changelog date

* Add vep_plugin_files to files check

* CHANGELOG

* Update CHANGELOG.md

* always failing CI because of nf-core update

* CHANGELOG

* Update CHANGELOG.md

* Fix project samplesheet validation (#584)

* Fix mistake in usage docs (#586)

* Update usage.md

* changelog

* Prep release 0.5.0 (#585)

* Bump version 0.5.0

* CHANGELOG

* update snaps

* Review comments (#589)

---------

Co-authored-by: padraicc <padraic.corcoran@scilifelab.uu.se>
Co-authored-by: Pádraic Corcoran <padraicc@users.noreply.github.com>
Co-authored-by: padraicc <padraic.corcoran£scilifelab.uu.se>
Co-authored-by: Anders Jemt <jemten@users.noreply.github.com>
Co-authored-by: Annick Renevey <47788523+rannick@users.noreply.github.com>
Co-authored-by: Annick Renevey <annick.renevey@scilifelab.se>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jemten jemten jemten approved these changes

Assignees
No one assigned
Labels
None yet
Projects
Nallo
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fellen31 @jemten