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

Add test for conic model with Ipopt #238

Closed odow closed 2 years ago

odow commented 2 years ago

Closes #212

There's no need to modify Juniper or Ipopt.

ccoffrin commented 2 years ago

Should the model in this test also include a RotatedSecondOrderCone() constraint or is that bridge tested in some other way?

odow commented 2 years ago

No I should add a test

codecov[bot] commented 2 years ago

Codecov Report

Merging #238 (9edcc89) into master (6637d78) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #238   +/-   ##
=======================================
  Coverage   91.71%   91.71%           
=======================================
  Files          20       20           
  Lines        2015     2015           
=======================================
  Hits         1848     1848           
  Misses        167      167
Impacted Files Coverage Δ
src/BnBTree.jl 94.04% <0.00%> (-0.30%) :arrow_down:
src/bb_strategies.jl 99.56% <0.00%> (+0.43%) :arrow_up:

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 6637d78...9edcc89. Read the comment docs.

odow commented 2 years ago

I'll add the test in a separate PR.