Skip to content

Conversation

@Steffengreiner
Copy link
Contributor

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

@Steffengreiner Steffengreiner changed the base branch from master to development December 8, 2022 15:48
@Steffengreiner Steffengreiner changed the title Feature/dm 583 introduce new nanopore schema Introduce new Nanopore schema to account for files generated due to second basecalling Dec 8, 2022
@Steffengreiner Steffengreiner marked this pull request as ready for review December 12, 2022 11:58
@Steffengreiner Steffengreiner requested a review from a team as a code owner December 12, 2022 11:58
wow-such-code
wow-such-code previously approved these changes Dec 12, 2022
Copy link
Member

@wow-such-code wow-such-code left a 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

Steffengreiner and others added 2 commits December 12, 2022 13:39
Co-authored-by: wow-such-code <[email protected]>
Co-authored-by: wow-such-code <[email protected]>
Copy link
Member

@wow-such-code wow-such-code left a 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

Copy link
Member

@wow-such-code wow-such-code left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏

@Steffengreiner Steffengreiner merged commit d6168a7 into development Dec 12, 2022
@Steffengreiner Steffengreiner deleted the feature/dm-583-introduce-new-nanopore-schema branch December 12, 2022 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants