{"action":"created","issue":{"url":"https://api.github.com/repos/fchollet/keras/issues/2152","repository_url":"https://api.github.com/repos/fchollet/keras","labels_url":"https://api.github.com/repos/fchollet/keras/issues/2152/labels{/name}","comments_url":"https://api.github.com/repos/fchollet/keras/issues/2152/comments","events_url":"https://api.github.com/repos/fchollet/keras/issues/2152/events","html_url":"https://github.com/fchollet/keras/pull/2152","id":144989462,"number":2152,"title":"Keras 1.0 preview","user":{"login":"fchollet","id":710255,"avatar_url":"https://avatars.githubusercontent.com/u/710255?v=3","gravatar_id":"","url":"https://api.github.com/users/fchollet","html_url":"https://github.com/fchollet","followers_url":"https://api.github.com/users/fchollet/followers","following_url":"https://api.github.com/users/fchollet/following{/other_user}","gists_url":"https://api.github.com/users/fchollet/gists{/gist_id}","starred_url":"https://api.github.com/users/fchollet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fchollet/subscriptions","organizations_url":"https://api.github.com/users/fchollet/orgs","repos_url":"https://api.github.com/users/fchollet/repos","events_url":"https://api.github.com/users/fchollet/events{/privacy}","received_events_url":"https://api.github.com/users/fchollet/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":18,"created_at":"2016-03-31T19:03:56Z","updated_at":"2016-04-02T17:36:54Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/fchollet/keras/pulls/2152","html_url":"https://github.com/fchollet/keras/pull/2152","diff_url":"https://github.com/fchollet/keras/pull/2152.diff","patch_url":"https://github.com/fchollet/keras/pull/2152.patch"},"body":"This is a near-total rewriting of the Keras codebase, still undergoing but already 95% complete. It will not be merged into master, rather it will replace master, which means that the merged PRs from the past two weeks will need to be backported.\r\n\r\nSome notes:\r\n- this introduces the functional API and deprecates the `Graph` model. See `docs/templates/complete_guide_to_the_keras_functional_api.md` for an (as of yet incomplete) introduction. You can also check out the unit tests `tests/keras/engine/test_topology.py`.\r\n- this introduces modular metrics, including the ability to monitor arbitrary lists of metrics on arbitrary outputs of a multi-output model.\r\n- this introduces support for multi-input and multi-output layers.\r\n- this improves RNNs by providing 4 different implementations to choose from in Theano and 2 different implementations in TensorFlow, each with a different memory/cpu profile. This will allow users to have fast RNNs across very different setups and tasks.\r\n- this improves `Lambda`.\r\n- and much more.\r\n\r\nWhat works at the current time:\r\n- everything, modulo a few things that were overlooked and a few potentially remaining bugs (only one is currently known, fix pending). All examples are running, and the following tests are passing: `test_topology` (functional API test), `test_sequential`, `test_graph`, all integration tests...\r\n\r\nTODO:\r\n- backport recent modifications from master into `keras-1`\r\n- rewrite unit tests. In the process of getting them to pass, some bugs will have to be fixed (the set of tests that are passing is already very extensive, so the bugs should be few and minor).\r\n- rewrite documentation.\r\n\r\nGet involved:\r\nYou can take a look at unit tests.\r\nAll further PRs to Keras should be made to the present branch, not to master.\r\n"},"comment":{"url":"https://api.github.com/repos/fchollet/keras/issues/comments/204763694","html_url":"https://github.com/fchollet/keras/pull/2152#issuecomment-204763694","issue_url":"https://api.github.com/repos/fchollet/keras/issues/2152","id":204763694,"user":{"login":"fchollet","id":710255,"avatar_url":"https://avatars.githubusercontent.com/u/710255?v=3","gravatar_id":"","url":"https://api.github.com/users/fchollet","html_url":"https://github.com/fchollet","followers_url":"https://api.github.com/users/fchollet/followers","following_url":"https://api.github.com/users/fchollet/following{/other_user}","gists_url":"https://api.github.com/users/fchollet/gists{/gist_id}","starred_url":"https://api.github.com/users/fchollet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fchollet/subscriptions","organizations_url":"https://api.github.com/users/fchollet/orgs","repos_url":"https://api.github.com/users/fchollet/repos","events_url":"https://api.github.com/users/fchollet/events{/privacy}","received_events_url":"https://api.github.com/users/fchollet/received_events","type":"User","site_admin":false},"created_at":"2016-04-02T17:36:54Z","updated_at":"2016-04-02T17:36:54Z","body":"I have found the root cause, this commit: https://github.com/fchollet/keras/pull/2152/commits/91a819fb343d27e05a5cdf44ac84b43892bdaa56\r\n\r\nSo now I am confident that deduping weights before updates computation is a good fix."}}