search

AGV-IIT-KGP / freezing-batman

AGV
GNU General Public License v3.0
14 stars 26 forks source link

Remove MRPT dependency from position_from_target #56

Closed hargup closed 10 years ago

hargup commented 10 years ago

We have a position_from_target package in environment/interpretation. It takes That package takes the current gps position of the bot, the gps position of the target and gives the target pose. The package is dependent on MRTP library for the conversion of GPS points to 3D points in space. I feel is it worth the effort to remove the MRPT dependency. See the code of the conversion in MRPT from here https://github.com/jlblancoc/mrpt/blob/master/libs/topography/src/conversions.cpp and port it to our package. The conversion code is pretty straight forward.

hargup commented 10 years ago

@krishna95 worked on this issue in this PR https://github.com/AGV-IIT-KGP/freezing-batman/pull/61. I have merged it but it has not yet been tested. Keep the issue open till the testing is over and it is verified that the new code will work.

dragonfly91 commented 10 years ago

There is a problem here. I've listed the nodes we need to use here: https://github.com/AGV-IIT-KGP/freezing-batman/wiki/Node-and-Topic-Nomenclature

So, we need to redesign the entire structure accordingly, for cleaner understanding. I'm creating a new issue for this in #66