Merge pull request #982 from JuliaOpt/bl/fix980

Workaround for issue #980

Author: blegat

src/Utilities/functions.jl

@@ -833,7 +833,8 @@ end
 function operate_term(::typeof(*), α::T, t::MOI.ScalarAffineTerm{T}) where T
     MOI.ScalarAffineTerm(α * t.coefficient, t.variable_index)
 end
-function operate_term(::typeof(*), t::MOI.ScalarAffineTerm{T}, β::T) where T
+# `<:Number` is a workaround for https://github.com/JuliaOpt/MathOptInterface.jl/issues/980
+function operate_term(::typeof(*), t::MOI.ScalarAffineTerm{T}, β::T) where T<:Number
     MOI.ScalarAffineTerm(t.coefficient * β, t.variable_index)
 end
 function operate_term(::typeof(*), α::T, t::MOI.ScalarAffineTerm{T}, β::T) where T
@@ -1443,9 +1444,10 @@ function operate(::typeof(*), ::Type{T},
     return operate(*, T, α, f)
 end
 
+# `<:Number` is a workaround for https://github.com/JuliaOpt/MathOptInterface.jl/issues/980
 function operate!(::typeof(*), ::Type{T},
                   f::Union{MOI.ScalarAffineFunction{T},
-                           MOI.ScalarQuadraticFunction{T}}, α::T) where T
+                           MOI.ScalarQuadraticFunction{T}}, α::T) where T<:Number
     map_terms!(term -> operate_term(*, term, α), f)
     f.constant *= α
     return f

test/issue980.jl

@@ -0,0 +1,17 @@
+module Issue980
+
+import MathOptInterface
+const MOI = MathOptInterface
+
+abstract type StaticArray end
+(::Type{SA})(x...) where {SA <: StaticArray} = SA(x)
+
+function crash()
+    model = MOI.Utilities.Model{Float64}()
+    bridged = MOI.Bridges.LazyBridgeOptimizer(MOI.Utilities.Model{Float64}())
+    MOI.copy_to(bridged, model)
+end
+
+crash()
+
+end

test/runtests.jl

@@ -6,6 +6,9 @@ const MOIB = MathOptInterface.Bridges
 
 using Test
 
+# It needs to be called first to trigger the crash.
+include("issue980.jl")
+
 # Tests for solvers are located in MOI.Test.
 
 include("dummy.jl")