-
Notifications
You must be signed in to change notification settings - Fork 4
Introduce new Nanopore schema to account for files generated due to second basecalling #344
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
Introduce new Nanopore schema to account for files generated due to second basecalling #344
Conversation
src/main/groovy/life/qbic/datamodel/instruments/OxfordNanoporeInstrumentOutputV4.groovy
Outdated
Show resolved
Hide resolved
src/main/resources/schemas/nanopore-instrument-output_v2.schema.json
Outdated
Show resolved
Hide resolved
src/main/resources/schemas/nanopore-instrument-output_v3.schema.json
Outdated
Show resolved
Hide resolved
wow-such-code
left a comment
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.
nice work! especially including all of the tests for new file types
Co-authored-by: wow-such-code <[email protected]>
Co-authored-by: wow-such-code <[email protected]>
wow-such-code
left a comment
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.
oops, forgot about my suggestions. also please increase snapshot version, we forgot that after the last release
Co-authored-by: wow-such-code <[email protected]>
wow-such-code
left a comment
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.
👏
What was changed
This PR introduces a new nanopore schema to account for files generated during the performance of a manual higher accuracy basecalling after the initial fast basecall.
More information
Addresses DM-583