Skip to content

Commit c3708aa

Browse files
authored
Fix incorrect usage of Utilities.operate! (#2207)
1 parent 4526e6b commit c3708aa

File tree

5 files changed

+9
-9
lines changed

5 files changed

+9
-9
lines changed

src/Bridges/Constraint/bridges/count_belongs.jl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -312,7 +312,7 @@ function MOI.Bridges.final_touch(
312312
end
313313
end
314314
end
315-
MOI.Utilities.operate!(-, T, f, scalars[1])
315+
f = MOI.Utilities.operate!(-, T, f, scalars[1])
316316
push!(
317317
bridge.equal_to,
318318
MOI.Utilities.normalize_and_add_constraint(

src/Bridges/Constraint/bridges/count_distinct.jl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -354,7 +354,7 @@ function MOI.Bridges.final_touch(
354354
)
355355
end
356356
count_f = MOI.ScalarAffineFunction(count_terms, zero(T))
357-
MOI.Utilities.operate!(-, T, count_f, scalars[1])
357+
count_f = MOI.Utilities.operate!(-, T, count_f, scalars[1])
358358
push!(
359359
bridge.equal_to,
360360
MOI.Utilities.normalize_and_add_constraint(

src/Bridges/Constraint/bridges/count_distinct_reif.jl

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -377,9 +377,9 @@ function MOI.Bridges.final_touch(
377377
MOI.add_constraint(model, z[2], MOI.ZeroOne())
378378
# ∑y - n - δ⁺ + δ⁻ = 0
379379
f_0 = MOI.ScalarAffineFunction(count_terms, zero(T))
380-
MOI.Utilities.operate!(-, T, f_0, scalars[2])
381-
MOI.Utilities.operate!(-, T, f_0, z[3])
382-
MOI.Utilities.operate!(+, T, f_0, z[4])
380+
f_0 = MOI.Utilities.operate!(-, T, f_0, scalars[2])
381+
f_0 = MOI.Utilities.operate!(-, T, f_0, z[3])
382+
f_0 = MOI.Utilities.operate!(+, T, f_0, z[4])
383383
push!(
384384
bridge.equal_to,
385385
MOI.Utilities.normalize_and_add_constraint(

src/Bridges/Constraint/bridges/norm_to_power.jl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ function bridge_constraint(
5858
for ri in r
5959
f = MOI.Utilities.operate!(+, T, f, ri)
6060
end
61-
MOI.Utilities.operate!(-, T, f, fi_s[1])
61+
f = MOI.Utilities.operate!(-, T, f, fi_s[1])
6262
equal_ci = MOI.add_constraint(model, f, MOI.EqualTo(zero(T)))
6363
return NormToPowerBridge{T,F}(power_ci, r, equal_ci, s)
6464
end

src/Bridges/Constraint/bridges/quad_to_soc.jl

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -414,11 +414,11 @@ function MOI.get(
414414
for i in 3:MOI.output_dimension(f)
415415
term = MOI.Utilities.operate(*, T, fs[i], fs[i])
416416
term = MOI.Utilities.operate!(/, T, term, 2 * one(T))
417-
MOI.Utilities.operate!(+, T, q, term)
417+
q = MOI.Utilities.operate!(+, T, q, term)
418418
end
419-
MOI.Utilities.operate!(-, T, q, fs[2])
419+
q = MOI.Utilities.operate!(-, T, q, fs[2])
420420
if !b.less_than
421-
MOI.Utilities.operate!(-, T, q)
421+
q = MOI.Utilities.operate!(-, T, q)
422422
end
423423
q.constant += b.set_constant
424424
return q

0 commit comments

Comments
 (0)