{"action":"created","issue":{"url":"https://api.github.com/repos/rdeits/iris-distro/issues/52","repository_url":"https://api.github.com/repos/rdeits/iris-distro","labels_url":"https://api.github.com/repos/rdeits/iris-distro/issues/52/labels{/name}","comments_url":"https://api.github.com/repos/rdeits/iris-distro/issues/52/comments","events_url":"https://api.github.com/repos/rdeits/iris-distro/issues/52/events","html_url":"https://github.com/rdeits/iris-distro/pull/52","id":203460131,"number":52,"title":"Drop MATLAB and SWIG and switch to pybind11 for Python bindings","user":{"login":"rdeits","id":591886,"avatar_url":"https://avatars.githubusercontent.com/u/591886?v=3","gravatar_id":"","url":"https://api.github.com/users/rdeits","html_url":"https://github.com/rdeits","followers_url":"https://api.github.com/users/rdeits/followers","following_url":"https://api.github.com/users/rdeits/following{/other_user}","gists_url":"https://api.github.com/users/rdeits/gists{/gist_id}","starred_url":"https://api.github.com/users/rdeits/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rdeits/subscriptions","organizations_url":"https://api.github.com/users/rdeits/orgs","repos_url":"https://api.github.com/users/rdeits/repos","events_url":"https://api.github.com/users/rdeits/events{/privacy}","received_events_url":"https://api.github.com/users/rdeits/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2017-01-26T19:07:45Z","updated_at":"2017-01-26T22:12:35Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/rdeits/iris-distro/pulls/52","html_url":"https://github.com/rdeits/iris-distro/pull/52","diff_url":"https://github.com/rdeits/iris-distro/pull/52.diff","patch_url":"https://github.com/rdeits/iris-distro/pull/52.patch"},"body":"The matlab interface has been a burden for a long time (since I don't use it and can't test it on Travis), so I'm dropping it. Existing users can keep using an older version of IRIS with Matlab support if they need it, and we can always create a branch and backport patches if it's really really necessary. \r\n\r\nAlso, SWIG was a good choice when I needed Matlab and Python bindings, but that's no longer the case. Switching to pybind11 made everything much easier, especially the magical conversion of numpy<->eigen types. \r\n\r\nThis also cleans up the build substantially. IRIS is no longer a Pod, but rather just a regular CMake project. It also clears up some lingering licensing issues (although the issue with commercial use of cvxgen code remains). \r\n\r\nThis PR also adds python3 support. "},"comment":{"url":"https://api.github.com/repos/rdeits/iris-distro/issues/comments/275530511","html_url":"https://github.com/rdeits/iris-distro/pull/52#issuecomment-275530511","issue_url":"https://api.github.com/repos/rdeits/iris-distro/issues/52","id":275530511,"user":{"login":"rdeits","id":591886,"avatar_url":"https://avatars.githubusercontent.com/u/591886?v=3","gravatar_id":"","url":"https://api.github.com/users/rdeits","html_url":"https://github.com/rdeits","followers_url":"https://api.github.com/users/rdeits/followers","following_url":"https://api.github.com/users/rdeits/following{/other_user}","gists_url":"https://api.github.com/users/rdeits/gists{/gist_id}","starred_url":"https://api.github.com/users/rdeits/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rdeits/subscriptions","organizations_url":"https://api.github.com/users/rdeits/orgs","repos_url":"https://api.github.com/users/rdeits/repos","events_url":"https://api.github.com/users/rdeits/events{/privacy}","received_events_url":"https://api.github.com/users/rdeits/received_events","type":"User","site_admin":false},"created_at":"2017-01-26T22:12:35Z","updated_at":"2017-01-26T22:12:35Z","body":"55,000 lines deleted here. About 10,000 of that is related to removing SWIG, 40,000 from some amazingly large cython files that were just wasting space, and another 5,000 of matlab code. "}}
{"actor":{"display_login":"rdeits"}}