autonohm / ohm_tsd_slam

SLAM approach based on truncated signed distance transform
21 stars 25 forks source link

changes on mutex lock in localization #9

Closed kuehnma closed 9 years ago

kuehnma commented 9 years ago

ThreadLocalize.cpp:166:(alter Commit) Allein der Check, ob ein Scan vorhanden ist, muss nicht gelockt werden oder? Entweder ein Scan existiert bereits oder er wird nicht geschrieben, weil der Mutex gelockt wurde.

ThreadLocalize.cpp:168:(neuer Commit) Welchen Zweck erfüllt ros::sleep hier? Wir könnten doch so schnell wie nur möglich? Oder gibt es hier Probleme?

Reduce Resolution: Kannst du das mal für einen Hokuyo Datensatz testen? Mit den 1080 Punkten sollte der SLAM nicht so schnell sein, wie für den Intel Datensatz. Dann solten wir es natürlich drin lassen.