-
Notifications
You must be signed in to change notification settings - Fork 8
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
[FEATURE] handle short read data #175
Conversation
Codecov Report
@@ Coverage Diff @@
## master #175 +/- ##
==========================================
+ Coverage 96.95% 97.09% +0.14%
==========================================
Files 19 19
Lines 853 862 +9
==========================================
+ Hits 827 837 +10
+ Misses 26 25 -1
Continue to review full report at Codecov.
|
Signed-off-by: Lydia Buntrock <lydia.buntrock@fu-berlin.de>
e1c1e23
to
ba81aa1
Compare
Signed-off-by: Lydia Buntrock <lydia.buntrock@fu-berlin.de>
Signed-off-by: Lydia Buntrock <lydia.buntrock@fu-berlin.de>
ba81aa1
to
71a12d4
Compare
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.
LGTM
Signed-off-by: Lydia Buntrock <lydia.buntrock@fu-berlin.de>
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.
Great, thanks!
Resolves #27
We use cigar and split read method for short reads aswell.
I added a new test case for this (combination of short and long reads)
I also found a small mistake in the paired end SAM file, which gets fixed aswell.