search

TomTranter / JellyBaMM

Couple pybamm and openpnm to solve a thermal battery problem
BSD 3-Clause "New" or "Revised" License
11 stars 5 forks source link

remove redundant functions #8

Closed TomTranter closed 3 years ago

codecov[bot] commented 3 years ago

Codecov Report

Merging #8 (6645588) into master (63031bb) will increase coverage by 13.51%. The diff coverage is 63.47%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master       #8       +/-   ##
===========================================
+ Coverage   77.62%   91.14%   +13.51%     
===========================================
  Files          15       13        -2     
  Lines        2454     2145      -309     
===========================================
+ Hits         1905     1955       +50     
+ Misses        549      190      -359
Impacted Files Coverage Δ
ecm/__utilities__.py 45.45% <19.35%> (-35.32%) :arrow_down:
tests/unit/test_funcs.py 92.30% <50.00%> (-7.70%) :arrow_down:
ecm/__postprocess__.py 90.57% <55.17%> (+21.75%) :arrow_up:
tests/unit/test_postprocess.py 91.66% <97.43%> (-0.56%) :arrow_down:
ecm/__definitions__.py 100.00% <100.00%> (ø)
ecm/__funcs__.py 91.00% <100.00%> (+16.84%) :arrow_up:
utility/test_SOC.py 100.00% <100.00%> (ø)

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 63031bb...6645588. Read the comment docs.