From 6292d89eaa623d86a7ca2555093b7b2c0d67b800 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Beno=C3=AEt=20Legat?= Date: Sun, 23 May 2021 18:16:41 -0400 Subject: [PATCH] Fix pass_nonvariable_constraints with bridged variables bis --- src/Bridges/bridge_optimizer.jl | 2 +- test/Bridges/lazy_bridge_optimizer.jl | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/Bridges/bridge_optimizer.jl b/src/Bridges/bridge_optimizer.jl index 1d14256596..88db69eb9d 100644 --- a/src/Bridges/bridge_optimizer.jl +++ b/src/Bridges/bridge_optimizer.jl @@ -360,7 +360,7 @@ function MOIU.pass_nonvariable_constraints( if Variable.has_bridges(Variable.bridges(dest)) # The functions may contained bridged variables which needs to be # substituted so we use the fallback. - MOIU.pass_nonvariable_constraints_fallback( + return MOIU.pass_nonvariable_constraints_fallback( dest, src, idxmap, diff --git a/test/Bridges/lazy_bridge_optimizer.jl b/test/Bridges/lazy_bridge_optimizer.jl index 869d704ff8..cef339f81b 100644 --- a/test/Bridges/lazy_bridge_optimizer.jl +++ b/test/Bridges/lazy_bridge_optimizer.jl @@ -1179,7 +1179,9 @@ end ci_bridged = first(MOI.get(model, MOI.ListOfConstraintIndices{F,S}())) attr = MOI.ConstraintFunction() @test MOI.get(bridged, attr, ci_bridged) ≈ f(vi_bridged) - ci_sdpa = first(MOI.get(model, MOI.ListOfConstraintIndices{F,S}())) + cis = MOI.get(model, MOI.ListOfConstraintIndices{F,S}()) + @test length(cis) == 1 + ci_sdpa = first(cis) func = [1.0, -1.0]'MOI.SingleVariable.(vis_sdpa) @test MOI.get(model, attr, ci_sdpa) ≈ func end