{"action":"opened","issue":{"url":"https://api.github.com/repos/JuliaOpt/JuMP.jl/issues/830","repository_url":"https://api.github.com/repos/JuliaOpt/JuMP.jl","labels_url":"https://api.github.com/repos/JuliaOpt/JuMP.jl/issues/830/labels{/name}","comments_url":"https://api.github.com/repos/JuliaOpt/JuMP.jl/issues/830/comments","events_url":"https://api.github.com/repos/JuliaOpt/JuMP.jl/issues/830/events","html_url":"https://github.com/JuliaOpt/JuMP.jl/issues/830","id":171003859,"number":830,"title":"Internal Model not unloaded when SDP constraint added","user":{"login":"blegat","id":1048205,"avatar_url":"https://avatars.githubusercontent.com/u/1048205?v=3","gravatar_id":"","url":"https://api.github.com/users/blegat","html_url":"https://github.com/blegat","followers_url":"https://api.github.com/users/blegat/followers","following_url":"https://api.github.com/users/blegat/following{/other_user}","gists_url":"https://api.github.com/users/blegat/gists{/gist_id}","starred_url":"https://api.github.com/users/blegat/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/blegat/subscriptions","organizations_url":"https://api.github.com/users/blegat/orgs","repos_url":"https://api.github.com/users/blegat/repos","events_url":"https://api.github.com/users/blegat/events{/privacy}","received_events_url":"https://api.github.com/users/blegat/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-08-13T10:50:34Z","updated_at":"2016-08-13T10:50:34Z","closed_at":null,"body":"In lines [468 -> 470 of macros.jl](https://github.com/JuliaOpt/JuMP.jl/blob/8b3074af0c51cdb12c9411e9a71175614bfe2866/src/macros.jl#L468-L470) (in macro `SDconstraint`), it is written\r\n\r\n        c = SDConstraint($newaff)\r\n        push!($(m).sdpconstr, c)\r\n        c\r\n\r\nwhile I would have expected something like\r\n\r\n        addconstraint($m, SDConstraint($newaff))\r\n\r\nsince this is what is done, e.g., in the macro `constraint`. Indeed, those two blocks of code do not have the same behaviors as the second one also does `m.internalModelLoaded = false` inside the `addconstraint` function which seems important since one constraint has been added. Am I getting it wrong ?"}}
{"actor":{"display_login":"blegat"}}