{"action":"created","issue":{"url":"https://api.github.com/repos/JdeRobot/dl-digitclassifier/issues/4","repository_url":"https://api.github.com/repos/JdeRobot/dl-digitclassifier","labels_url":"https://api.github.com/repos/JdeRobot/dl-digitclassifier/issues/4/labels{/name}","comments_url":"https://api.github.com/repos/JdeRobot/dl-digitclassifier/issues/4/comments","events_url":"https://api.github.com/repos/JdeRobot/dl-digitclassifier/issues/4/events","html_url":"https://github.com/JdeRobot/dl-digitclassifier/issues/4","id":295328588,"number":4,"title":"[Urgent] Merge TensorFlow/Keras structures, YAML config.","user":{"login":"naxvm","id":23213360,"avatar_url":"https://avatars3.githubusercontent.com/u/23213360?v=4","gravatar_id":"","url":"https://api.github.com/users/naxvm","html_url":"https://github.com/naxvm","followers_url":"https://api.github.com/users/naxvm/followers","following_url":"https://api.github.com/users/naxvm/following{/other_user}","gists_url":"https://api.github.com/users/naxvm/gists{/gist_id}","starred_url":"https://api.github.com/users/naxvm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/naxvm/subscriptions","organizations_url":"https://api.github.com/users/naxvm/orgs","repos_url":"https://api.github.com/users/naxvm/repos","events_url":"https://api.github.com/users/naxvm/events{/privacy}","received_events_url":"https://api.github.com/users/naxvm/received_events","type":"User","site_admin":false},"labels":[{"id":766652860,"url":"https://api.github.com/repos/JdeRobot/dl-digitclassifier/labels/enhancement","name":"enhancement","color":"84b6eb","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-02-07T23:15:24Z","updated_at":"2018-02-08T10:24:16Z","closed_at":null,"author_association":"COLLABORATOR","body":"We need to merge both Keras/TensorFlow-powered versions of the component, as we have a lot of redundant code.\r\n\r\nThis should be made by providing a transparent API from the models to the classifier (for example, by providing an identical function <code>network.predict(img)</code>), so the component code itself has to worry about which backend is being used as less as possible.\r\n\r\nMaybe @dpascualhe or me could fork the repo, and we both push changes to that branch, for finally submitting a Pull Request to the master branch.\r\n\r\nLet's get on it!"},"comment":{"url":"https://api.github.com/repos/JdeRobot/dl-digitclassifier/issues/comments/364068015","html_url":"https://github.com/JdeRobot/dl-digitclassifier/issues/4#issuecomment-364068015","issue_url":"https://api.github.com/repos/JdeRobot/dl-digitclassifier/issues/4","id":364068015,"user":{"login":"naxvm","id":23213360,"avatar_url":"https://avatars3.githubusercontent.com/u/23213360?v=4","gravatar_id":"","url":"https://api.github.com/users/naxvm","html_url":"https://github.com/naxvm","followers_url":"https://api.github.com/users/naxvm/followers","following_url":"https://api.github.com/users/naxvm/following{/other_user}","gists_url":"https://api.github.com/users/naxvm/gists{/gist_id}","starred_url":"https://api.github.com/users/naxvm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/naxvm/subscriptions","organizations_url":"https://api.github.com/users/naxvm/orgs","repos_url":"https://api.github.com/users/naxvm/repos","events_url":"https://api.github.com/users/naxvm/events{/privacy}","received_events_url":"https://api.github.com/users/naxvm/received_events","type":"User","site_admin":false},"created_at":"2018-02-08T10:24:16Z","updated_at":"2018-02-08T10:24:16Z","author_association":"COLLABORATOR","body":"\r\nOn the other hand, as <code>comm</code> library will receive an update soon, the way to indicate it which framework is being used to serve the images (ICE/ROS) will change. Until now, the YAML node <code>Camera.Server</code> has had numerical values (0=ICE, 1=ROS...). But from this now update, it will be indicated via a string value (\"ice\", \"ros\", etc.). As this is not implemented yet, we shall not change the implementation until the <code>comm</code> library is updated.\r\n"}}
{"actor":{"display_login":"naxvm"}}