{"comment":{"url":"https://api.github.com/repos/chonho/incubator-singa/comments/14094908","html_url":"https://github.com/chonho/incubator-singa/commit/fced1820a12e579e5719270c3c7ffd169c9d2960#commitcomment-14094908","id":14094908,"user":{"login":"nudles","id":3797447,"avatar_url":"https://avatars.githubusercontent.com/u/3797447?v=3","gravatar_id":"","url":"https://api.github.com/users/nudles","html_url":"https://github.com/nudles","followers_url":"https://api.github.com/users/nudles/followers","following_url":"https://api.github.com/users/nudles/following{/other_user}","gists_url":"https://api.github.com/users/nudles/gists{/gist_id}","starred_url":"https://api.github.com/users/nudles/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nudles/subscriptions","organizations_url":"https://api.github.com/users/nudles/orgs","repos_url":"https://api.github.com/users/nudles/repos","events_url":"https://api.github.com/users/nudles/events{/privacy}","received_events_url":"https://api.github.com/users/nudles/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"fced1820a12e579e5719270c3c7ffd169c9d2960","created_at":"2015-10-30T14:34:38Z","updated_at":"2015-10-30T14:34:38Z","body":"how about this design:\r\n1. creating a base builder/configure, which provide functions for adding individual layers, e.g., convolutional layer, pooling layer, etc. For layers with parameters, we follow the current design, i.e., SetParamProto(layer, ...).\r\n2. creating sub-builder/configure, which provide functions for adding layers for specific models, e.g., MLP, Auto-Encoder and CNN.\r\n\r\nmaybe we should try to use reference, as python does not have the pointer type."}}