search

udacity / robotics-nanodegree-issues

Public waffleboard to track Robotics Nanodegree Issues
2 stars 0 forks source link

Lesson 13 Build MCL in C++, Robot Class #202

Closed douglasteeple closed 6 years ago

douglasteeple commented 6 years ago

double* sense() is a poor interface. Returning a pointer to a static array is just not great sw design. It is not thread safe, for example. Better to have sense() return a vector of values...

karimchamaa commented 6 years ago

Thanks for reporting this. Just updated it in GitHub to "vector sense()". Check it out: https://github.com/udacity/RoboND-MCL-Lab