{"action":"closed","issue":{"url":"https://api.github.com/repos/claritylab/lucida/issues/113","repository_url":"https://api.github.com/repos/claritylab/lucida","labels_url":"https://api.github.com/repos/claritylab/lucida/issues/113/labels{/name}","comments_url":"https://api.github.com/repos/claritylab/lucida/issues/113/comments","events_url":"https://api.github.com/repos/claritylab/lucida/issues/113/events","html_url":"https://github.com/claritylab/lucida/issues/113","id":149226348,"number":113,"title":"[dev] High Level Issues/Comments","user":{"login":"jhauswald","id":2567493,"avatar_url":"https://avatars.githubusercontent.com/u/2567493?v=3","gravatar_id":"","url":"https://api.github.com/users/jhauswald","html_url":"https://github.com/jhauswald","followers_url":"https://api.github.com/users/jhauswald/followers","following_url":"https://api.github.com/users/jhauswald/following{/other_user}","gists_url":"https://api.github.com/users/jhauswald/gists{/gist_id}","starred_url":"https://api.github.com/users/jhauswald/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jhauswald/subscriptions","organizations_url":"https://api.github.com/users/jhauswald/orgs","repos_url":"https://api.github.com/users/jhauswald/repos","events_url":"https://api.github.com/users/jhauswald/events{/privacy}","received_events_url":"https://api.github.com/users/jhauswald/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-04-18T18:01:27Z","updated_at":"2016-07-09T18:46:19Z","closed_at":"2016-07-09T18:46:19Z","body":"As of a37daa626a4dc78f7dc9ea8a745aae17a8ae1a15\r\n**Issues**:\r\n- All new services should be integrated into the build system and should pass continuous integration\r\n- OpenEphyra doesn't build, needs a Makefile.\r\n- `thrift-gen.sh` breaks because OpenEphyra was renamed\r\n- calendar service readme looks somewhat confusing\r\n- OpenEphyra has some autogenerated thrift source committed, should be removed. Only generate at runtime.\r\n\r\n**Questions**:\r\n- IMM seems incomplete (confirm?) \r\n- how is Kaldi supposed to be built? Source seems to be removed.\r\n- Need documentation to teach how to hook into build system and how each service should use/generate thrift files. Ideally all services use the same thrift generated files. Generate their own depending on the language its written in.\r\n- build system looks completely broken. [needs revert]\r\n- learn service should be removed\r\n- `.thrift` files have been copied everywhere, these should be generated once either for all the services or subclassed and then generated. See the build system.\r\n- each service should have a very simple python client showing the functionality\r\n- Register to command center can work like this:\r\n  - try to register, if it can't (catch exception from thrift) just open server and run stand-alone.\r\n\r\nWe need a status with for each service if its done. Done means:\r\n- Create, Learn, Infer (when applicable)\r\n- Integrated in build system and passes continuous integration\r\n- has a simple (python?) client for standalone testing\r\n- has register function for command center\r\n- sample data to integrate into new command center with query pipeline description.\r\n- READMEs done\r\n\r\n@haoyifan @yunshengb "}}
{"actor":{"display_login":"jhauswald"}}