{"action":"created","issue":{"url":"https://api.github.com/repos/davidwhogg/AnniesLasso/issues/1","labels_url":"https://api.github.com/repos/davidwhogg/AnniesLasso/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/davidwhogg/AnniesLasso/issues/1/comments","events_url":"https://api.github.com/repos/davidwhogg/AnniesLasso/issues/1/events","html_url":"https://github.com/davidwhogg/AnniesLasso/pull/1","id":118193796,"number":1,"title":"Initial code base","user":{"login":"andycasey","id":504436,"avatar_url":"https://avatars.githubusercontent.com/u/504436?v=3","gravatar_id":"","url":"https://api.github.com/users/andycasey","html_url":"https://github.com/andycasey","followers_url":"https://api.github.com/users/andycasey/followers","following_url":"https://api.github.com/users/andycasey/following{/other_user}","gists_url":"https://api.github.com/users/andycasey/gists{/gist_id}","starred_url":"https://api.github.com/users/andycasey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andycasey/subscriptions","organizations_url":"https://api.github.com/users/andycasey/orgs","repos_url":"https://api.github.com/users/andycasey/repos","events_url":"https://api.github.com/users/andycasey/events{/privacy}","received_events_url":"https://api.github.com/users/andycasey/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-11-21T12:35:53Z","updated_at":"2015-11-23T09:35:45Z","closed_at":"2015-11-22T19:33:30Z","pull_request":{"url":"https://api.github.com/repos/davidwhogg/AnniesLasso/pulls/1","html_url":"https://github.com/davidwhogg/AnniesLasso/pull/1","diff_url":"https://github.com/davidwhogg/AnniesLasso/pull/1.diff","patch_url":"https://github.com/davidwhogg/AnniesLasso/pull/1.patch"},"body":"Yo @davidwhogg and @mkness! I wrote some code.\r\n\r\nI had to fork the repository so that I could set up Travis, Coveralls and Scrutinizer. The code includes an abstracted model and has appropriate getters/setters, however we will need to clarify some terms and nomenclature. The models (including sub-classes) are serialisable, and \"expensive\" components are optionally parallelised by specifying `threads` or a multiprocessing pool when the class is instantiated (e.g., the same parallelism behaviour in `emcee`). The code has 100% test coverage on Python 2.7 and 3.4. \r\n\r\nIt's ready for more complex models to be sub-classed (e.g., see [lasso.py](https://github.com/andycasey/AnniesLasso/blob/master/AnniesLasso/lasso.py)), where the methods required for each sub-classed model are `train`, `fit`, and `predict`, and necessary private attributes are `_data_attributes`, `_descriptive_attributes` and `_trained_attributes`. If they're set, all I/O and cross-validation functionality will run from the base model.\r\n\r\nI still need to set the documentation pages up, specifically with some getting started/tutorial pages, but functions and classes have informative docstrings.\r\n\r\nI wouldn't merge this until I have had a chance to set up documentation (and auto-doc building), but I'm just opening this PR to give you a heads up on what I did these last few days. Next week I'll set up the documentation and look to see what exists in Anna's version that's not implemented here."},"comment":{"url":"https://api.github.com/repos/davidwhogg/AnniesLasso/issues/comments/158887766","html_url":"https://github.com/davidwhogg/AnniesLasso/pull/1#issuecomment-158887766","issue_url":"https://api.github.com/repos/davidwhogg/AnniesLasso/issues/1","id":158887766,"user":{"login":"andycasey","id":504436,"avatar_url":"https://avatars.githubusercontent.com/u/504436?v=3","gravatar_id":"","url":"https://api.github.com/users/andycasey","html_url":"https://github.com/andycasey","followers_url":"https://api.github.com/users/andycasey/followers","following_url":"https://api.github.com/users/andycasey/following{/other_user}","gists_url":"https://api.github.com/users/andycasey/gists{/gist_id}","starred_url":"https://api.github.com/users/andycasey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andycasey/subscriptions","organizations_url":"https://api.github.com/users/andycasey/orgs","repos_url":"https://api.github.com/users/andycasey/repos","events_url":"https://api.github.com/users/andycasey/events{/privacy}","received_events_url":"https://api.github.com/users/andycasey/received_events","type":"User","site_admin":false},"created_at":"2015-11-23T09:35:45Z","updated_at":"2015-11-23T09:35:45Z","body":"Yep, I made sure I rebased to include your document changes before opening the PR. `andycasey/AnniesLasso` is set as the parent now, and I've added you both as collaborators to that repo."}}