PlanSys2 / ros2_planning_system

This repo contains a PDDL-based planning system for ROS2.
Apache License 2.0
396 stars 87 forks source link

include "object" in getType #321

Closed Rezenders closed 2 months ago

Rezenders commented 2 months ago

Return the "object" type in the getType method, as instances could be defined with the object type since it is included by default.

Also, I don´t understand why the following is necessary: https://github.com/PlanSys2/ros2_planning_system/blob/a7971965431fc5c837074e6a15059650f75645b8/plansys2_pddl_parser/include/plansys2_pddl_parser/Domain.h#L221-L225

It basically creates duplicates of the constants for all its supertypes. Is this really needed?

Related to #317