Conversation
…subworkflow for alternative use of chopper for lambda-removal instead of nanolyse, and nanoq for longread filtering instead of filtlong
- Update modules - Update integration in mag and with other tools (bin_summary, gtdb-tk) - Update test - Update schema
Also, simplify bin_summary regarding bin qc
Co-authored-by: James A. Fellows Yates <[email protected]>
Post patch version bump
Add parameter to exclude unbinned data from post-binning steps
Bin QC Improvements
Fix input to single-end MEGAHIT runs
Add support for Prokka compliance mode
Important! Template update for nf-core/tools v3.1.0
Bump version for 3.3.0 release
|
sateeshperi
left a comment
There was a problem hiding this comment.
minor suggestions.
Looks good overall. I would aim to have to local modules & sub-wfs nf-tests by next release.
Co-authored-by: Sateesh_Peri <[email protected]>
Thank you! Slowly slowly getting there 😅 most of the local modules are being ripped and replaced, and I need pipeline tests done first becuase we are getting so many new contributors/ions, but I need to rejig the tests first , but that keeps getting pushed because we then get more bugs/contributors 😅 (I swear this pipeline is a full time job...) |
charles-plessy
left a comment
There was a problem hiding this comment.
I looked at all the changes except for the nf-core modules and subworkflows, which I expect to be exact copies from nf-core/modules.
charles-plessy
left a comment
There was a problem hiding this comment.
Thanks for the changes and explanations.
PR checklist
nf-core pipelines lint).nextflow run . -profile test,docker --outdir <OUTDIR>).nextflow run . -profile debug,test,docker --outdir <OUTDIR>).docs/usage.mdis updated.docs/output.mdis updated.CHANGELOG.mdis updated.README.mdis updated (including new tool citations and authors/contributors).