search

lanl-ansi / Juniper.jl

A JuMP-based Nonlinear Integer Program Solver
https://lanl-ansi.github.io/Juniper.jl/stable/
MIT License
179 stars 22 forks source link

Various clean-ups in tests #234

Closed odow closed 2 years ago

odow commented 2 years ago
codecov[bot] commented 2 years ago

Codecov Report

Merging #234 (d173f2d) into master (a91731c) will decrease coverage by 0.09%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #234      +/-   ##
==========================================
- Coverage   91.86%   91.76%   -0.10%     
==========================================
  Files          20       20              
  Lines        2015     2015              
==========================================
- Hits         1851     1849       -2     
- Misses        164      166       +2
Impacted Files Coverage Δ
src/MOI_wrapper/MOI_wrapper.jl 86.53% <0.00%> (-0.49%) :arrow_down:
src/bb_strategies.jl 99.12% <0.00%> (-0.44%) :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 a91731c...d173f2d. Read the comment docs.

ccoffrin commented 2 years ago

Possibly this is still a WIP but in test/basic.jl I think I spotted a few places where Juniper.getnsolutions could be replaced by JuMP.result_count?