Open ashander opened 7 years ago
Merging #25 into master will decrease coverage by
94.96%
. The diff coverage is33.33%
.
@@ Coverage Diff @@
## master #25 +/- ##
=========================================
- Coverage 99.36% 4.4% -94.97%
=========================================
Files 3 3
Lines 158 159 +1
=========================================
- Hits 157 7 -150
- Misses 1 152 +151
Flag | Coverage Δ | |
---|---|---|
#unit | 4.4% <33.33%> (-59.53%) |
:arrow_down: |
Impacted Files | Coverage Δ | |
---|---|---|
ftprime/argrecorder.py | 6.12% <33.33%> (-92.85%) |
:arrow_down: |
ftprime/recomb_collector.py | 0% <0%> (-100%) |
:arrow_down: |
ftprime/__init__.py | 50% <0%> (-50%) |
:arrow_down: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 20f7769...32a6fd1. Read the comment docs.
Results of line profiling arg_recorder are in devel
. I've tried to do the same (add @profile
decorator) for functions in RecombCollector
as well but ran into segfaults
WITHOUT simplification/checking/computing tree sequence or other
argrecorder.py
argrecorder.py:add_record
and 3/4 of that in stuff that is notmsprime
(which takes up total of 10% of the 46% for argrecorder.py in the flame graph from command below) --- so mostly the logic inargrecorder.py:merge_records
and buildin logic of OrderedDict.you can re-run (from project root after first
pip install vprof
):vprof -c cmp tests/test_recomb_collector.py