Electa-Git / PowerModelsDistributionStateEstimation.jl

A Julia Package for Power System State Estimation.
BSD 3-Clause "New" or "Revised" License
32 stars 12 forks source link

PR to check bug #14

Closed MartaVanin closed 4 years ago

MartaVanin commented 4 years ago

Added:

codecov[bot] commented 4 years ago

Codecov Report

Merging #14 into master will increase coverage by 21.44%. The diff coverage is 55.31%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #14       +/-   ##
===========================================
+ Coverage   43.74%   65.18%   +21.44%     
===========================================
  Files          12       14        +2     
  Lines         871      968       +97     
===========================================
+ Hits          381      631      +250     
+ Misses        490      337      -153     
Impacted Files Coverage Δ
src/PowerModelsDSSE.jl 100.00% <ø> (ø)
src/core/constraint.jl 48.64% <ø> (ø)
src/core/variable.jl 100.00% <ø> (+18.33%) :arrow_up:
src/prob/reduced_pf.jl 0.00% <0.00%> (ø)
src/form/reduced_ivr.jl 37.81% <37.81%> (ø)
src/core/measurement_conversion.jl 61.84% <80.76%> (+60.04%) :arrow_up:
src/prob/se.jl 64.23% <84.31%> (+10.47%) :arrow_up:
src/form/adapted_pmd_constraints.jl 86.32% <97.82%> (ø)
src/form/reduced_ac.jl 100.00% <100.00%> (ø)
src/io/measurement_parser.jl 94.62% <100.00%> (+22.09%) :arrow_up:
... and 7 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 a6387c1...7f97ace. Read the comment docs.

MartaVanin commented 4 years ago

should we merge? there's a couple of test fail in mac. Would be interesting to see if it also happens on your machine or if it's just a travis thing.