JuliaSmoothOptimizers / SolverTools.jl

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

Create ExecutionStats #49

Closed abelsiqueira closed 6 years ago

abelsiqueira commented 6 years ago

Breaking down #35

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-2.2%) to 78.475% when pulling 6403fb944d989722514b8fbbdec746acecd25905 on abelsiqueira:create-stats into 336e695f23479b259b6a876f6b5dec3fc972c043 on JuliaSmoothOptimizers:master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-1.9%) to 72.533% when pulling 26493faa301d25e9b253537dbdae5e9638cb1559 on abelsiqueira:create-stats into 8132d6578592a8298c5ae1684f4cbf8bc87e9514 on JuliaSmoothOptimizers:master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-1.8%) to 72.652% when pulling 1e13ad0428277310540ac64ea8845f9b8e51f200 on abelsiqueira:create-stats into 8132d6578592a8298c5ae1684f4cbf8bc87e9514 on JuliaSmoothOptimizers:master.

abelsiqueira commented 6 years ago

I rebased and made the changes, removing the various ways to pass a stat. Now it's more specific, and should be easier to mantain. I also changed the counters input to an nlp input, because there is no easy way to access the counters type of a NLPModel.

abelsiqueira commented 6 years ago

Updated and rebased. I created AbstractExtraStats for the specific solver stats.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-1.4%) to 72.906% when pulling e26859edc69dd4b945f216713336e391996bae59 on abelsiqueira:create-stats into 9d98958b3a0f5b291b48ef699e76f40a05157749 on JuliaSmoothOptimizers:master.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+2.08%) to 76.364% when pulling 04b5ba712c0822251e7d9a316c565004769ef5f2 on abelsiqueira:create-stats into 9d98958b3a0f5b291b48ef699e76f40a05157749 on JuliaSmoothOptimizers:master.

abelsiqueira commented 6 years ago

Is there anything else to fix in this?

abelsiqueira commented 6 years ago

@dpo, can we merge this?

dpo commented 6 years ago

Sorry. Thank you.