MISTLab / Swarm-SLAM

Sparse Decentralized Collaborative Simultaneous Localization and Mapping Framework for Multi-Robot Systems
https://lajoiepy.github.io/cslam_documentation/html/index.html
MIT License
391 stars 40 forks source link

Robot IDs must start at 0 #34

Closed mjforan closed 5 months ago

mjforan commented 5 months ago

In neighbors_manager.py check_neighbors_in_range loops through range(self.max_nb_robots) and checks which neighbors are in range. This breaks if the robot ID is greater than max_nb_robots. You should add a check or at least a note in the documentation saying that robot IDs must start at 0 and not be greater than max_nb_robots.

lajoiepy commented 5 months ago

Thanks for pointing this out! I added it to the documentation.