{"action":"created","issue":{"url":"https://api.github.com/repos/pygeode/pygeode/issues/83","labels_url":"https://api.github.com/repos/pygeode/pygeode/issues/83/labels{/name}","comments_url":"https://api.github.com/repos/pygeode/pygeode/issues/83/comments","events_url":"https://api.github.com/repos/pygeode/pygeode/issues/83/events","html_url":"https://github.com/pygeode/pygeode/issues/83","id":115373976,"number":83,"title":"Move operations to more consistent locations in the library","user":{"login":"aph42","id":11032070,"avatar_url":"https://avatars.githubusercontent.com/u/11032070?v=3","gravatar_id":"","url":"https://api.github.com/users/aph42","html_url":"https://github.com/aph42","followers_url":"https://api.github.com/users/aph42/followers","following_url":"https://api.github.com/users/aph42/following{/other_user}","gists_url":"https://api.github.com/users/aph42/gists{/gist_id}","starred_url":"https://api.github.com/users/aph42/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aph42/subscriptions","organizations_url":"https://api.github.com/users/aph42/orgs","repos_url":"https://api.github.com/users/aph42/repos","events_url":"https://api.github.com/users/aph42/events{/privacy}","received_events_url":"https://api.github.com/users/aph42/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/pygeode/pygeode/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/pygeode/pygeode/labels/Priority-Medium","name":"Priority-Medium","color":"ededed"}],"state":"open","locked":false,"assignee":{"login":"aph42","id":11032070,"avatar_url":"https://avatars.githubusercontent.com/u/11032070?v=3","gravatar_id":"","url":"https://api.github.com/users/aph42","html_url":"https://github.com/aph42","followers_url":"https://api.github.com/users/aph42/followers","following_url":"https://api.github.com/users/aph42/following{/other_user}","gists_url":"https://api.github.com/users/aph42/gists{/gist_id}","starred_url":"https://api.github.com/users/aph42/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aph42/subscriptions","organizations_url":"https://api.github.com/users/aph42/orgs","repos_url":"https://api.github.com/users/aph42/repos","events_url":"https://api.github.com/users/aph42/events{/privacy}","received_events_url":"https://api.github.com/users/aph42/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/pygeode/pygeode/milestones/1","html_url":"https://github.com/pygeode/pygeode/milestones/release-1.1","labels_url":"https://api.github.com/repos/pygeode/pygeode/milestones/1/labels","id":1377660,"number":1,"title":"release-1.1","description":"Features that should be completed before the final release of a version 1.1.","creator":{"login":"neishm","id":1554921,"avatar_url":"https://avatars.githubusercontent.com/u/1554921?v=3","gravatar_id":"","url":"https://api.github.com/users/neishm","html_url":"https://github.com/neishm","followers_url":"https://api.github.com/users/neishm/followers","following_url":"https://api.github.com/users/neishm/following{/other_user}","gists_url":"https://api.github.com/users/neishm/gists{/gist_id}","starred_url":"https://api.github.com/users/neishm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/neishm/subscriptions","organizations_url":"https://api.github.com/users/neishm/orgs","repos_url":"https://api.github.com/users/neishm/repos","events_url":"https://api.github.com/users/neishm/events{/privacy}","received_events_url":"https://api.github.com/users/neishm/received_events","type":"User","site_admin":false},"open_issues":12,"closed_issues":0,"state":"open","created_at":"2015-10-26T17:04:39Z","updated_at":"2015-11-05T21:04:29Z","due_on":null,"closed_at":null},"comments":1,"created_at":"2015-11-05T21:04:29Z","updated_at":"2015-11-05T21:44:54Z","closed_at":null,"body":"There are a handful of 'orphaned' or otherwise less than obvious bits of functionality that seem more awkward to use (and to document) than they need to be. I'd like to suggest the following:\r\n\r\n- add 'concatenate' to top-level namespace \r\nThis could be done as a synonym for concat.concat(); one issue is how to deal with the dataset version of this; I'm not sure.\r\n\r\n- 'asdataset' to top-level namespace\r\nTo me this seems useful enough to have in the top-level namespace\r\n\r\n- hook 'composite', 'flatten', 'fft_smooth', and 'lag' as member functions of Var\r\nAll but the last of these take a var as a first argument and are natural member functions. There isn't a lag helper function for the Lag class in timeutils.py, but there should be, and it would also be a natural addition.\r\n\r\n- rename 'ensemble' module to 'ens', add 'ensemble', 'make_ensemble', to top level?\r\nI don't use these much to be honest, though it does seem like a potentially useful feature. However \r\nright now ensemble is the name of the module so something would have to be renamed to retain access to the module.\r\n\r\nAll but the last retain backward compatibility; I'm happy to leave the latter as some kind of future change, though i think it could be cleaner than it is, so it would be good to keep this in mind.\r\n\r\nI've added this to 1.1 as it's a 10 minute job and mostly housekeeping. "},"comment":{"url":"https://api.github.com/repos/pygeode/pygeode/issues/comments/154202683","html_url":"https://github.com/pygeode/pygeode/issues/83#issuecomment-154202683","issue_url":"https://api.github.com/repos/pygeode/pygeode/issues/83","id":154202683,"user":{"login":"neishm","id":1554921,"avatar_url":"https://avatars.githubusercontent.com/u/1554921?v=3","gravatar_id":"","url":"https://api.github.com/users/neishm","html_url":"https://github.com/neishm","followers_url":"https://api.github.com/users/neishm/followers","following_url":"https://api.github.com/users/neishm/following{/other_user}","gists_url":"https://api.github.com/users/neishm/gists{/gist_id}","starred_url":"https://api.github.com/users/neishm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/neishm/subscriptions","organizations_url":"https://api.github.com/users/neishm/orgs","repos_url":"https://api.github.com/users/neishm/repos","events_url":"https://api.github.com/users/neishm/events{/privacy}","received_events_url":"https://api.github.com/users/neishm/received_events","type":"User","site_admin":false},"created_at":"2015-11-05T21:44:54Z","updated_at":"2015-11-05T21:44:54Z","body":"I'm ok with all but the last one, for the same reason you state (backward compatibility). I've created another milestone (release-2.0) which could include renaming 'ensemble' to 'ens'. It's probably nitpicking, but I'd rather save all these minor compatibility breaks for one big update in the future."}}