{"action":"created","issue":{"url":"https://api.github.com/repos/scipy/scipy/issues/4515","repository_url":"https://api.github.com/repos/scipy/scipy","labels_url":"https://api.github.com/repos/scipy/scipy/issues/4515/labels{/name}","comments_url":"https://api.github.com/repos/scipy/scipy/issues/4515/comments","events_url":"https://api.github.com/repos/scipy/scipy/issues/4515/events","html_url":"https://github.com/scipy/scipy/pull/4515","id":57323666,"number":4515,"title":"Controllability","user":{"login":"I--P","id":8004855,"avatar_url":"https://avatars.githubusercontent.com/u/8004855?v=3","gravatar_id":"","url":"https://api.github.com/users/I--P","html_url":"https://github.com/I--P","followers_url":"https://api.github.com/users/I--P/followers","following_url":"https://api.github.com/users/I--P/following{/other_user}","gists_url":"https://api.github.com/users/I--P/gists{/gist_id}","starred_url":"https://api.github.com/users/I--P/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/I--P/subscriptions","organizations_url":"https://api.github.com/users/I--P/orgs","repos_url":"https://api.github.com/users/I--P/repos","events_url":"https://api.github.com/users/I--P/events{/privacy}","received_events_url":"https://api.github.com/users/I--P/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/scipy/scipy/labels/enhancement","name":"enhancement","color":"bfe5bf"},{"url":"https://api.github.com/repos/scipy/scipy/labels/scipy.signal","name":"scipy.signal","color":"207de5"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/scipy/scipy/milestones/28","html_url":"https://github.com/scipy/scipy/milestones/0.18.0","labels_url":"https://api.github.com/repos/scipy/scipy/milestones/28/labels","id":1253031,"number":28,"title":"0.18.0","description":"","creator":{"login":"rgommers","id":98330,"avatar_url":"https://avatars.githubusercontent.com/u/98330?v=3","gravatar_id":"","url":"https://api.github.com/users/rgommers","html_url":"https://github.com/rgommers","followers_url":"https://api.github.com/users/rgommers/followers","following_url":"https://api.github.com/users/rgommers/following{/other_user}","gists_url":"https://api.github.com/users/rgommers/gists{/gist_id}","starred_url":"https://api.github.com/users/rgommers/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rgommers/subscriptions","organizations_url":"https://api.github.com/users/rgommers/orgs","repos_url":"https://api.github.com/users/rgommers/repos","events_url":"https://api.github.com/users/rgommers/events{/privacy}","received_events_url":"https://api.github.com/users/rgommers/received_events","type":"User","site_admin":false},"open_issues":19,"closed_issues":334,"state":"open","created_at":"2015-08-13T05:52:53Z","updated_at":"2016-06-12T12:41:24Z","due_on":"2016-06-14T22:00:00Z","closed_at":null},"comments":60,"created_at":"2015-02-11T14:40:28Z","updated_at":"2016-06-12T14:00:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/scipy/scipy/pulls/4515","html_url":"https://github.com/scipy/scipy/pull/4515","diff_url":"https://github.com/scipy/scipy/pull/4515.diff","patch_url":"https://github.com/scipy/scipy/pull/4515.patch"},"body":"A simple PR with two functions: ctrb() and obsv() to compute the controllability and the observability matrices of a linear system. The code is rather straightforward I'm more worried about the docstring as I'm not used to explain this in english.\r\nctrb() could be helpful for PR #4295 also as it would allow to warn the user if he tries to place poles on an (nearly) uncontrollable system."},"comment":{"url":"https://api.github.com/repos/scipy/scipy/issues/comments/225436327","html_url":"https://github.com/scipy/scipy/pull/4515#issuecomment-225436327","issue_url":"https://api.github.com/repos/scipy/scipy/issues/4515","id":225436327,"user":{"login":"rgommers","id":98330,"avatar_url":"https://avatars.githubusercontent.com/u/98330?v=3","gravatar_id":"","url":"https://api.github.com/users/rgommers","html_url":"https://github.com/rgommers","followers_url":"https://api.github.com/users/rgommers/followers","following_url":"https://api.github.com/users/rgommers/following{/other_user}","gists_url":"https://api.github.com/users/rgommers/gists{/gist_id}","starred_url":"https://api.github.com/users/rgommers/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rgommers/subscriptions","organizations_url":"https://api.github.com/users/rgommers/orgs","repos_url":"https://api.github.com/users/rgommers/repos","events_url":"https://api.github.com/users/rgommers/events{/privacy}","received_events_url":"https://api.github.com/users/rgommers/received_events","type":"User","site_admin":false},"created_at":"2016-06-12T14:00:07Z","updated_at":"2016-06-12T14:00:07Z","body":"Okay, so it looks like we won't be merging this anytime soon then. Removing the 0.18.0 milestone.\r\n\r\nI had another good look at ``python-control``, and there is indeed a bit too much overlap between the LTI stuff in ``signal`` with that package. A longer-term plan would be good. About this PR, the functions here are useful and from what I can tell they are in better shape than what is in ``python-control``. So it would be a waste to throw this code away. Maybe it can be added to ``python-control`` instead. @murrayrm @cwrowley would this make sense?"}}
{"actor":{"display_login":"rgommers"}}