{"action":"created","issue":{"url":"https://api.github.com/repos/pfnet/chainercv/issues/160","repository_url":"https://api.github.com/repos/pfnet/chainercv","labels_url":"https://api.github.com/repos/pfnet/chainercv/issues/160/labels{/name}","comments_url":"https://api.github.com/repos/pfnet/chainercv/issues/160/comments","events_url":"https://api.github.com/repos/pfnet/chainercv/issues/160/events","html_url":"https://github.com/pfnet/chainercv/issues/160","id":227590596,"number":160,"title":"Directory structure of links","user":{"login":"yuyu2172","id":2062128,"avatar_url":"https://avatars0.githubusercontent.com/u/2062128?v=3","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},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2017-05-10T07:32:26Z","updated_at":"2017-05-10T10:16:39Z","closed_at":null,"body":"Since there are losses that can be reused between models, it is better to make a directory for losses.\r\nThis is from the discussion I had with @mitmul and @rezoo .\r\n\r\nI will post my idea of how `links` should be organized.\r\nI put functions (not links) necessary for Faster RCNN under `links.model.faster_rcnn.utils`. I would like a comment on this design.\r\n\r\n\r\n```\r\n.\r\n└── links\r\n    ├── loss\r\n    │   ├── faster_rcnn_loss.py\r\n    │   ├── semantic_segmentation_loss.py\r\n    │   └── ssd_loss.py\r\n    └── model\r\n        ├── faster_rcnn\r\n        │   ├── faster_rcnn.py\r\n        │   ├── faster_rcnn_vgg.py\r\n        │   └── utils\r\n        │       ├── anchor_target_creator.py\r\n        │       ├── bbox_regression_target.py\r\n        │       ├── generate_anchor.py\r\n        │       ├── proposal_creator.py\r\n        │       └── proposal_target_creator.py\r\n        ├── segnet\r\n        │   └── segnet.py\r\n        └── ssd\r\n            ├── ssd.py\r\n            └── ssd_vgg.py\r\n\r\n```"},"comment":{"url":"https://api.github.com/repos/pfnet/chainercv/issues/comments/300439163","html_url":"https://github.com/pfnet/chainercv/issues/160#issuecomment-300439163","issue_url":"https://api.github.com/repos/pfnet/chainercv/issues/160","id":300439163,"user":{"login":"yuyu2172","id":2062128,"avatar_url":"https://avatars0.githubusercontent.com/u/2062128?v=3","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-05-10T10:16:39Z","updated_at":"2017-05-10T10:16:39Z","body":"Yeah, `ssd_loss.py` and `faster_rcnn_loss.py` should be discussed.\r\nOn `faster_rcnn_loss.py`, it is very difficult to provide it in a general form because it assumes heavily the structure of Faster RCNN.\r\nI think that if losses assume too much about the underlying models, it would be better to put it under `model/MODEL_NAME`."}}
{"actor":{"display_login":"yuyu2172"}}