-
Notifications
You must be signed in to change notification settings - Fork 0
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 input_assure.py to support updates in mlst.json allele reports with the introduction of locidex:0.2.3 #19
Conversation
… revised mlst.json report format
|
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.
Had some deja vu reviewing this one! Looks good.
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.
Thanks so much for the updates Kyla. Great work 😄 . A few small comments.
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.
Everything looks great to me. Thanks so much Kyla 😄
This PR upgrades the version of the
locidex/merge
module tolocidex:0.2.3
in the container directive. The new version of locidex includes changes to the formatting of generatedmlst.json
allele report files (such as those outputted from Mikrokondo).To ensure compatibility, the
input_assure
module and python script have been updated to accommodate these changes in themlst.json
allele reports. The test data has also been updated to reflect the newmlst.json
format.New
mlst.json
formatting:PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).CHANGELOG.md
is updated.