Skip to content

Commit

Permalink
Add very important warning to cache class
Browse files Browse the repository at this point in the history
Signed-off-by: methylDragon <[email protected]>
  • Loading branch information
methylDragon committed Oct 17, 2023
1 parent 04ba305 commit 3ed60c6
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion nexus_motion_planner/src/motion_plan_cache.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,16 @@ namespace motion_planner {
* how long they took to execute. This allows for the lookup and reuse of the
* best performing plans found so far.
*
* WARNING: This cache does NOT yet support collision detection!
* WARNING:
* This cache does NOT support collision detection!
* Plans will be put into and fetched from the cache IGNORING collision.
* If your planning scene is expected to change between cache lookups, do NOT
* use this cache, fetched plans are likely to result in collision then.
*
* To handle collisions this class will need to hash the planning scene world
* msg (after zeroing out std_msgs/Header timestamps and sequences) and do an
* appropriate lookup.
*
* Relevant ROS Parameters:
* - `warehouse_plugin`: What database to use
* - `warehouse_host`: Where the database should be created
Expand Down

0 comments on commit 3ed60c6

Please sign in to comment.