Make sure sup-components that live under source/isaaclab/ are called "sub-packages", not extras or anything else

Author: myurasov-nv

docs/source/setup/installation/include/src_build_isaaclab.rst

@@ -60,8 +60,8 @@ Installation
 
             isaaclab.bat --install rl_games :: or "isaaclab.bat -i rl_games"
 
-   To install only specific Isaac Lab sub-packages, pass a comma-separated list of extras. The available
-   extras are: ``assets``, ``physx``, ``contrib``, ``mimic``, ``newton``, ``rl``, ``tasks``,
+   To install only specific sub-packages, pass a comma-separated list of sub-package names. The available
+   sub-packages are: ``assets``, ``physx``, ``contrib``, ``mimic``, ``newton``, ``rl``, ``tasks``,
    ``teleop``. For example, to install only the ``mimic`` and ``assets`` sub-packages:
 
    .. tab-set::

docs/source/setup/installation/include/src_clone_isaaclab.rst

@@ -46,13 +46,13 @@ respectively that provides utilities to manage extensions.
          options:
            -h, --help            show this help message and exit
            -i [INSTALL], --install [INSTALL]
-                                 Install the Isaac Lab extras and RL frameworks.
-                                 Accepts a comma-separated list of extras, one of the RL frameworks, or a special value.
+                                 Install Isaac Lab sub-packages and RL frameworks.
+                                 Accepts a comma-separated list of sub-package names, one of the RL frameworks, or a special value.
 
-                                 Extras: assets, physx, contrib, mimic, newton, rl, tasks, teleop.
+                                 Sub-packages: assets, physx, contrib, mimic, newton, rl, tasks, teleop.
                                  RL frameworks: rl_games, rsl_rl, sb3, skrl, robomimic.
 
-                                 Passing an RL framework name installs all extensions + that framework.
+                                 Passing an RL framework name installs all sub-packages + that framework.
 
                                  Special values:
                                  - all  - Install all extensions + all RL frameworks (default).
@@ -84,13 +84,13 @@ respectively that provides utilities to manage extensions.
          options:
            -h, --help            show this help message and exit
            -i [INSTALL], --install [INSTALL]
-                                 Install the Isaac Lab extras and RL frameworks.
-                                 Accepts a comma-separated list of extras, one of the RL frameworks, or a special value.
+                                 Install Isaac Lab sub-packages and RL frameworks.
+                                 Accepts a comma-separated list of sub-package names, one of the RL frameworks, or a special value.
 
-                                 Extras: assets, physx, contrib, mimic, newton, rl, tasks, teleop.
+                                 Sub-packages: assets, physx, contrib, mimic, newton, rl, tasks, teleop.
                                  RL frameworks: rl_games, rsl_rl, sb3, skrl, robomimic.
 
-                                 Passing an RL framework name installs all extensions + that framework.
+                                 Passing an RL framework name installs all sub-packages + that framework.
 
                                  Special values:
                                  - all  - Install all extensions + all RL frameworks (default).

source/isaaclab/isaaclab/cli/__init__.py

@@ -36,18 +36,18 @@ def cli() -> None:
         nargs="?",
         const="all",
         help=(
-            "Install the Isaac Lab extras and RL frameworks.\n"
-            "Accepts a comma-separated list of extras, one of the RL frameworks, or a special value.\n"
+            "Install Isaac Lab sub-packages and RL frameworks.\n"
+            "Accepts a comma-separated list of sub-package names, one of the RL frameworks, or a special value.\n"
             "\n"
-            "Extras: assets, physx, contrib, mimic, newton, rl, tasks, teleop.\n"
+            "Sub-packages: assets, physx, contrib, mimic, newton, rl, tasks, teleop.\n"
             "RL frameworks: rl_games, rsl_rl, sb3, skrl, robomimic.\n"
             "\n"
-            "Passing an RL framework name installs all extensions + that framework.\n"
+            "Passing an RL framework name installs all sub-packages + that framework.\n"
             "\n"
             "Special values:\n"
-            "- all  - Install all extensions + all RL frameworks (default).\n"
+            "- all  - Install all sub-packages + all RL frameworks (default).\n"
             "- none - Install only the core 'isaaclab' package.\n"
-            "- <empty> (-i or --install without value) - Install all extensions + all RL frameworks.\n"
+            "- <empty> (-i or --install without value) - Install all sub-packages + all RL frameworks.\n"
         ),
     )
     parser.add_argument(

source/isaaclab/isaaclab/cli/commands/install.py

@@ -121,9 +121,9 @@ def _ensure_cuda_torch() -> None:
     )
 
 
-# Valid "extras" names that can be passed to --install.
-# Each "extra" maps to a source directory named "isaaclab_<extra>" under source/.
-VALID_ISAACLAB_EXTRAS: set[str] = {"assets", "physx", "contrib", "mimic", "newton", "rl", "tasks", "teleop"}
+# Valid sub-package names that can be passed to --install.
+# Each sub-package maps to a source directory named "isaaclab_<name>" under source/.
+VALID_ISAACLAB_SUBPACKAGES: set[str] = {"assets", "physx", "contrib", "mimic", "newton", "rl", "tasks", "teleop"}
 
 # RL framework names accepted.
 # Passing one of these installs all extensions + that framework.
@@ -257,15 +257,15 @@ def command_install(install_type: str = "all") -> None:
         extensions = None
         framework_type = install_type
     else:
-        # Parse comma-separated extras into source directory names.
+        # Parse comma-separated sub-package names into source directory names.
         extensions = ["isaaclab"]  # core is always required
-        for extra in (s.strip() for s in install_type.split(",") if s.strip()):
-            if extra in VALID_ISAACLAB_EXTRAS:
-                extensions.append(f"isaaclab_{extra}")
+        for name in (s.strip() for s in install_type.split(",") if s.strip()):
+            if name in VALID_ISAACLAB_SUBPACKAGES:
+                extensions.append(f"isaaclab_{name}")
             else:
-                valid = sorted(VALID_ISAACLAB_EXTRAS) + sorted(VALID_RL_FRAMEWORKS)
-                print_warning(f"Unknown extra '{extra}'. Valid values: {', '.join(valid)}. Skipping.")
-        framework_type = "none"  # RL framework extras not applied in selective mode
+                valid = sorted(VALID_ISAACLAB_SUBPACKAGES) + sorted(VALID_RL_FRAMEWORKS)
+                print_warning(f"Unknown sub-package '{name}'. Valid values: {', '.join(valid)}. Skipping.")
+        framework_type = "none"  # RL frameworks not applied in selective mode
 
     # if on ARM arch, temporarily clear LD_PRELOAD
     # LD_PRELOAD is restored below, after installation