Closed harshangrjn closed 5 years ago
@kaarthiksundar Once the tests pass, confirm if this PR is good to merge.
Merging #130 into master will decrease coverage by
0.2%
. The diff coverage is58.62%
.
@@ Coverage Diff @@
## master #130 +/- ##
==========================================
- Coverage 77.28% 77.08% -0.21%
==========================================
Files 15 15
Lines 3135 3155 +20
==========================================
+ Hits 2423 2432 +9
- Misses 712 723 +11
Impacted Files | Coverage Δ | |
---|---|---|
src/presolve.jl | 92.17% <0%> (ø) |
:arrow_up: |
src/nlexpr.jl | 94.29% <100%> (+0.07%) |
:arrow_up: |
src/operators.jl | 74.86% <100%> (+0.03%) |
:arrow_up: |
src/solver.jl | 97.43% <100%> (+0.01%) |
:arrow_up: |
src/algorithm.jl | 74.48% <22.22%> (-2.64%) |
:arrow_down: |
src/amp.jl | 89% <50%> (-0.89%) |
:arrow_down: |
src/log.jl | 92.42% <60%> (-1.38%) |
: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 423d9ae...6f8ec01. Read the comment docs.
126 and other misc. bug fixes for feasibility NLPs.