Update

Author: odow

src/Bridges/Variable/set_map.jl

@@ -84,9 +84,6 @@ function MOI.get(
     bridge::SetMapBridge,
 )
     f = MOI.get(model, attr, bridge.constraint)
-    if f === nothing
-        return nothing
-    end
     return MOI.Bridges.inverse_map_function(bridge, f)
 end
 
@@ -98,12 +95,8 @@ function MOI.set(
     bridge::SetMapBridge,
     value,
 )
-    if value === nothing
-        MOI.set(model, attr, bridge.constraint, value)
-    else
-        bridged_value = MOI.Bridges.map_function(bridge, value)
-        MOI.set(model, attr, bridge.constraint, bridged_value)
-    end
+    bridged_value = MOI.Bridges.map_function(bridge, value)
+    MOI.set(model, attr, bridge.constraint, bridged_value)
     return
 end
 
@@ -143,9 +136,6 @@ function MOI.get(
     bridge::SetMapBridge,
 )
     value = MOI.get(model, attr, bridge.constraint)
-    if value === nothing
-        return nothing
-    end
     return MOI.Bridges.inverse_adjoint_map_function(typeof(bridge), value)
 end
 
@@ -157,12 +147,8 @@ function MOI.set(
     bridge::SetMapBridge,
     value,
 )
-    if value === nothing
-        MOI.set(model, attr, bridge.constraint, value)
-    else
-        bridged_value = MOI.Bridges.adjoint_map_function(bridge, value)
-        MOI.set(model, attr, bridge.constraint, bridged_value)
-    end
+    bridged_value = MOI.Bridges.adjoint_map_function(bridge, value)
+    MOI.set(model, attr, bridge.constraint, bridged_value)
     return
 end
 
@@ -235,9 +221,6 @@ function MOI.get(
     bridge::SetMapBridge,
 )
     value = MOI.get(model, attr, bridge.variable)
-    if value === nothing
-        return nothing
-    end
     return MOI.Bridges.map_function(bridge, value)
 end
 
@@ -247,12 +230,8 @@ function MOI.set(
     bridge::SetMapBridge,
     value,
 )
-    if value === nothing
-        MOI.set(model, attr, bridge.variable, nothing)
-    else
-        bridged_value = MOI.Bridges.inverse_map_function(bridge, value)
-        MOI.set(model, attr, bridge.variable, bridged_value)
-    end
+    bridged_value = MOI.Bridges.inverse_map_function(bridge, value)
+    MOI.set(model, attr, bridge.variable, bridged_value)
     return
 end
 
@@ -320,12 +299,8 @@ function MOI.set(
     value,
     i::MOI.Bridges.IndexInVector,
 )
-    if value === nothing
-        MOI.set(model, attr, bridge.variables[i.value], nothing)
-    else
-        bridged_value = MOI.Bridges.inverse_map_function(bridge, value)
-        MOI.set(model, attr, bridge.variables[i.value], bridged_value)
-    end
+    bridged_value = MOI.Bridges.inverse_map_function(bridge, value)
+    MOI.set(model, attr, bridge.variables[i.value], bridged_value)
     return
 end
 

src/Bridges/set_map.jl

@@ -67,6 +67,8 @@ function map_function(bridge::AbstractBridge, func)
     return map_function(typeof(bridge), func)
 end
 
+map_function(::AbstractBridge, ::Nothing) = nothing
+
 """
     map_function(::Type{BT}, func, i::IndexInVector) where {BT}
 
@@ -106,6 +108,8 @@ function inverse_map_function(bridge::AbstractBridge, func)
     return inverse_map_function(typeof(bridge), func)
 end
 
+inverse_map_function(::AbstractBridge, ::Nothing) = nothing
+
 """
     adjoint_map_function(bridge::MOI.Bridges.AbstractBridge, func)
     adjoint_map_function(::Type{BT}, func) where {BT}
@@ -123,6 +127,8 @@ function adjoint_map_function(bridge::AbstractBridge, func)
     return adjoint_map_function(typeof(bridge), func)
 end
 
+adjoint_map_function(::AbstractBridge, ::Nothing) = nothing
+
 """
     inverse_adjoint_map_function(bridge::MOI.Bridges.AbstractBridge, func)
     inverse_adjoint_map_function(::Type{BT}, func) where {BT}
@@ -142,3 +148,5 @@ interface is kept for backward compatibility.
 function inverse_adjoint_map_function(bridge::AbstractBridge, func)
     return inverse_adjoint_map_function(typeof(bridge), func)
 end
+
+inverse_adjoint_map_function(::AbstractBridge, ::Nothing) = nothing