Update

Author: odow

src/Bridges/Variable/set_map.jl

@@ -50,8 +50,11 @@ function bridge_constrained_variable(
     model::MOI.ModelLike,
     set::S2,
 ) where {T,S1,S2}
-    variables, constraint =
-        _add_constrained_var(model, MOI.Bridges.inverse_map_set(BT, set))
+    # In a Variable.SetMapBridge:
+    #  map_set goes from Bridge -> User
+    #  inverse_map_set goes from User -> Bridge
+    new_set = MOI.Bridges.inverse_map_set(BT, set)
+    variables, constraint = _add_constrained_var(model, new_set)
     return BT(variables, constraint)
 end
 
@@ -85,6 +88,9 @@ function MOI.get(
     if value === nothing
         return
     end
+    # In a Variable.SetMapBridge:
+    #  map_function goes from Bridge -> User
+    #  inverse_map_function goes from User -> Bridge
     return MOI.Bridges.map_function(bridge, value)
 end
 
@@ -98,6 +104,9 @@ function MOI.set(
         MOI.set(model, attr, bridge.constraint, nothing)
         return
     end
+    # In a Variable.SetMapBridge:
+    #  map_function goes from Bridge -> User
+    #  inverse_map_function goes from User -> Bridge
     bridged_value = MOI.Bridges.inverse_map_function(bridge, value)
     MOI.set(model, attr, bridge.constraint, bridged_value)
     return
@@ -109,6 +118,9 @@ function MOI.get(
     bridge::SetMapBridge,
 )
     set = MOI.get(model, attr, bridge.constraint)
+    # In a Variable.SetMapBridge:
+    #  map_set goes from Bridge -> User
+    #  inverse_map_set goes from User -> Bridge
     return MOI.Bridges.map_set(bridge, set)
 end
 
@@ -118,8 +130,11 @@ function MOI.set(
     bridge::SetMapBridge{T,S1},
     set::S1,
 ) where {T,S1}
-    mapped = MOI.Bridges.inverse_map_set(bridge, set)
-    MOI.set(model, attr, bridge.constraint, mapped)
+    # In a Variable.SetMapBridge:
+    #  map_set goes from Bridge -> User
+    #  inverse_map_set goes from User -> Bridge
+    mapped_set = MOI.Bridges.inverse_map_set(bridge, set)
+    MOI.set(model, attr, bridge.constraint, mapped_set)
     return
 end
 
@@ -132,7 +147,10 @@ function MOI.get(
     if value === nothing
         return
     end
-    return MOI.Bridges.adjoint_map_function(bridge, value)
+    # In a Variable.SetMapBridge:
+    #  inverse_adjoint_map_function goes from Bridge -> User
+    #  adjoint_map_function goes from User -> Bridge
+    return MOI.Bridges.inverse_adjoint_map_function(bridge, value)
 end
 
 function MOI.set(
@@ -145,7 +163,10 @@ function MOI.set(
         MOI.set(model, attr, bridge.constraint, nothing)
         return
     end
-    bridged_value = MOI.Bridges.inverse_adjoint_map_function(bridge, value)
+    # In a Variable.SetMapBridge:
+    #  inverse_adjoint_map_function goes from Bridge -> User
+    #  adjoint_map_function goes from User -> Bridge
+    bridged_value = MOI.Bridges.adjoint_map_function(bridge, value)
     MOI.set(model, attr, bridge.constraint, bridged_value)
     return
 end
@@ -160,6 +181,9 @@ end
 
 function unbridged_map(bridge::SetMapBridge{T}, vi::MOI.VariableIndex) where {T}
     F = MOI.ScalarAffineFunction{T}
+    # In a Variable.SetMapBridge:
+    #  map_function goes from Bridge -> User
+    #  inverse_map_function goes from User -> Bridge
     mapped = MOI.Bridges.inverse_map_function(bridge, vi)
     return Pair{MOI.VariableIndex,F}[bridge.variable=>mapped]
 end
@@ -222,6 +246,9 @@ function MOI.get(
     if value === nothing
         return
     end
+    # In a Variable.SetMapBridge:
+    #  map_function goes from Bridge -> User
+    #  inverse_map_function goes from User -> Bridge
     return MOI.Bridges.map_function(bridge, value)
 end
 
@@ -235,6 +262,9 @@ function MOI.set(
         MOI.set(model, attr, bridge.variable, nothing)
         return
     end
+    # In a Variable.SetMapBridge:
+    #  map_function goes from Bridge -> User
+    #  inverse_map_function goes from User -> Bridge
     bridged_value = MOI.Bridges.inverse_map_function(bridge, value)
     MOI.set(model, attr, bridge.variable, bridged_value)
     return
@@ -294,6 +324,9 @@ function MOI.get(
     if any(isnothing, value)
         return
     end
+    # In a Variable.SetMapBridge:
+    #  map_function goes from Bridge -> User
+    #  inverse_map_function goes from User -> Bridge
     return MOI.Bridges.map_function(bridge, value, i)
 end
 
@@ -308,6 +341,9 @@ function MOI.set(
         MOI.set(model, attr, bridge.variables[i.value], nothing)
         return
     end
+    # In a Variable.SetMapBridge:
+    #  map_function goes from Bridge -> User
+    #  inverse_map_function goes from User -> Bridge
     bridged_value = MOI.Bridges.inverse_map_function(bridge, value)
     MOI.set(model, attr, bridge.variables[i.value], bridged_value)
     return
@@ -337,6 +373,9 @@ function unbridged_map(
 ) where {T}
     F = MOI.ScalarAffineFunction{T}
     func = MOI.VectorOfVariables(vis)
+    # In a Variable.SetMapBridge:
+    #  map_function goes from Bridge -> User
+    #  inverse_map_function goes from User -> Bridge
     funcs = MOI.Bridges.inverse_map_function(bridge, func)
     scalars = MOI.Utilities.eachscalar(funcs)
     # FIXME not correct for SetDotProducts, it won't recover the dot product variables