personalrobotics / aikido

Artificial Intelligence for Kinematics, Dynamics, and Optimization

Home Page:https://personalrobotics.github.io/aikido/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Update CatkinResourceRetriever to use ROS Package Framework

egordon opened this issue · comments

Currently, CatkinResourceRetriever builds up the list of packages from scratch using environment variables (e.g. CMAKE_PREFIX_PATH).

This takes a while. But ROS already has a built-in way to resolve packages: https://wiki.ros.org/Packages

Ideally, we can modify CatkinResourceRetriever (or create a new RosResourceRetriever if we insist on keeping ros deps separate) to lazily check for package resolution when a resource is requested (and optionally cache the resolution so we minimize calls to rospack, though my guess is calls to rospack will be fast enough that this won't be necessary) rather than building it beforehand.