-
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 to accommodate changes in locidex mlst.json allele reports #28
Conversation
… Locidex mlst.json report format
|
…rom ci.yml and add to modules.conf
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.
This is amazing work @kylacochrane 😄 . Thanks so much for all these changes. I have only a few 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.
Looks good! In the reviewing the changes I noticed indirectly that the .mlst.json file is not described in README.md or docs/usage.md, adding it might not be in the scope of this PR. I could create an issue if you think it is a good idea.
@sgsutcliffe That would be fantastic! Creating an issue for this would be really helpful. I agree that it is important to add some information about the |
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.
This looks great. Thanks so much Kyla 😄
@@ -3,6 +3,15 @@ | |||
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) | |||
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html). | |||
|
|||
## [0.3.0] - 2024-08-21 |
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.
Could you also add the link to this tag like the other releases, even if the tag isn't created yet (in anticipation of when we release)?
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.
done here: cf5248e
😃
This PR upgrades the version of the
locidex/merge
module tolocidex:0.2.2
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. All pipeline tests are passing.New
mlst.json
formatting:Additionally, container directives and closures have been aligned with the
phac-nml/pipeline-standards
as outlined in the Module Software Requirements and Container RegistresPR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).nextflow run . -profile debug,test,docker --outdir <OUTDIR>
).CHANGELOG.md
is updated.