jump-dev / Pajarito.jl

A solver for mixed-integer convex optimization
Mozilla Public License 2.0
131 stars 22 forks source link

add comments for add_cut_sdp! #262

Closed mlubin closed 7 years ago

mlubin commented 7 years ago

So that SOC cuts can be understood without extra context. The V argument is mysterious.

mlubin commented 7 years ago

The comment:

                    # Use norm and transformation from RSOC to SOC
                    # yz >= ||x||^2, y,z >= 0 <==> norm2(2x, y-z) <= y + z

is also a bit confusing since everywhere else we use RSOC to mean the version with the 2 in front.

chriscoey commented 7 years ago

True, OK. Same in initial sdp soc cuts

On Feb 27, 2017 2:08 PM, "Miles Lubin" notifications@github.com wrote:

The comment:

                # Use norm and transformation from RSOC to SOC
                # yz >= ||x||^2, y,z >= 0 <==> norm2(2x, y-z) <= y + z

is also a bit confusing since everywhere else we use RSOC to mean the version with the 2 in front.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/mlubin/Pajarito.jl/issues/262#issuecomment-282817898, or mute the thread https://github.com/notifications/unsubscribe-auth/AJq0kyCod3yyZX8mHDzZD3GL5lYYLVXsks5rgx84gaJpZM4MNdyX .