Method error in value(::Function, ::Number) about jump.jl

[odow]

See #3776

I'm marking this as a bug fix, because we already had value(::Number), just not value(::Function, ::Number).

[odow]

I understand the motivation, but this makes me go hmmmmmmmm.

julia> using JuMP

julia> model = Model();

julia> @variable(model, x, start = 2);

julia> @expression(model, expr[i in 0:2], (1 + x)^i)
1-dimensional DenseAxisArray{Any,1,...} with index sets:
    Dimension 1, 0:2
And data, a 3-element Vector{Any}:
 1.0
  x + 1
  x² + 2 x + 1

julia> value.(start_value, expr)
ERROR: MethodError: no method matching value(::typeof(start_value), ::Float64)

[mitchphillipson]

Yes, that feels like it should error. Do you have a suggested workaround? In my package I have essentially the code I suggested, which isn't the best.

I'm also moderately embarrassed I didn't think to just put start_value in value. Obviously that's better than building a dictionary.

[mitchphillipson]

I'm thinking about this more and value(F, 1) feels like it should still be 1. You're taking the value and the value is always 1, regardless of the function. Does this cause issues if F is some "weird" function?