codecov-commenter
commented
4 years ago Codecov Report
Merging #107 into main will decrease coverage by
5.12%. The diff coverage is31.91%.
@@ Coverage Diff @@
## main #107 +/- ##
==========================================
- Coverage 77.63% 72.51% -5.13%
==========================================
Files 14 16 +2
Lines 1516 1681 +165
==========================================
+ Hits 1177 1219 +42
- Misses 339 462 +123 | Impacted Files | Coverage Δ | |
|---|---|---|
| thejoker/likelihood_helpers.py | 23.59% <0.00%> (-0.55%) |
:arrow_down: |
| thejoker/prior_sb2.py | 18.98% <18.98%> (ø) |
|
| thejoker/thejoker_sb2.py | 24.32% <24.32%> (ø) |
|
| thejoker/prior_helpers.py | 76.74% <50.00%> (-1.31%) |
:arrow_down: |
| thejoker/thejoker.py | 46.62% <50.00%> (+0.36%) |
:arrow_up: |
| thejoker/__init__.py | 100.00% <100.00%> (ø) |
|
| thejoker/data.py | 84.81% <100.00%> (+0.07%) |
:arrow_up: |
| thejoker/multiproc_helpers.py | 84.93% <100.00%> (ø) |
|
| thejoker/prior.py | 88.48% <100.00%> (+0.24%) |
:arrow_up: |
| ... and 1 more |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 19016b5...d876621. Read the comment docs.
Right now, this only works if the velocity measurements can be identified with the correct binary component. Ideally, the implementation would not add new classes, but this is just to show that it is possible :)
A small demo here: https://gist.github.com/bf2ed66f33e65326ca511fd78754e097