-
Notifications
You must be signed in to change notification settings - Fork 105
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
Update SPAdes module to official nf-core and version bump #666
base: dev
Are you sure you want to change the base?
Conversation
|
Need to check other downstreeam files to make sure if there is an issue, however it seems that the original SPADES(HYBIRD) modules exported the the main scaffolds as uncompressed for downstream.( even though it also exported a version as compressed). I think I will need to add an additional GUNZIP module after the assembly for safety... |
Added the GUNZIP module actually does somthing similar to pre-assemblies, but then local MEGAHIT module also exports uncompressed files... so I guess time to update this too But I need to udpate the MEGAHIT module to support co-assemblies (basically |
TODO:
PR checklist
nf-core 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).