Fix agentic asset defaults (#847)

## Summary
Fix agentic asset defaults

## Detailed description
- Switched agent embodiment preference from `ik` to `default`.
- Moved the DROID default tag to `droid_abs_joint_pos`.
- Removed redundant object-library initializers so instance names
propagate.
- Added prompt guidance for identifier-safe generated node ids.

---------

Signed-off-by: Qian Lin <qianl@nvidia.com>

Author: qianl-nv

isaaclab_arena/agentic_environment_generation/asset_matcher.py

@@ -66,7 +66,7 @@ def match_asset(
 
         required_tags: Tags every candidate must carry (e.g. ``["object"]``).
         preferred_tags: Additional tags that narrow the first-pass pool
-            (e.g. item ``category_tags`` or ``["ik"]`` for embodiments).
+            (e.g. item ``category_tags`` or ``["default"]`` for embodiments).
             When ``None`` or empty, stage 2 is skipped and matching falls
             through directly to the required-tag pool.
 

isaaclab_arena/agentic_environment_generation/environment_generation_agent.py

@@ -330,6 +330,8 @@ def _system_prompt(self) -> str:
 - params values are node ids or the background name, not registry asset names.
 - NODE IDS: an item's id is its instance_name if set, else its query. For multiple
   items of the same kind, give each a unique instance_name and use those exact ids everywhere.
+- Use underscore_connected identifiers for every query and instance_name (e.g.
+  'bbq_sauce_bottle', not 'bbq sauce bottle') so node ids are valid Python identifiers.
 - Every relation subject/reference and object task param must name one node id — never
   a bare query that maps to several instances. Each must name exactly one;
   if the prompt doesn't say which, pick any.

isaaclab_arena/agentic_environment_generation/environment_intent_spec.py

@@ -87,7 +87,7 @@ class EnvironmentIntentSpec(BaseModel):
         description=(
             "Robot embodiment to control. Use a bare family name ('franka', "
             "'droid', 'g1', 'gr1') when the prompt does not specify a "
-            "control mode — the resolver defaults each to its IK variant. "
+            "control mode — the resolver defaults each to its default-tagged variant. "
             "Use a full registered name (e.g. 'franka_joint_pos') only when "
             "the prompt explicitly requests joint control."
         ),

isaaclab_arena/agentic_environment_generation/intent_compiler.py

@@ -89,7 +89,7 @@ def compile(
             trace_prefix="embodiment",
             node_type=ArenaEnvGraphNodeType.EMBODIMENT,
             required_tags=["embodiment"],
-            preferred_tags=["ik"],
+            preferred_tags=["default"],
         )
         if embodiment_node is not None:
             nodes.append(embodiment_node)

isaaclab_arena/assets/object_library.py

@@ -79,15 +79,6 @@ class CrackerBox(LibraryObject):
     tags = ["object"]
     usd_path = f"{ISAAC_NUCLEUS_DIR}/Props/YCB/Axis_Aligned_Physics/003_cracker_box.usd"
 
-    def __init__(
-        self,
-        instance_name: str | None = None,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(instance_name=instance_name, prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class MustardBottle(LibraryObject):
@@ -99,15 +90,6 @@ class MustardBottle(LibraryObject):
     tags = ["object"]
     usd_path = f"{ISAAC_NUCLEUS_DIR}/Props/YCB/Axis_Aligned_Physics/006_mustard_bottle.usd"
 
-    def __init__(
-        self,
-        instance_name: str | None = None,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(instance_name=instance_name, prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class SugarBox(LibraryObject):
@@ -119,15 +101,6 @@ class SugarBox(LibraryObject):
     tags = ["object"]
     usd_path = f"{ISAAC_NUCLEUS_DIR}/Props/YCB/Axis_Aligned_Physics/004_sugar_box.usd"
 
-    def __init__(
-        self,
-        instance_name: str | None = None,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(instance_name=instance_name, prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class TomatoSoupCan(LibraryObject):
@@ -139,15 +112,6 @@ class TomatoSoupCan(LibraryObject):
     tags = ["object"]
     usd_path = f"{ISAAC_NUCLEUS_DIR}/Props/YCB/Axis_Aligned_Physics/005_tomato_soup_can.usd"
 
-    def __init__(
-        self,
-        instance_name: str | None = None,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(instance_name=instance_name, prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class PowerDrill(LibraryObject):
@@ -159,15 +123,6 @@ class PowerDrill(LibraryObject):
     tags = ["object"]
     usd_path = f"{ISAACLAB_NUCLEUS_DIR}/Arena/assets/object_library/power_drill_physics/power_drill_physics.usd"
 
-    def __init__(
-        self,
-        instance_name: str | None = None,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(instance_name=instance_name, prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class Microwave(LibraryObject, Openable):
@@ -266,15 +221,6 @@ class OfficeTable(LibraryObject):
     usd_path = f"{ISAACLAB_NUCLEUS_DIR}/Mimic/nut_pour_task/nut_pour_assets/table.usd"
     scale = (1.0, 1.0, 0.7)
 
-    def __init__(
-        self,
-        instance_name: str | None = None,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(instance_name=instance_name, prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class BlueSortingBin(LibraryObject):
@@ -287,15 +233,6 @@ class BlueSortingBin(LibraryObject):
     usd_path = f"{ISAACLAB_NUCLEUS_DIR}/Mimic/exhaust_pipe_task/exhaust_pipe_assets/blue_sorting_bin.usd"
     scale = (4.0, 2.0, 1.0)
 
-    def __init__(
-        self,
-        instance_name: str | None = None,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(instance_name=instance_name, prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class BlueExhaustPipe(LibraryObject):
@@ -308,15 +245,6 @@ class BlueExhaustPipe(LibraryObject):
     usd_path = f"{ISAACLAB_NUCLEUS_DIR}/Mimic/exhaust_pipe_task/exhaust_pipe_assets/blue_exhaust_pipe.usd"
     scale = (0.55, 0.55, 1.4)
 
-    def __init__(
-        self,
-        instance_name: str | None = None,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(instance_name=instance_name, prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class BrownBox(LibraryObject):
@@ -329,15 +257,6 @@ class BrownBox(LibraryObject):
     usd_path = f"{ISAACLAB_NUCLEUS_DIR}/Arena/assets/object_library/brown_box/brown_box.usd"
     scale = (1.0, 1.0, 1.0)
 
-    def __init__(
-        self,
-        instance_name: str | None = None,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(instance_name=instance_name, prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class Mug(LibraryObject, Placeable):
@@ -348,7 +267,6 @@ class Mug(LibraryObject, Placeable):
     name = "mug"
     tags = ["object"]
     usd_path = f"{ISAACLAB_NUCLEUS_DIR}/Objects/Mug/mug.usd"
-    object_type = ObjectType.RIGID
     scale = (1.0, 1.0, 1.0)
 
     # Placeable affordance parameters
@@ -415,7 +333,6 @@ class Sphere(LibraryObject):
 
     name = "sphere"
     tags = ["object"]
-    object_type = ObjectType.RIGID
     scale = (1.0, 1.0, 1.0)
     default_spawner_cfg = sim_utils.SphereCfg(
         radius=0.1,
@@ -533,16 +450,6 @@ class DexCube(LibraryObject):
     tags = ["object"]
     usd_path = f"{ISAAC_NUCLEUS_DIR}/Props/Blocks/DexCube/dex_cube_instanceable.usd"
     scale = (0.8, 0.8, 0.8)
-    object_type = ObjectType.RIGID
-
-    def __init__(
-        self,
-        instance_name: str | None = None,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(instance_name=instance_name, prim_path=prim_path, initial_pose=initial_pose, scale=scale)
 
 
 @register_asset
@@ -680,16 +587,6 @@ class Broccoli(LibraryObject):
     name = "broccoli"
     tags = ["object", "vegetable", "graspable"]
     usd_path = LightwheelLazyPath(registry_type="objects", registry_name=["broccoli"], file_type="USD")
-    object_type = ObjectType.RIGID
-
-    def __init__(
-        self,
-        instance_name: str | None = None,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(instance_name=instance_name, prim_path=prim_path, initial_pose=initial_pose, scale=scale)
 
 
 @register_asset
@@ -701,18 +598,8 @@ class SweetPotato(LibraryObject):
     name = "sweet_potato"
     tags = ["object", "vegetable", "graspable"]
     usd_path = LightwheelLazyPath(registry_type="objects", file_name="SweetPotato005", file_type="USD")
-    object_type = ObjectType.RIGID
     scale = (1.5, 1.5, 1.5)
 
-    def __init__(
-        self,
-        instance_name: str | None = None,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(instance_name=instance_name, prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class Jug(LibraryObject):
@@ -723,18 +610,8 @@ class Jug(LibraryObject):
     name = "jug"
     tags = ["object", "graspable"]
     usd_path = LightwheelLazyPath(registry_type="objects", file_name="Jug005", file_type="USD")
-    object_type = ObjectType.RIGID
     scale = (2.0, 2.0, 2.0)
 
-    def __init__(
-        self,
-        instance_name: str | None = None,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(instance_name=instance_name, prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class BeerBottle(LibraryObject):
@@ -745,18 +622,8 @@ class BeerBottle(LibraryObject):
     name = "beer_bottle"
     tags = ["object", "graspable"]
     usd_path = LightwheelLazyPath(registry_type="objects", file_name="beer016", file_type="USD")
-    object_type = ObjectType.RIGID
     scale = (1.2, 1.2, 1.2)
 
-    def __init__(
-        self,
-        instance_name: str | None = None,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(instance_name=instance_name, prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class RedCube(LibraryObject):
@@ -769,18 +636,8 @@ class RedCube(LibraryObject):
 
     usd_path = f"{ISAAC_NUCLEUS_DIR}/Props/Blocks/red_block.usd"
 
-    object_type = ObjectType.RIGID
-    default_prim_path = "{ENV_REGEX_NS}/RedCube"
     scale = (0.02, 0.02, 0.02)
 
-    def __init__(
-        self,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class GreenCube(LibraryObject):
@@ -792,18 +649,8 @@ class GreenCube(LibraryObject):
     tags = ["object"]
 
     usd_path = f"{ISAAC_NUCLEUS_DIR}/Props/Blocks/green_block.usd"
-    object_type = ObjectType.RIGID
-    default_prim_path = "{ENV_REGEX_NS}/GreenCube"
     scale = (0.02, 0.02, 0.02)
 
-    def __init__(
-        self,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class RedContainer(LibraryObject):
@@ -814,18 +661,8 @@ class RedContainer(LibraryObject):
     name = "red_container"
     tags = ["object"]
     usd_path = f"{ISAACLAB_NUCLEUS_DIR}/Arena/assets/object_library/isaac_container/container_h20_red.usd"
-    object_type = ObjectType.RIGID
-    default_prim_path = "{ENV_REGEX_NS}/red_container"
     scale = (0.5, 0.5, 0.5)
 
-    def __init__(
-        self,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class GreenContainer(LibraryObject):
@@ -836,18 +673,8 @@ class GreenContainer(LibraryObject):
     name = "green_container"
     tags = ["object"]
     usd_path = f"{ISAACLAB_NUCLEUS_DIR}/Arena/assets/object_library/isaac_container/container_h20_green.usd"
-    object_type = ObjectType.RIGID
-    default_prim_path = "{ENV_REGEX_NS}/green_container"
     scale = (0.5, 0.5, 0.5)
 
-    def __init__(
-        self,
-        prim_path: str | None = None,
-        initial_pose: Pose | None = None,
-        scale: tuple[float, float, float] | None = None,
-    ):
-        super().__init__(prim_path=prim_path, initial_pose=initial_pose, scale=scale)
-
 
 @register_asset
 class BlueBlockBasicRobolab(LibraryObject):

isaaclab_arena/embodiments/droid/droid.py

@@ -100,7 +100,6 @@ class DroidDifferentialIKEmbodiment(DroidEmbodimentBase):
     """Embodiment for the DROID setup with differential inverse kinematics action controller."""
 
     name = "droid_differential_ik"
-    tags = ["embodiment", "ik"]
     default_arm_mode = ArmMode.SINGLE_ARM
 
     def __init__(
@@ -139,6 +138,7 @@ class DroidAbsoluteJointPositionEmbodiment(DroidEmbodimentBase):
     """Embodiment for the DROID setup with absolute joint position actions."""
 
     name = "droid_abs_joint_pos"
+    tags = ["embodiment", "default"]
     default_arm_mode = ArmMode.SINGLE_ARM
 
     def __init__(

isaaclab_arena/embodiments/franka/franka.py

@@ -101,7 +101,7 @@ class FrankaIKEmbodiment(FrankaEmbodimentBase):
     """Franka with differential IK (relative) arm control and high-PD defaults."""
 
     name = "franka_ik"
-    tags = ["embodiment", "ik"]
+    tags = ["embodiment", "default"]
 
     def __init__(
         self,

isaaclab_arena/embodiments/g1/g1.py

@@ -142,7 +142,7 @@ class G1WBCPinkEmbodiment(G1EmbodimentBase):
     """
 
     name = "g1_wbc_pink"
-    tags = ["embodiment", "ik"]
+    tags = ["embodiment", "default"]
 
     def __init__(
         self,