ros-perception / slam_gmapping

http://www.ros.org/wiki/slam_gmapping
668 stars 528 forks source link

Fix map mutex use (gmapping issue #11) #16

Closed skohlbr closed 10 years ago

skohlbr commented 10 years ago

See #11. Tested with logged data as in http://wiki.ros.org/slam_gmapping/Tutorials/MappingFromLoggedData. Should I create another pull request for merging into groovy_devel?

wjwwood commented 10 years ago

@skohlbr Thanks I totally lost track of this, it fell through the cracks.

wjwwood commented 10 years ago

I'm not convinced that these are entirely necessary at all, but for now we can fix the obvious miss use and maybe look at removing them altogether at some other time.

I can back-port this to groovy.

wjwwood commented 10 years ago

Groovy backport: ac003dc64c3802e7f1ca5650f789aab502b35b93