FirstOrderLp.jl Default tolerances

There appears to be no defaults for

absolute_optimality_tol relative_optimality_tol eps_primal_infeasible eps_dual_infeasible

However if these values are not specified the solver does not throw an error.

Nov 26 '22 22:11 ohinder

Looks like the defaults are here: https://github.com/google-research/FirstOrderLp.jl/blob/e2792c8dd7edde6d6d20da16954dfeb2db42eca8/src/termination.jl#L102-L105

Nov 26 '22 22:11 mlubin

Thanks! I think I might have even written that code. Haha. Do you think we should point to this in solver_qp.jl?

Nov 26 '22 22:11 ohinder

I got a little confused when looking at solver_qp.jl at the command line arguments help descriptions to understand what happens by default.

Nov 26 '22 22:11 ohinder

Maybe we could add something to the --help?

Nov 26 '22 22:11 ohinder

We have this: https://github.com/google-research/FirstOrderLp.jl/blob/4ccab200d4247dec930754fa47e6517860dd45ec/scripts/solve_qp.jl#L367-L370

What do you propose to add to --help?

Nov 26 '22 23:11 mlubin

Oh I missed that. I guess I was suggesting adding to --help the default value for each of these parameters or a pointer to termination.jl. Admittedly, I missed this comment so maybe it is fine as it is.

Nov 26 '22 23:11 ohinder

Copying the default values in the help message is a bad idea because they will become stale if the defaults change. Other tweaks to --help would be fine.

Nov 27 '22 01:11 mlubin