{"action":"created","issue":{"url":"https://api.github.com/repos/b-it-bots/mas_perception/issues/3","repository_url":"https://api.github.com/repos/b-it-bots/mas_perception","labels_url":"https://api.github.com/repos/b-it-bots/mas_perception/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/b-it-bots/mas_perception/issues/3/comments","events_url":"https://api.github.com/repos/b-it-bots/mas_perception/issues/3/events","html_url":"https://github.com/b-it-bots/mas_perception/pull/3","id":323993140,"node_id":"MDExOlB1bGxSZXF1ZXN0MTg4NzE0MTA5","number":3,"title":"refactor `perception_libs` from mas_domestic_robotics","user":{"login":"minhnh","id":1168534,"node_id":"MDQ6VXNlcjExNjg1MzQ=","avatar_url":"https://avatars3.githubusercontent.com/u/1168534?v=4","gravatar_id":"","url":"https://api.github.com/users/minhnh","html_url":"https://github.com/minhnh","followers_url":"https://api.github.com/users/minhnh/followers","following_url":"https://api.github.com/users/minhnh/following{/other_user}","gists_url":"https://api.github.com/users/minhnh/gists{/gist_id}","starred_url":"https://api.github.com/users/minhnh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/minhnh/subscriptions","organizations_url":"https://api.github.com/users/minhnh/orgs","repos_url":"https://api.github.com/users/minhnh/repos","events_url":"https://api.github.com/users/minhnh/events{/privacy}","received_events_url":"https://api.github.com/users/minhnh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":9,"created_at":"2018-05-17T12:03:05Z","updated_at":"2018-08-23T14:01:48Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/b-it-bots/mas_perception/pulls/3","html_url":"https://github.com/b-it-bots/mas_perception/pull/3","diff_url":"https://github.com/b-it-bots/mas_perception/pull/3.diff","patch_url":"https://github.com/b-it-bots/mas_perception/pull/3.patch"},"body":"---\r\nname: Refactoring\r\nabout: Refactoring packages\r\n---\r\n\r\n# Description\r\n- rename to mas_perception_libs\r\n- replace bounding_box usage in mcr_scene_segmentation\r\n\r\n## What is being refactored?\r\n- [x] Code\r\n- [x] Package organization\r\n\r\n# Motivation\r\n- expose functionalities unrelated to `scene_segmentation` for use in external packages\r\n- add a common location for these functionalities so they can be used by other packages\r\n- add abstraction to common perception steps (i.e. detection, image classification) so new approaches can be more easily integrated. (i.e. when another object detection method is available, simply write an extension of the service proxy, and the rest of the code won't be affected)\r\n\r\n# Organization\r\n## Current\r\n\r\n```\r\n.\r\n└── mcr_scene_segmentation\r\n    └── common\r\n        ├── include\r\n        │   └── bounding_box.h\r\n        └── src\r\n            └── bounding_box.cpp\r\n```\r\n## Proposed\r\n```\r\n.\r\n└── mas_perception_libs\r\n    └── common\r\n        ├── include\r\n        │   └── bounding_box.h\r\n        └── src\r\n            └── bounding_box.cpp\r\n```\r\n\r\n# Checklist\r\n- [ ] I have updated the CHANGELOG.rst file"},"comment":{"url":"https://api.github.com/repos/b-it-bots/mas_perception/issues/comments/415424735","html_url":"https://github.com/b-it-bots/mas_perception/pull/3#issuecomment-415424735","issue_url":"https://api.github.com/repos/b-it-bots/mas_perception/issues/3","id":415424735,"node_id":"MDEyOklzc3VlQ29tbWVudDQxNTQyNDczNQ==","user":{"login":"minhnh","id":1168534,"node_id":"MDQ6VXNlcjExNjg1MzQ=","avatar_url":"https://avatars3.githubusercontent.com/u/1168534?v=4","gravatar_id":"","url":"https://api.github.com/users/minhnh","html_url":"https://github.com/minhnh","followers_url":"https://api.github.com/users/minhnh/followers","following_url":"https://api.github.com/users/minhnh/following{/other_user}","gists_url":"https://api.github.com/users/minhnh/gists{/gist_id}","starred_url":"https://api.github.com/users/minhnh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/minhnh/subscriptions","organizations_url":"https://api.github.com/users/minhnh/orgs","repos_url":"https://api.github.com/users/minhnh/repos","events_url":"https://api.github.com/users/minhnh/events{/privacy}","received_events_url":"https://api.github.com/users/minhnh/received_events","type":"User","site_admin":false},"created_at":"2018-08-23T14:01:48Z","updated_at":"2018-08-23T14:01:48Z","author_association":"NONE","body":"@alex-mitrevski @sthoduka @argenos \r\nI updated the branch to use actions now. Could someone take a look and review/merge?"}}
{"actor":{"display_login":"minhnh"}}