{"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":2,"created_at":"2015-12-05T14:00:22Z","updated_at":"2015-12-05T15:17:41Z","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- [ ] Test examples ( @olegsinyavskiy started work on this in #1177 )\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/162211781","html_url":"https://github.com/fchollet/keras/pull/1179#issuecomment-162211781","issue_url":"https://api.github.com/repos/fchollet/keras/issues/1179","id":162211781,"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},"created_at":"2015-12-05T15:17:41Z","updated_at":"2015-12-05T15:17:41Z","body":"Great, I have given you push access to my repo, so you can update the branch as you see fit. I am doing the advanced activations right now.\r\n\r\nI agree testing the examples is important. Once the automated documentation is here, (it's on the december roadmap) we should also test the code that shows up in the documentation.\r\n\r\nCoveralls actually does a pretty decent job of aggregating the coverage results from the parallel runs, have a look at https://coveralls.io/github/phreeza/keras , there is cumulative coverage reported as well as for the individual runs."}}