{"action":"created","issue":{"url":"https://api.github.com/repos/MRPT/GSoC2017-discussions/issues/3","repository_url":"https://api.github.com/repos/MRPT/GSoC2017-discussions","labels_url":"https://api.github.com/repos/MRPT/GSoC2017-discussions/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/MRPT/GSoC2017-discussions/issues/3/comments","events_url":"https://api.github.com/repos/MRPT/GSoC2017-discussions/issues/3/events","html_url":"https://github.com/MRPT/GSoC2017-discussions/issues/3","id":226439057,"number":3,"title":"GUI app to benchmark image feature detectors and descriptors (Raghavender Sahdev)","user":{"login":"jlblancoc","id":5497818,"avatar_url":"https://avatars2.githubusercontent.com/u/5497818?v=4","gravatar_id":"","url":"https://api.github.com/users/jlblancoc","html_url":"https://github.com/jlblancoc","followers_url":"https://api.github.com/users/jlblancoc/followers","following_url":"https://api.github.com/users/jlblancoc/following{/other_user}","gists_url":"https://api.github.com/users/jlblancoc/gists{/gist_id}","starred_url":"https://api.github.com/users/jlblancoc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jlblancoc/subscriptions","organizations_url":"https://api.github.com/users/jlblancoc/orgs","repos_url":"https://api.github.com/users/jlblancoc/repos","events_url":"https://api.github.com/users/jlblancoc/events{/privacy}","received_events_url":"https://api.github.com/users/jlblancoc/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":95,"created_at":"2017-05-04T23:49:21Z","updated_at":"2017-08-22T13:59:06Z","closed_at":null,"body":"## Initial description: \r\n\r\nIn this proposal we aim to build a GUI application which has the ability to benchmark various combinations of detectors/descriptors.\r\n\r\nSee attached PDF:\r\n[6307731490209792_1491218516_Raghavender_GSoC2017_proposal.pdf](https://github.com/MRPT/GSoC2017-discussions/files/977835/6307731490209792_1491218516_Raghavender_GSoC2017_proposal.pdf)\r\n"},"comment":{"url":"https://api.github.com/repos/MRPT/GSoC2017-discussions/issues/comments/324035006","html_url":"https://github.com/MRPT/GSoC2017-discussions/issues/3#issuecomment-324035006","issue_url":"https://api.github.com/repos/MRPT/GSoC2017-discussions/issues/3","id":324035006,"user":{"login":"famoreno","id":6714580,"avatar_url":"https://avatars2.githubusercontent.com/u/6714580?v=4","gravatar_id":"","url":"https://api.github.com/users/famoreno","html_url":"https://github.com/famoreno","followers_url":"https://api.github.com/users/famoreno/followers","following_url":"https://api.github.com/users/famoreno/following{/other_user}","gists_url":"https://api.github.com/users/famoreno/gists{/gist_id}","starred_url":"https://api.github.com/users/famoreno/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/famoreno/subscriptions","organizations_url":"https://api.github.com/users/famoreno/orgs","repos_url":"https://api.github.com/users/famoreno/repos","events_url":"https://api.github.com/users/famoreno/events{/privacy}","received_events_url":"https://api.github.com/users/famoreno/received_events","type":"User","site_admin":false},"created_at":"2017-08-22T13:59:05Z","updated_at":"2017-08-22T13:59:05Z","body":"Hi @raghavendersahdev. \r\nGreat work with the rawlog support for the GUI app and with the ROS bridge. Congrats again!\r\nAnd thanks @jlblancoc for taking care of this thread these days ;-)!\r\n\r\nRegarding this:\r\n\r\n> mrpt::obs::CObservationRange <--> sensor_msgs/Range : the variable radiation_type is not available in CObservationRange, so I am not setting it too..the ROS range message has the range field taking single float number whereas the CObservationRange has a list of ranges as a TMeasurementList. that takes a list of ranges; there seems to be an incompatibility here. What should be done here?\r\n\r\nI'm not sure why CObservationRange includes a `deque` of measurements (maybe @jlblancoc can explain why `sensedData` is a `deque` instead of a single number) but I think that the ROS->MRPT conversion should be just copying the value to the position zero ([as you already do](https://github.com/raghavendersahdev/mrpt_navigation/blob/master/mrpt_bridge/src/range.cpp#L21)). BTW, I think that the best way to do this in here is by clearing the `sensedData` `deque` and then using `push_back()`. The way it is currently implemented would crash if `sensedData` is empty. \r\n\r\nFor the MRPT->ROS conversion, I also think that copying the value at position zero is OK instead of computing the mean value as you do [here](https://github.com/raghavendersahdev/mrpt_navigation/blob/master/mrpt_bridge/src/range.cpp#L40-L44). But, in any case, I think that the deque will most probably contain only one value so in the end it will not affect the result.\r\n\r\nAll the best\r\n\r\n\r\n"}}
{"actor":{"display_login":"famoreno"}}