{"action":"created","issue":{"url":"https://api.github.com/repos/chainer/chainer/issues/3473","repository_url":"https://api.github.com/repos/chainer/chainer","labels_url":"https://api.github.com/repos/chainer/chainer/issues/3473/labels{/name}","comments_url":"https://api.github.com/repos/chainer/chainer/issues/3473/comments","events_url":"https://api.github.com/repos/chainer/chainer/issues/3473/events","html_url":"https://github.com/chainer/chainer/issues/3473","id":261495354,"number":3473,"title":"Remove @property  from model.vision.vgg or show a official way to modify pretrained vgg","user":{"login":"apple2373","id":8258531,"avatar_url":"https://avatars0.githubusercontent.com/u/8258531?v=4","gravatar_id":"","url":"https://api.github.com/users/apple2373","html_url":"https://github.com/apple2373","followers_url":"https://api.github.com/users/apple2373/followers","following_url":"https://api.github.com/users/apple2373/following{/other_user}","gists_url":"https://api.github.com/users/apple2373/gists{/gist_id}","starred_url":"https://api.github.com/users/apple2373/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apple2373/subscriptions","organizations_url":"https://api.github.com/users/apple2373/orgs","repos_url":"https://api.github.com/users/apple2373/repos","events_url":"https://api.github.com/users/apple2373/events{/privacy}","received_events_url":"https://api.github.com/users/apple2373/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-09-28T23:59:22Z","updated_at":"2017-09-29T22:47:39Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"A problem happens when we try to delete certain layers (e.g. fully connected layers) from chainer implemented vgg, using pretrained weights. \r\n\r\nIn v2.0.0, I was able to do simply\r\n```\r\nvgg = VGG16Layers(pretrained_model=pretrained_model)\r\ndel vgg.fc6\r\ndel vgg.fc7\r\ndel vgg.fc8\r\n```\r\nHowever because some of the vgg is declared as `@property` after v2.0.1, this no longer works.  Jiren explains the problem here: https://chainer.slack.com/archives/C0LC5A6C9/p1501822862324456\r\n\r\nThis is from a slack conversation before. I revised this problem and found it's still there. No chainer developer cares it. So I just raise an issue here. \r\n\r\nThe whole discussion is posted \r\nfrom https://chainer.slack.com/archives/C0LC5A6C9/p1501815339058362\r\nto https://chainer.slack.com/archives/C0LC5A6C9/p1501828268794828\r\n\r\n As far as I know, the current solution is to downgrade to v2.0.0.  Because removing some layers are very common operation, chainer should provide a way to remove layers. I don't know why chaienr's developer added `@ property `.... If there is no significant reason, just delete it! If there is, they should provide official instructions to modify the vgg model. A solution I can think is, to provide an official function to remove, which works like this:\r\n```\r\nvgg = VGG16Layers(pretrained_model=pretrained_model)\r\nvgg.delete(\"fc6\")\r\nvgg.delete(\"fc7\")\r\nvgg.delete(\"fc8\")\r\n```"},"comment":{"url":"https://api.github.com/repos/chainer/chainer/issues/comments/333256481","html_url":"https://github.com/chainer/chainer/issues/3473#issuecomment-333256481","issue_url":"https://api.github.com/repos/chainer/chainer/issues/3473","id":333256481,"user":{"login":"yuyu2172","id":2062128,"avatar_url":"https://avatars3.githubusercontent.com/u/2062128?v=4","gravatar_id":"","url":"https://api.github.com/users/yuyu2172","html_url":"https://github.com/yuyu2172","followers_url":"https://api.github.com/users/yuyu2172/followers","following_url":"https://api.github.com/users/yuyu2172/following{/other_user}","gists_url":"https://api.github.com/users/yuyu2172/gists{/gist_id}","starred_url":"https://api.github.com/users/yuyu2172/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yuyu2172/subscriptions","organizations_url":"https://api.github.com/users/yuyu2172/orgs","repos_url":"https://api.github.com/users/yuyu2172/repos","events_url":"https://api.github.com/users/yuyu2172/events{/privacy}","received_events_url":"https://api.github.com/users/yuyu2172/received_events","type":"User","site_admin":false},"created_at":"2017-09-29T22:47:39Z","updated_at":"2017-09-29T22:47:39Z","author_association":"CONTRIBUTOR","body":"Also, you can check the reason why `@property` was added from here.\r\nhttps://github.com/chainer/chainer/pull/2810\r\n"}}
{"actor":{"display_login":"yuyu2172"}}