search

JuliaSmoothOptimizers / SolverTools.jl

Tools for developing nonlinear optimization solvers.
Other
26 stars 18 forks source link

Tests on Windows #41

Closed dpo closed 6 years ago

dpo commented 6 years ago

The Windows tests keep failing because of AmplNLReader. We should eliminate that package from the tests and e.g., hard-code one problem and its derivatives instead.

abelsiqueira commented 6 years ago

Drop only for windows?

dpo commented 6 years ago

Drop everywhere I think. This package is useable without AMPL and the tests should be platform independent.

abelsiqueira commented 6 years ago

tests should be platform independent

That would mean dropping CUTEst for Windows.

dpo commented 6 years ago

The tests don't use CUTEst?!?!

abelsiqueira commented 6 years ago

My mistake, CUTEst is on the repo REQUIRE. Will change it shortly. Or do I drop it?

dpo commented 6 years ago

Ah, that's right. But it's not really required, is it?

abelsiqueira commented 6 years ago

We use it here and here. But it's true, it's not really required, since it doesn't test anything new. We can drop it.

dpo commented 6 years ago

We use it but it isn't in REQUIRE. I believe we wanted to check that the solvers performed similarly whatever the provenance of the problem. It's useful I guess. We could keep it but there should be tests that don't need CUTEst or AmpNLReader.