vectornav icon indicating copy to clipboard operation
vectornav copied to clipboard

Odom != ECEF?? Why is an ECEF Position parsed into the Odom frame?

Open bjajoh opened this issue 2 years ago • 5 comments

Hi,

for some reason the ECEF Position is used for the odom message. However this doesn't make any sense in my opinion, because ECEF is completely different from the UTM Coordinate System which should be used. This is not conform with the ROS Odom message or in general anyone who would use it.

Is there any reason for it?

bjajoh avatar Jul 17 '21 22:07 bjajoh

Please feel free to send a PR; using ECEF directly doesn't really make sense for the Odom frame....

dawonn avatar Jul 17 '21 23:07 dawonn

Ok thank you, yes I'll think about a fix. Otherwise if people know why this is in the code, it would be helpful to know.

bjajoh avatar Jul 17 '21 23:07 bjajoh

ECEF is probably a fine choice for a data source, but generate a transform to an ENU frame with the origin at the starting location and translate the pos into that frame.

Thank you for the help! I don't regularly use this interface obviously. ;)

dawonn avatar Jul 17 '21 23:07 dawonn

Take a look at http://wiki.ros.org/enu. Since the VectorNav cannot provide an ENU/NED position this would be a good example of how to calculate an ENU position with the data the VectorNav provides. It may be worth implementing an LLA to ENU conversion in the odom topic of this driver. I may even make a PR out of this if wanted.

CollinNHays avatar Nov 17 '21 18:11 CollinNHays

PR's are very welcome. :)

On Wed, Nov 17, 2021, 13:59 CollinNHays @.***> wrote:

Take a look at http://wiki.ros.org/enu. Since the VectorNav cannot provide an ENU/NED position this would be a good example of how to calculate an ENU position with the data the VectorNav provides. It may be worth implementing an LLA to ENU conversion in the odom topic of this driver. I may even make a PR out of this if wanted.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/dawonn/vectornav/issues/73#issuecomment-971880736, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXHVF5JZHC3EMSMA6N23JLUMP3PRANCNFSM5ARPOULQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

dawonn avatar Nov 17 '21 19:11 dawonn