Closed HeloiseS closed 4 years ago
Merging #13 into master will increase coverage by
0.36%
. The diff coverage is98.30%
.
@@ Coverage Diff @@
## master #13 +/- ##
==========================================
+ Coverage 72.42% 72.78% +0.36%
==========================================
Files 8 9 +1
Lines 602 610 +8
==========================================
+ Hits 436 444 +8
Misses 166 166
Impacted Files | Coverage Δ | |
---|---|---|
hoki/utils/hoki_object.py | 85.71% <85.71%> (ø) |
|
hoki/age_utils.py | 88.75% <100.00%> (+0.77%) |
:arrow_up: |
hoki/cmd.py | 38.16% <100.00%> (+0.47%) |
:arrow_up: |
hoki/hrdiagrams.py | 69.11% <100.00%> (-0.45%) |
: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 69c5c71...8f92ae9. Read the comment docs.
Refactors
calculate_pdfs
is nowcalculate_individual_pdfs
and the functionality has been split withcalculate_distributions
(see below)multiple_pdfs
fully deprecated.It has come to my attention that multiplying the probability density functions of the ages of the sources in a cluster was incorrect since these ages are correlated.
New Functions
calculate_distributions
: Searches through CMD/HRDs to collate the age distributions for a set of locations on the CMD/HRDs