AndrejOrsula / drl_grasping

Deep Reinforcement Learning for Robotic Grasping from Octrees
https://arxiv.org/pdf/2208.00818
BSD 3-Clause "New" or "Revised" License
404 stars 54 forks source link

Pyoctree Attribute Error #103

Closed lucarei closed 1 year ago

lucarei commented 1 year ago

Hi Andrej. Thanks for your work and for a previous answer. I am trying to make a local installation (yet tried Docker, it works perfectly but i need much more personalization; for example i want to use the UR5 and a modified environment). I installed all the needed dependancies following the instructions found in Dockerfile (even colcon command seems to finish correctly).

However, when i run an example (like "ros2 run drl_grasping ex_random_agent.bash") RVIZ gives me the following warning "severe warning for a namespace collision" and Gazebo is not opened: i think that this warning is not a problem since it appears also when i use Docker.

The main error is related to pyoctree -> "Attribute Error: module 'pyoctree' has no attribute 'Points'.

image image

PS: are all the commands from the Dockerfile enough to install correctly or can you share with me other tips?

lucarei commented 1 year ago

Issue solved using “export PYTHONPATH=~/ws/src/OCNN/octree/build/python:${PYTHONPATH}

AndrejOrsula commented 1 year ago

Glad you were able to resolve it!

PS: are all the commands from the Dockerfile enough to install correctly or can you share with me other tips?

The Dockerfile "was" enough to install everything correctly. However, the versions of dependencies for some ROS packages and all Gazebo libraries are not fixed (these dependencies are installed as Debian packages that do not archive past releases, so it is not possible to fix them). Therefore, if these introduce any breaking changes (ideally, this should not occur) - the Dockerfile might not be valid anymore, and some issues might appear (e.g. compile-time errors, run-time errors or change in computational/learning performance). ROS 2 galactic is now EOL, so it won't be getting updates - which means that these dependencies should be fine. However, updates to fortress could possibly introduce changes that affect this repository.