Fix sequence validation to handle NaN values in DRAMP database download#538
Conversation
Release PR: 4.0.0 Chinese Baozi
This PR is against the
|
jfy133
left a comment
There was a problem hiding this comment.
Thank you very much @hindrek ! DRAMP is being the bane of my life at the moment 🙃
Simple enough fix, so it's ready to go - please however add an entry in the CHANGELOG and add yourself to the contributors list in README and the nextflow.config manifest section :), then I will merge
Updated CHANGELOG.md to include a fix.
|
Thanks @hindrek |
Fixes #537.
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).