Update

Author: odow

ext/MathOptInterfaceCliqueTreesExt.jl

@@ -22,8 +22,8 @@ function MOI.Utilities.compute_sparse_sqrt(
         LinearAlgebra.RowMaximum(),
     )
     U = SparseArrays.sparse(G.U) * G.P
-    # Verify the factorization reconstructs Q. This catches indefinite
-    # matrices where the diagonal is all zeros (e.g., [0 -1; -1 0]).
+    # Verify the factorization reconstructs Q. We don't have something like
+    # LinearAlgebra.issuccess(G)
     if !isapprox(Q, U' * U; atol = 1e-10)
         return nothing
     end

src/Bridges/Constraint/bridges/QuadtoSOCBridge.jl

@@ -100,10 +100,10 @@ function compute_sparse_sqrt(Q, ::F, ::S) where {F,S}
     if (ret = MOI.Utilities.compute_sparse_sqrt(Q)) !== nothing
         return ret
     elseif !MOI.Utilities.is_defined(MOI.Utilities.LDLFactorizationsExt())
-        msg =_error_msg(MOI.Utilities.LDLFactorizationsExt())
+        msg = _error_msg(MOI.Utilities.LDLFactorizationsExt())
         return throw(MOI.AddConstraintNotAllowed{F,S}(msg))
     elseif !MOI.Utilities.is_defined(MOI.Utilities.CliqueTreesExt())
-        msg =_error_msg(MOI.Utilities.CliqueTreesExt())
+        msg = _error_msg(MOI.Utilities.CliqueTreesExt())
         return throw(MOI.AddConstraintNotAllowed{F,S}(msg))
     end
     msg = """

test/Bridges/Constraint/test_QuadtoSOCBridge.jl

@@ -480,7 +480,7 @@ function test_clique_trees_early_zero_pivot()
     x = MOI.add_variables(model, 3)
     # (x[1] + x[2])^2 + x[3]^2 = x[1]^2 + 2*x[1]*x[2] + x[2]^2 + x[3]^2
     # Q = [1 1 0; 1 1 0; 0 0 1]
-    f = 0.5 * x[1] * x[1] + 1.0 * x[1] * x[2] + 0.5 * x[2] * x[2] + 0.5 * x[3] * x[3]
+    f = sum(0.5 * x[i] * x[i] for i in 1:3) + 1.0 * x[1] * x[2]
     c = MOI.add_constraint(model, f, MOI.LessThan(1.0))
     F, S = MOI.VectorAffineFunction{Float64}, MOI.RotatedSecondOrderCone
     ci = only(MOI.get(inner, MOI.ListOfConstraintIndices{F,S}()))