-
Notifications
You must be signed in to change notification settings - Fork 86
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
flat (0) maps for non-existing chrom maps #33
flat (0) maps for non-existing chrom maps #33
Conversation
This comment has been minimized.
This comment has been minimized.
Codecov Report
@@ Coverage Diff @@
## master #33 +/- ##
=========================================
+ Coverage 94.32% 96.92% +2.6%
=========================================
Files 8 8
Lines 282 293 +11
Branches 14 16 +2
=========================================
+ Hits 266 284 +18
+ Misses 13 5 -8
- Partials 3 4 +1
Continue to review full report at Codecov.
|
6f2cd92
to
1ec4b98
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.
This is a good start @jgallowa07, but we need to be a lot more careful about the corner cases. See the comments above (sorry, I should have made them part of the review).
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.
Looking good, thanks @jgallowa07. A few more minor changes and I think we're there.
375acea
to
ae0e4ad
Compare
@petrelharp @andrewkern does this look good to you? |
looks good @jgallowa07! |
Simple fix for defining a recombination map across all chromosomes on a species genome.