{"action":"created","issue":{"url":"https://api.github.com/repos/fchollet/keras/issues/1179","labels_url":"https://api.github.com/repos/fchollet/keras/issues/1179/labels{/name}","comments_url":"https://api.github.com/repos/fchollet/keras/issues/1179/comments","events_url":"https://api.github.com/repos/fchollet/keras/issues/1179/events","html_url":"https://github.com/fchollet/keras/pull/1179","id":120556714,"number":1179,"title":"Sprint to bring up test coverage","user":{"login":"phreeza","id":123894,"avatar_url":"https://avatars.githubusercontent.com/u/123894?v=3","gravatar_id":"","url":"https://api.github.com/users/phreeza","html_url":"https://github.com/phreeza","followers_url":"https://api.github.com/users/phreeza/followers","following_url":"https://api.github.com/users/phreeza/following{/other_user}","gists_url":"https://api.github.com/users/phreeza/gists{/gist_id}","starred_url":"https://api.github.com/users/phreeza/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/phreeza/subscriptions","organizations_url":"https://api.github.com/users/phreeza/orgs","repos_url":"https://api.github.com/users/phreeza/repos","events_url":"https://api.github.com/users/phreeza/events{/privacy}","received_events_url":"https://api.github.com/users/phreeza/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-12-05T14:00:22Z","updated_at":"2015-12-05T14:58:29Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/fchollet/keras/pulls/1179","html_url":"https://github.com/fchollet/keras/pull/1179","diff_url":"https://github.com/fchollet/keras/pull/1179.diff","patch_url":"https://github.com/fchollet/keras/pull/1179.patch"},"body":"This is a work in progress. I will be working on this all weekend (I hope). My timezone is CET. If anyone wants to join me, let me know ( @olegsinyavskiy @EderSantana @farizrahman4u ?) and I will give you push access.\r\n\r\n# Todos\r\n## Meta \r\nThings that are not tests as such but that would be nice to have\r\n- [x] Get coveralls to work (now available for my fork on https://coveralls.io/github/phreeza/keras )\r\n- [ ] Tag front page with coveralls and travis tags\r\n- [ ] clean up tests directory with a structure that matches the main project\r\n\r\n## Tests\r\nTests that need to be written, in approximate order of coverage increase they will bring\r\n- [ ] preprocessing code\r\n- [ ] Siamese, Lambda and LambdaMerge layers (just saw @farizrahman4u is working on this in #1171)\r\n- [ ] Changed interface in Graph and Sequential containers\r\n- [ ] Reuters make_dataset\r\n- [ ] advanced activations\r\n- [ ] sklearn wrapper\r\n- [ ] I/O utils\r\n- [ ] overwrite protection in model saving\r\n\r\nI recon if all these are met, we should be pretty close to 90%, which seems like a more presentable number."},"comment":{"url":"https://api.github.com/repos/fchollet/keras/issues/comments/162206561","html_url":"https://github.com/fchollet/keras/pull/1179#issuecomment-162206561","issue_url":"https://api.github.com/repos/fchollet/keras/issues/1179","id":162206561,"user":{"login":"olegsinyavskiy","id":2086260,"avatar_url":"https://avatars.githubusercontent.com/u/2086260?v=3","gravatar_id":"","url":"https://api.github.com/users/olegsinyavskiy","html_url":"https://github.com/olegsinyavskiy","followers_url":"https://api.github.com/users/olegsinyavskiy/followers","following_url":"https://api.github.com/users/olegsinyavskiy/following{/other_user}","gists_url":"https://api.github.com/users/olegsinyavskiy/gists{/gist_id}","starred_url":"https://api.github.com/users/olegsinyavskiy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/olegsinyavskiy/subscriptions","organizations_url":"https://api.github.com/users/olegsinyavskiy/orgs","repos_url":"https://api.github.com/users/olegsinyavskiy/repos","events_url":"https://api.github.com/users/olegsinyavskiy/events{/privacy}","received_events_url":"https://api.github.com/users/olegsinyavskiy/received_events","type":"User","site_admin":false},"created_at":"2015-12-05T14:58:29Z","updated_at":"2015-12-05T14:58:29Z","body":"@phreeza, great plan! I'm interested in joining. I went ahead yesterday and started covering examples:\r\nhttps://github.com/fchollet/keras/pull/1177 I argue that its as important as covering the core stuff. Can you add it to your plan?\r\n\r\nAs for coverage, I just realized that it might be tricky to make it work with travis parallel test matrix.. I'm not sure how can you assemble coverage results..\r\n"}}