{"action":"created","issue":{"url":"https://api.github.com/repos/fchollet/keras/issues/635","labels_url":"https://api.github.com/repos/fchollet/keras/issues/635/labels{/name}","comments_url":"https://api.github.com/repos/fchollet/keras/issues/635/comments","events_url":"https://api.github.com/repos/fchollet/keras/issues/635/events","html_url":"https://github.com/fchollet/keras/issues/635","id":104855490,"number":635,"title":"Layer params are nameless","user":{"login":"rodrigob","id":65367,"avatar_url":"https://avatars.githubusercontent.com/u/65367?v=3","gravatar_id":"","url":"https://api.github.com/users/rodrigob","html_url":"https://github.com/rodrigob","followers_url":"https://api.github.com/users/rodrigob/followers","following_url":"https://api.github.com/users/rodrigob/following{/other_user}","gists_url":"https://api.github.com/users/rodrigob/gists{/gist_id}","starred_url":"https://api.github.com/users/rodrigob/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rodrigob/subscriptions","organizations_url":"https://api.github.com/users/rodrigob/orgs","repos_url":"https://api.github.com/users/rodrigob/repos","events_url":"https://api.github.com/users/rodrigob/events{/privacy}","received_events_url":"https://api.github.com/users/rodrigob/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-09-04T09:16:48Z","updated_at":"2015-09-06T10:58:44Z","closed_at":null,"body":"Current values for `model.param[i].name` is None. When inspecting the created model it is hard to figure out what is what.\r\n\r\nFor now I am doing\r\n```\r\nself.params = [self.W, self.b]\r\nself.W.name = \"Dense W\"\r\nself.b.name = \"Dense b\"\r\n```\r\n\r\nbut I guess this should be done better.\r\n\r\nI saw Layer.set_name but:\r\na) It does not keep the naming W and b\r\nb) Would break the usual `model.add(Convolution2D(32, 32, 3, 3))`\r\nc) Is not called by default\r\n\r\nWhat about being able to do\r\n`model.add(Convolution2D(32, 32, 3, 3, name=\"conv5\"))`\r\n\r\nso that parameters names end up being `conv5.W` and `conv5.b`; and if name is None (default) then have them named (by default) as `Convolution2D.W` and `Convolution2D.b` ?\r\n"},"comment":{"url":"https://api.github.com/repos/fchollet/keras/issues/comments/138069768","html_url":"https://github.com/fchollet/keras/issues/635#issuecomment-138069768","issue_url":"https://api.github.com/repos/fchollet/keras/issues/635","id":138069768,"user":{"login":"wuaalb","id":8669162,"avatar_url":"https://avatars.githubusercontent.com/u/8669162?v=3","gravatar_id":"","url":"https://api.github.com/users/wuaalb","html_url":"https://github.com/wuaalb","followers_url":"https://api.github.com/users/wuaalb/followers","following_url":"https://api.github.com/users/wuaalb/following{/other_user}","gists_url":"https://api.github.com/users/wuaalb/gists{/gist_id}","starred_url":"https://api.github.com/users/wuaalb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wuaalb/subscriptions","organizations_url":"https://api.github.com/users/wuaalb/orgs","repos_url":"https://api.github.com/users/wuaalb/repos","events_url":"https://api.github.com/users/wuaalb/events{/privacy}","received_events_url":"https://api.github.com/users/wuaalb/received_events","type":"User","site_admin":false},"created_at":"2015-09-06T10:58:44Z","updated_at":"2015-09-06T10:58:44Z","body":"Did you check `keras.layers.containers.Graph` rather than `keras.models.Graph`?\r\n\r\nMaybe for `Sequential` it would be nice if the default name could also includes the index of the parameter's `Layer` in the stack..? E.g.\r\n```python\r\nname = 'Layer%d_%s' % (self.layers.index(layer), type(layer).__name__)\r\n```\r\n\r\nI guess in that case `name` might be made a property of `Layer` rather than just being assigned to part of the parameter names.."}}