JuliaSmoothOptimizers / AmplNLReader.jl

Julia AMPL Models Conforming to NLPModels.jl
Other
18 stars 10 forks source link

Drop ampl/mp #53

Closed dpo closed 5 years ago

dpo commented 5 years ago

This resolves #52 and #13.

dpo commented 5 years ago

It seems ok, though the dreaded out-of-range indices showed up on Win 32bit: https://ci.appveyor.com/project/dpo/amplnlreader-jl/builds/19409121/job/j5fdijtv6v9xc8cj#L340.

Would you mind testing on Linux locally when you get a chance?

abelsiqueira commented 5 years ago

Thanks.

dpo commented 5 years ago

Great. Linux support is back and Windows 64bit as a bonus! It feels much lighter to not use ampl/mp.