-
Notifications
You must be signed in to change notification settings - Fork 5
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
Release 0.4.0 #514
Conversation
Bump version to 0.4.0dev
Add metro map
Add metro map
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
Merge patch into dev
* 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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing work! A few comments on things that I think we should make sure work as intended
withLabel:error_ignore { | ||
errorStrategy = 'ignore' | ||
} | ||
withLabel:error_retry { | ||
errorStrategy = 'retry' | ||
maxRetries = 2 | ||
} | ||
|
||
withName: '.*:SAMTOOLS_MERGE' { | ||
withName: 'SAMTOOLS_MERGE|SAMTOOLS_INDEX' { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the base.config? I would have put it in a more specific config, but if it complicates things too much, nevermind
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was suggested to put it here previously, do you think it would be better in the subworkflow config?
|
||
cat <<-END_VERSIONS > versions.yml | ||
"${task.process}": | ||
add_most_severe_pli: v1.0 | ||
add_most_severe_pli: 1.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it make sense to version the script in addition?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think so, I added a version (as 1.0) for the MultiQC-report, but the script is from raredisease: https://github.com/nf-core/raredisease/blob/dev/bin/add_most_severe_pli.py
Do you have a good idea on how to do it, for both pipelines?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about a module? One such as DOUBLETDETECTION, with the script in a templates folder? I am not sure it solves the versioning issue completely but at least you would have an easy way to share the script
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, I was not aware that it could be done that way. I will look into it (but not before for the release), and see if we can have it as a module in nf-core.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Continuing here: nf-core/raredisease#663
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
Co-authored-by: Annick Renevey <[email protected]>
* without collect only one family per run is filtered * CHANGELOG * CHANGELOG
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great Job! 🚀
Release 0.4.0 which brings:
FOUND_IN=
tags in VCFsPR checklist
nf-core pipelines lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).nextflow run . -profile debug,test,docker --outdir <OUTDIR>
).docs/usage.md
is updated.docs/output.md
is updated.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors).