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.

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

Release 0.4.0 #514

Open
wants to merge 91 commits into
base: master
Choose a base branch
from
Open

Release 0.4.0 #514

wants to merge 91 commits into from

Conversation

fellen31
Copy link
Collaborator

@fellen31 fellen31 commented Nov 22, 2024

Release 0.4.0 which brings:

  • Severus as the new default SV caller
  • LongPhase as the new default phaser
  • Family (case) level annotations for all variants
  • FOUND_IN= tags in VCFs
  • CNV and SV merging with SVDB
  • GitHub pages documentation
  • DeepVariant vcfstatsreport
  • Annotation of SVs and CNVs
  • Ranking of SVs and CNVs
  • Filtering of variants (clinical set)
  • Renamed output files, directories and parameters
  • More and improved nf-tests
  • Dynamic methods description
  • A metro map
  • and more...

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

fellen31 and others added 30 commits August 30, 2024 11:34
Rename and add nf-test to structural variant calling
Update paraphase to compress and index VCFs within the module
Remove filter_vep, add missing SNV section and PED file information t…
Change SV multisample merging to allow for other callers
* Remove fastq QC concatenation

* Fix configs

* Fix outputs and tests

* fix config formatting

* fix schema

* Fix CHANGELOG.md
* Add Zenodo badge

* Add Zenodo text
* Add CI test profiles to nf-test, and only use nf-test

* Fix configs

* Fix configs
* Sort output SNVs

* Link to issue
* Fixed, added and updated minimap2 presets

* Update CHANGELOG.md

Co-authored-by: Anders Jemt <[email protected]>

* prettier

---------

Co-authored-by: Anders Jemt <[email protected]>
* Fix release test profile paths

* Update changelog and schema
fellen31 and others added 21 commits November 5, 2024 16:43
* Fix rank SVs with multiple families

* use updated genmod modules

* fix tests
* Update nf-core modules

* fix tests

* update methylation config

* update test

* update modules

* update tests
* Add keep a changelog enforcer

* changelog
* Annotate repeats per family

* docs

* test pdiff ci

* remove stranger diff
* Refactor nallo.nf

* review suggestions
* Annotate and rank SNVs per family

* Update CHANGELOG again

* remove mistakenly added file

* Update output.md

* Update subworkflows/local/short_variant_calling/main.nf

Co-authored-by: Lucía Peña-Pérez <[email protected]>

---------

Co-authored-by: Lucía Peña-Pérez <[email protected]>
* rename parameters

* formatting

* fix files

* Update CHANGELOG.md

Co-authored-by: Daniel Schmitz <[email protected]>

* Update CHANGELOG.md

Co-authored-by: Daniel Schmitz <[email protected]>

* review suggestions

* fix CHANGELOG

* Update nextflow.config

Co-authored-by: Daniel Schmitz <[email protected]>

* Update nextflow.config

Co-authored-by: Daniel Schmitz <[email protected]>

* Update nextflow.config

Co-authored-by: Daniel Schmitz <[email protected]>

---------

Co-authored-by: Daniel Schmitz <[email protected]>
* Filter variants

* CHANGELOG

* Update CHANGELOG.md

* Update assets/schema_hgnc_ids.json

* Update subworkflows/local/filter_variants/main.nf

Co-authored-by: Daniel Schmitz <[email protected]>

* merge and review suggestions

* review suggestions

* Update subworkflows/local/filter_variants/main.nf

Co-authored-by: Anders Jemt <[email protected]>

* Review suggestions

---------

Co-authored-by: Daniel Schmitz <[email protected]>
Co-authored-by: Anders Jemt <[email protected]>
* update docs

* prettier

* update params

* prettier

* parameters

* changelog
* Update nallo.nf

* changelog
* Update README.md

* Update CHANGELOG.md
…512)

* Change missed  output directory to

* Change missed multi_sample output directory to family

* snv to snvs

* docs
* Dynamically build methods description from ch_versions

* Update CONTRIBUTING.md
Copy link

github-actions bot commented Nov 22, 2024

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

Posted for pipeline commit 21fece4

+| ✅ 185 tests passed       |+
#| ❔  22 tests were ignored |#
!| ❗   1 tests had warnings |!

❗ Test warnings:

  • included_configs - Pipeline config does not include custom configs. Please add the includeConfig line.

❔ 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
  • 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.0.2
  • Run at 2024-11-22 14:39:54

@jemten
Copy link
Collaborator

jemten commented Nov 22, 2024

Do we want to include PR #505 in the new release?

@fellen31
Copy link
Collaborator Author

Do we want to include PR #505 in the new release?

It would be nice to update TRGT, especially since the version used in the pipeline is quite old. Previously I had some issues with using stranger with newer versions of TRGT so I'd like to do a full-sized test and upload to Scout before. I'd also like to add a repeat-caller for ONT in the coming releases, and that can't be merged by trgt merge.

Let's see, perhaps we could do a smaller release where that PR is included.

@fellen31 fellen31 marked this pull request as ready for review November 22, 2024 15:22
@fellen31 fellen31 requested a review from a team as a code owner November 22, 2024 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants