{"action":"created","issue":{"url":"https://api.github.com/repos/JuliaOpt/JuMP.jl/issues/763","repository_url":"https://api.github.com/repos/JuliaOpt/JuMP.jl","labels_url":"https://api.github.com/repos/JuliaOpt/JuMP.jl/issues/763/labels{/name}","comments_url":"https://api.github.com/repos/JuliaOpt/JuMP.jl/issues/763/comments","events_url":"https://api.github.com/repos/JuliaOpt/JuMP.jl/issues/763/events","html_url":"https://github.com/JuliaOpt/JuMP.jl/pull/763","id":154078186,"number":763,"title":"handling the \"local cuts\" variant for lazy constraints and user cuts","user":{"login":"madanim","id":17952398,"avatar_url":"https://avatars.githubusercontent.com/u/17952398?v=3","gravatar_id":"","url":"https://api.github.com/users/madanim","html_url":"https://github.com/madanim","followers_url":"https://api.github.com/users/madanim/followers","following_url":"https://api.github.com/users/madanim/following{/other_user}","gists_url":"https://api.github.com/users/madanim/gists{/gist_id}","starred_url":"https://api.github.com/users/madanim/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/madanim/subscriptions","organizations_url":"https://api.github.com/users/madanim/orgs","repos_url":"https://api.github.com/users/madanim/repos","events_url":"https://api.github.com/users/madanim/events{/privacy}","received_events_url":"https://api.github.com/users/madanim/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-05-10T18:45:33Z","updated_at":"2016-05-16T19:55:21Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/JuliaOpt/JuMP.jl/pulls/763","html_url":"https://github.com/JuliaOpt/JuMP.jl/pull/763","diff_url":"https://github.com/JuliaOpt/JuMP.jl/pull/763.diff","patch_url":"https://github.com/JuliaOpt/JuMP.jl/pull/763.patch"},"body":"@mlubin\r\n\r\nThanks for the follow-up about local cuts. Here is the suggestion with an optional keyword arg for the lazyconstraint/usercut macros, to specify if the cuts are local or not. The corresponding suggestions in MathProgBase/CPLEX follow but are indeed identical to the previous ones (first pull request). Apologies for having created a new pull request instead of updating the last one.\r\n\r\nI gave it a try here starting from the latest master branches today and reusing what is done in the @variable macro in case other (optional) kw args should be added later. The suggestions are really minor, and it works (provided that CPLEX does the job on its side). However, it may not fit everyone's taste regarding names, etc. \r\n\r\nN.B. A small encoutered issue is related to the deprecation of 'MathProgBase.updatemodel!' which is still needed with the current release of the CPLEX package 'as is', so it has been temporarily added back (one line in MathProgBase/LinearQuadratic.jl) to focus on the locally valid cuts question...\r\n\r\n--> Many thanks in advance for any modifications/comments.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/JuliaOpt/JuMP.jl/issues/comments/219528762","html_url":"https://github.com/JuliaOpt/JuMP.jl/pull/763#issuecomment-219528762","issue_url":"https://api.github.com/repos/JuliaOpt/JuMP.jl/issues/763","id":219528762,"user":{"login":"mlubin","id":1733683,"avatar_url":"https://avatars.githubusercontent.com/u/1733683?v=3","gravatar_id":"","url":"https://api.github.com/users/mlubin","html_url":"https://github.com/mlubin","followers_url":"https://api.github.com/users/mlubin/followers","following_url":"https://api.github.com/users/mlubin/following{/other_user}","gists_url":"https://api.github.com/users/mlubin/gists{/gist_id}","starred_url":"https://api.github.com/users/mlubin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mlubin/subscriptions","organizations_url":"https://api.github.com/users/mlubin/orgs","repos_url":"https://api.github.com/users/mlubin/repos","events_url":"https://api.github.com/users/mlubin/events{/privacy}","received_events_url":"https://api.github.com/users/mlubin/received_events","type":"User","site_admin":false},"created_at":"2016-05-16T19:55:21Z","updated_at":"2016-05-16T19:55:21Z","body":"Could you add a mention of the ``localcut`` option in the [documentation](http://www.juliaopt.org/JuMP.jl/0.13/callbacks.html) and a test [here](https://github.com/JuliaOpt/JuMP.jl/blob/master/test/callback.jl)? Since CPLEX is the only solver which supports local cuts at the moment, you can add a check to run the test only for CPLEX. Adding a test is your insurance that we won't unintentionally break this feature in the future."}}