Closed malramsay64 closed 6 years ago
Merging #43 into master will increase coverage by
2.75%
. The diff coverage is98.3%
.
@@ Coverage Diff @@
## master #43 +/- ##
=========================================
+ Coverage 80.45% 83.2% +2.75%
=========================================
Files 18 18
Lines 1310 1334 +24
=========================================
+ Hits 1054 1110 +56
+ Misses 256 224 -32
Impacted Files | Coverage Δ | |
---|---|---|
src/sdanalysis/util.py | 89.74% <ø> (ø) |
:arrow_up: |
src/sdanalysis/relaxation.py | 97.32% <100%> (+18.28%) |
:arrow_up: |
src/sdanalysis/main.py | 86.11% <85.71%> (+10.6%) |
:arrow_up: |
src/sdanalysis/dynamics.py | 84.02% <0%> (+0.51%) |
:arrow_up: |
src/sdanalysis/read.py | 94.56% <0%> (+3.8%) |
:arrow_up: |
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 de8cd81...712b440. Read the comment docs.
This updates the relaxation computations to be compatible with the newer method of calculating the dynamics quantities. This change makes everything more robust to running on multiple cores simultaneously or making this into a distributed job.
As part of this updated there is a significant improvement to the testing of functions, in particular ensuring the relaxations can be calculated from the output of the dynamics. This should avoid further issues with updates to either system.
Fixes #41