{"action":"created","issue":{"url":"https://api.github.com/repos/JuliaOpt/Convex.jl/issues/159","repository_url":"https://api.github.com/repos/JuliaOpt/Convex.jl","labels_url":"https://api.github.com/repos/JuliaOpt/Convex.jl/issues/159/labels{/name}","comments_url":"https://api.github.com/repos/JuliaOpt/Convex.jl/issues/159/comments","events_url":"https://api.github.com/repos/JuliaOpt/Convex.jl/issues/159/events","html_url":"https://github.com/JuliaOpt/Convex.jl/pull/159","id":176196561,"number":159,"title":"Adding support for optimization with complex variables","user":{"login":"Ayush-iitkgp","id":8552776,"avatar_url":"https://avatars.githubusercontent.com/u/8552776?v=3","gravatar_id":"","url":"https://api.github.com/users/Ayush-iitkgp","html_url":"https://github.com/Ayush-iitkgp","followers_url":"https://api.github.com/users/Ayush-iitkgp/followers","following_url":"https://api.github.com/users/Ayush-iitkgp/following{/other_user}","gists_url":"https://api.github.com/users/Ayush-iitkgp/gists{/gist_id}","starred_url":"https://api.github.com/users/Ayush-iitkgp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Ayush-iitkgp/subscriptions","organizations_url":"https://api.github.com/users/Ayush-iitkgp/orgs","repos_url":"https://api.github.com/users/Ayush-iitkgp/repos","events_url":"https://api.github.com/users/Ayush-iitkgp/events{/privacy}","received_events_url":"https://api.github.com/users/Ayush-iitkgp/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":8,"created_at":"2016-09-10T18:57:17Z","updated_at":"2016-09-26T14:55:20Z","closed_at":"2016-09-26T14:55:20Z","pull_request":{"url":"https://api.github.com/repos/JuliaOpt/Convex.jl/pulls/159","html_url":"https://github.com/JuliaOpt/Convex.jl/pull/159","diff_url":"https://github.com/JuliaOpt/Convex.jl/pull/159.diff","patch_url":"https://github.com/JuliaOpt/Convex.jl/pull/159.patch"},"body":"Hi,\r\nI am submitting the PR for my GSoC project under the mentorship of @madeleineudell and @dvij\r\nApart from the source code changes, it also includes tests, documentation, and an example."},"comment":{"url":"https://api.github.com/repos/JuliaOpt/Convex.jl/issues/comments/249594211","html_url":"https://github.com/JuliaOpt/Convex.jl/pull/159#issuecomment-249594211","issue_url":"https://api.github.com/repos/JuliaOpt/Convex.jl/issues/159","id":249594211,"user":{"login":"madeleineudell","id":829940,"avatar_url":"https://avatars.githubusercontent.com/u/829940?v=3","gravatar_id":"","url":"https://api.github.com/users/madeleineudell","html_url":"https://github.com/madeleineudell","followers_url":"https://api.github.com/users/madeleineudell/followers","following_url":"https://api.github.com/users/madeleineudell/following{/other_user}","gists_url":"https://api.github.com/users/madeleineudell/gists{/gist_id}","starred_url":"https://api.github.com/users/madeleineudell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/madeleineudell/subscriptions","organizations_url":"https://api.github.com/users/madeleineudell/orgs","repos_url":"https://api.github.com/users/madeleineudell/repos","events_url":"https://api.github.com/users/madeleineudell/events{/privacy}","received_events_url":"https://api.github.com/users/madeleineudell/received_events","type":"User","site_admin":false},"created_at":"2016-09-26T14:55:20Z","updated_at":"2016-09-26T14:55:20Z","body":"Ayush, I like your suggestion for a new type ComplexCoeff. I understand\nthat will be more work for you; but I think it's worth it. Let's merge this\nand make the ConicCoeff change your next PR.\n\nOn Mon, Sep 26, 2016 at 6:57 AM, Ayush Pandey <notifications@github.com>\nwrote:\n\n> I actually did try to incorporate your suggestions during the summers but\n> it seems like we would need lots of changes\n> ConicObj = DataStructures.OrderedDict{UInt64, Value}\n>\n> type ComplexConicObj\n> real_coef::ConicObj\n> imag_coef::ConicObj\n> end\n>\n> Below is the future roadmap I am presenting:\n> First of all, we would not need to redefine ConicObj as\n> *ConicObj = DataStructures.OrderedDict{UInt64, ComplexCoeff}*\n> where\n> type ComplexCoeff\n> real_coef::Value\n> imag_coef::Value\n> end\n>\n> Now, we need to redefine the methods defined in conic_form.jl for the new\n> ConicObj type. Also, we would need to overload the multiplication operator\n> for ComplexCoeff (with a matrix and a number). I believe these changes\n> won't break the existing code and would also make the code more readable.\n>\n> —\n> You are receiving this because you were mentioned.\n> Reply to this email directly, view it on GitHub\n> <https://github.com/JuliaOpt/Convex.jl/pull/159#issuecomment-249539847>,\n> or mute the thread\n> <https://github.com/notifications/unsubscribe-auth/AAyp9G835K3No-71_x1oBPojd29fuNX_ks5qt6UugaJpZM4J5zFD>\n> .\n>\n\n\n\n-- \nMadeleine Udell\nAssistant Professor, Operations Research and Information Engineering\nCornell University\nhttps://people.orie.cornell.edu/mru8/\n(415) 729-4115\n"}}
{"actor":{"display_login":"madeleineudell"}}