Closed malramsay64 closed 6 years ago
Merging #30 into master will increase coverage by
0.08%
. The diff coverage is85.29%
.
@@ Coverage Diff @@
## master #30 +/- ##
=========================================
+ Coverage 80.82% 80.9% +0.08%
=========================================
Files 17 18 +1
Lines 1241 1278 +37
=========================================
+ Hits 1003 1034 +31
- Misses 238 244 +6
Impacted Files | Coverage Δ | |
---|---|---|
src/sdanalysis/params.py | 98.57% <ø> (-0.03%) |
:arrow_down: |
src/sdanalysis/run_analysis.py | 0% <0%> (ø) |
:arrow_up: |
src/sdanalysis/figures/interactive_config.py | 31.54% <0%> (ø) |
:arrow_up: |
src/sdanalysis/molecules.py | 97.46% <100%> (ø) |
:arrow_up: |
src/sdanalysis/relaxation.py | 79.03% <100%> (ø) |
:arrow_up: |
src/sdanalysis/order.py | 93.44% <100%> (+0.98%) |
:arrow_up: |
src/sdanalysis/read.py | 91.86% <66.66%> (-0.1%) |
:arrow_down: |
src/sdanalysis/main.py | 71.42% <80%> (-0.87%) |
:arrow_down: |
src/sdanalysis/dynamics.py | 87.11% <81.81%> (-1.29%) |
:arrow_down: |
src/sdanalysis/math_util.py | 94.44% <94.44%> (ø) |
|
... and 1 more |
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 81d0bee...6d0d776. Read the comment docs.
This removes the requirement for cython, making this a python only package which only needs to be built for a single architecture. Additionally this moves all the quaternion code to the rowan package which removes the maintenance burden of that part of the code.