`set_start_value` silently fails for optimizers which do not support it

[araujoms]

using Hypatia, JuMP

model = Model(Hypatia.Optimizer)
@variable(model, x)
set_start_value(x, 3)
@constraint(model, 5x-2 ≥ 0)
@objective(model, Min, x)
optimize!(model)

The line set_start_value(x, 3) is silently ignored, it has no effect on the solver. Same thing happens with Clarabel.

[odow]

This is intended behavior

MathOptInterface.jl/src/Utilities/copy.jl

Line 74 in f6f7974

if attr == MOI.VariableName() || attr == MOI.VariablePrimalStart()

Start values are a hint. We long ago decided not to error. You could reasonably come to a different conclusion, but we’re not making breaking changes to MOI at this point.

[araujoms]

You could show a warning.

[odow]

We could, but I’m not very keen. We’d just be cluttering up the screen. And this isn’t something the user can do something about. And it also doesn’t change the solution etc. This is a really an issue that should be opened in the relevant solver wrappers. It’s not something MOI should deal with. Also, the skipping is only if wrappers use the default copying in MOI. If they write their own copy_to they can choose what to do.

[araujoms]

The user can do something about it: stop providing the useless starting value. A student of mine just wasted a lot of time trying to warm-start Hypatia as she believed it was supported due to the absence of errors or warnings.

[blegat]

The line is here

MathOptInterface.jl/src/Utilities/copy.jl

Line 75 in f6f7974

continue # Skipping names and start values is okay.

The reasoning is : if you write a solver independent model with starting values and then run it through many solvers, you don't want it to error on the solvers that don't support them.
Now I agree it already took me a lot of debugging time that a clear error with stacktrace would have helped.
Not sure what we can do without a breaking change though.

Ideally, I think we should have errored or at least warned and defined an attribute that, when set, would silent these errors. The default copy-to could read this attribute to see whether it should error or not
We could define this attribute and make the silent mode be default to be non breaking.
The user would set it to error or warning to help debugging.