{"action":"created","issue":{"url":"https://api.github.com/repos/scipy/scipy/issues/4576","labels_url":"https://api.github.com/repos/scipy/scipy/issues/4576/labels{/name}","comments_url":"https://api.github.com/repos/scipy/scipy/issues/4576/comments","events_url":"https://api.github.com/repos/scipy/scipy/issues/4576/events","html_url":"https://github.com/scipy/scipy/pull/4576","id":59393589,"number":4576,"title":"ENH: Split signal.lti into subclasses, part of #2912","user":{"login":"befelix","id":1336104,"avatar_url":"https://avatars.githubusercontent.com/u/1336104?v=3","gravatar_id":"","url":"https://api.github.com/users/befelix","html_url":"https://github.com/befelix","followers_url":"https://api.github.com/users/befelix/followers","following_url":"https://api.github.com/users/befelix/following{/other_user}","gists_url":"https://api.github.com/users/befelix/gists{/gist_id}","starred_url":"https://api.github.com/users/befelix/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/befelix/subscriptions","organizations_url":"https://api.github.com/users/befelix/orgs","repos_url":"https://api.github.com/users/befelix/repos","events_url":"https://api.github.com/users/befelix/events{/privacy}","received_events_url":"https://api.github.com/users/befelix/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":null,"comments":24,"created_at":"2015-03-01T12:07:37Z","updated_at":"2015-04-20T06:52:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/scipy/scipy/pulls/4576","html_url":"https://github.com/scipy/scipy/pull/4576","diff_url":"https://github.com/scipy/scipy/pull/4576.diff","patch_url":"https://github.com/scipy/scipy/pull/4576.patch"},"body":"new classes: ss, tf, zpk\r\n* backwards compatible\r\n* Remove redundant information in lti\r\n* Make internal transformations more transparent"},"comment":{"url":"https://api.github.com/repos/scipy/scipy/issues/comments/94373360","html_url":"https://github.com/scipy/scipy/pull/4576#issuecomment-94373360","issue_url":"https://api.github.com/repos/scipy/scipy/issues/4576","id":94373360,"user":{"login":"befelix","id":1336104,"avatar_url":"https://avatars.githubusercontent.com/u/1336104?v=3","gravatar_id":"","url":"https://api.github.com/users/befelix","html_url":"https://github.com/befelix","followers_url":"https://api.github.com/users/befelix/followers","following_url":"https://api.github.com/users/befelix/following{/other_user}","gists_url":"https://api.github.com/users/befelix/gists{/gist_id}","starred_url":"https://api.github.com/users/befelix/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/befelix/subscriptions","organizations_url":"https://api.github.com/users/befelix/orgs","repos_url":"https://api.github.com/users/befelix/repos","events_url":"https://api.github.com/users/befelix/events{/privacy}","received_events_url":"https://api.github.com/users/befelix/received_events","type":"User","site_admin":false},"created_at":"2015-04-20T06:52:53Z","updated_at":"2015-04-20T06:52:53Z","body":"> For the names, how are these intended to be used? If you're only ever going to create objects using a generic Lti or Filter name, and it returns one of these classes, does it matter that the names are long? Explicit names are generally more clear than abbreviations.\r\n\r\nI guess it depends on your use case. I have a lot of background in Matlab and would be inclined to use the classes directly rather than ```lti``` (even though it's super nice to have one class to call for everything). However, typically these classes are called super rarely: Create your system once and then do a lot of analysis and design on top. So even though Ss/Zpk/Tf seem snappy once you're used to it, I think the goal of python code is usually to be super readable, without necessarily having to read the docs for every class / function call. I think this would be true for something like ```StateSpace(...)```, but not necessarily for ```Ss(...)```. On the other hand most people would know the abbreviations anyways, since most functions seem to use them. So I'm not sure if there is a 'best' solution."}}