CMU-TBD / SocNavBench

A Grounded Simulation Testing Framework for Evaluating Social Navigation: https://arxiv.org/abs/2103.00047
MIT License
35 stars 7 forks source link

Improved multi-robot rendering capabilities #13

Closed GustavoSilvera closed 2 years ago

GustavoSilvera commented 3 years ago
GustavoSilvera commented 3 years ago

This pr does not include any joysticks besides what was already included (sampling & random planner). So yes, this will raise a NotImplementedError for other joysticks unless they get installed.

GustavoSilvera commented 3 years ago

Putting this PR on hold for a while. It improves some non-critical functionality for users and we want to run more tests before merging.

GustavoSilvera commented 2 years ago

Bumping this PR. It will probably help with a lot of the installation issues from earlier.

@ajdroid wanna run the tests and make sure it looks good

ajdroid commented 2 years ago

Needed to change the opencv install version to 4.2.* in multi-robot-rendering/environment.yml

Had an issue with the pyopenGL similar to: https://github.com/MPI-IS/mesh/issues/23

File "/usr/lib/python3/dist-packages/OpenGL/raw/GLES2/_types.py", line 5, in _error_checker = _ErrorChecker( _p, _p.GLES2.glGetError )

Was able to bypass by setting _error_checker=None in that file. Can you reproduce @GustavoSilvera ?

After this workaround

image

GustavoSilvera commented 2 years ago

I can confirm that upgrading the opencv to 4.2.* still maintains fully-functional unit tests. But I did not need to perform the _error_checker=None workaround.

I also recently added some changes to address #17 and #18 by simplifying the installation process