swri-robotics / marti_common

Common utility functions for MARTI
BSD 3-Clause "New" or "Revised" License
54 stars 63 forks source link

Allowing wrapping across longitude 180/-180 line. (Issue #644) #645

Closed tothgl closed 2 years ago

tothgl commented 2 years ago

I think this fixes the wrapping issue. Let me know if there is anything further I should do. Thanks. Gabe

tothgl commented 2 years ago

Some of the automated tests failed. I saw something similar when I tried to build the master branch under noetic. I ended up having to manually clone the swri-robotics/marti_messages repo. (I previously tried running rosdep, but it gave me an error with marti_introspection_msgs.) I will investigate...

kriskozak commented 2 years ago

This fix looks good to me.