-
Notifications
You must be signed in to change notification settings - Fork 9
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
UMI Support for hts_SuperDeduper! #261
Merged
Merged
Changes from all commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
290aa54
hts_SuperDeduper umi support init
bnjenner 129bb4d
hts_SuperDeduper UMI Dedup implemented
bnjenner 9003e62
umi parameter rename and update test JSON files for SuperDeduper
bnjenner 8d36ed0
hts_ExtractUMI del paramter
bnjenner 45a370d
Revert "umi parameter rename and update test JSON files for SuperDedu…
bnjenner 89d64c8
merge with master branch for hts_ExtractUMI additional params
bnjenner 3095dc9
Working implementation of UMI mode in hts_SuperDeduper
bnjenner ed85968
hts_SuperDeduper help typo fix
bnjenner 68af22b
bitjoin() fix, watches for key to return boost::none
bnjenner 0a4010c
update hts_ExtractUMI
bnjenner 756b809
hts_ExtractUMI hotfix
bnjenner e0180c2
umi-support for hts_SuperDeduper
bnjenner 61465bc
hts_ExtractUMI typo
bnjenner 513fb47
Merge branch 's4hts:master' into master
bnjenner f52f4d3
hts_ExtractUMI merge conflict resolve
bnjenner 76e350e
Merge branch 'master' of github.com:bnjenner/HTStream
bnjenner 0d89a74
More explicit extraction of UMI for DRAGEN format, hts_SuperDeduper
bnjenner 370a436
Error message update, switch to const get_reads()
bnjenner 8987f88
get_umi(), result.size() < 8
bnjenner 1b3a7ee
remove SuperDeduper logic from bit_join()
bnjenner File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
result.size() < 7
should that be< 8
? you are getting the 8th element here.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.
So the read ID can actually only have 7 fields, the 8th field does not exist on a lot reads and is optionally for the UMI, from my understanding. That is why we I check if there are less than 7, which I understood to be the minimum.
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.
my point is you will get undefined behavior reading
result[7]
if the vector is size 7, so you need to check the size here.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.
OH, you're totally right, I don't know why I thought the check would be < 7, it is definitely < 8 here. Illumina headers would definitely need 8 fields in this situation. Making the changes now.