Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nlp_008_011 has a better solution than the recorded one #21

Closed
xhub opened this issue Oct 5, 2019 · 2 comments · Fixed by #55
Closed

nlp_008_011 has a better solution than the recorded one #21

xhub opened this issue Oct 5, 2019 · 2 comments · Fixed by #55

Comments

@xhub
Copy link

xhub commented Oct 5, 2019

It looks like the nlp_008_011 problem has a better solution than the recorded one. This was found by KNITRO:

> value.([x,y,z])
3-element Array{Float64,1}:
  0.006189119075609131
  3.717464829723498   
 -3.719719919556328

julia-1.2> value(y) - (exp(-value(x)-2) + exp(-value(z)-2) - 2)
-9.800737998943987e-11

julia-1.2> value(x)^2 - value(y)^2 - value(z)^2
-27.65582273498036

julia-1.2> value(y) - value(x)/2 - value(z)                                                                                                                
7.4340901897420215                                                                                                                                         
                                                                                                                                                                                                                                                                                                                                                                                                                     
julia-1.2> value(x) + value(y)^2 + value(z)^3                                                                                                              
-37.641487400482944 

There is a slight infeasibility for the 1rst constraint, other it all looks good. I check the model against BARON, Couenne, Antigone, and they all find this solution.

@ccoffrin
Copy link
Collaborator

ccoffrin commented Nov 1, 2019

@xhub thanks for reporting this. My recommendation would be to modify the test case so that all solvers find the same optimal solution. These tests should be "mathematically" trivial and focus only on testing the JuMP/MOI APIs.

A PR would be welcome and appreciated.

@odow
Copy link
Member

odow commented Nov 13, 2024

Ditto with 008_010.

I'll think about changing the test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

3 participants