plasmo-dev / Plasmo.jl

A Platform for Scalable Modeling and Optimization
Other
150 stars 21 forks source link

Fix nlp variable order #50

Closed jalving closed 2 years ago

jalving commented 2 years ago

This should fix an issue that @etatara was seeing with incorrect hessian-lagrangian and jacobian structures. The issue had to do with the constrained-on-creation approach that MOI uses to copy models which was leading to an incorrect variable order in an Ipopt model. Now we just always keep the variables in order due to the way Plasmo.jl has to construct the NLPBlock.