Fix failing CI

Author: StafaH

AGENTS.md

@@ -224,7 +224,7 @@ Notes:
 Snake-style `Test_<Filename_Stem_TitleCase>`, mirroring the filename. Each underscore-separated segment is TitleCase. Common acronyms stay uppercase when separate segments. **Do not** use CamelCase here; snake-style is the deliberate convention for these long compound names.
 
 - ✅ `Test_Cli_Install_Core_In_Uvenv_Correctness`
-- ✅ `Test_Uv_Pip_Install_Isaaclab_All_Isaacsim_Trains_Cartpole`
+- ✅ `Test_Uv_Pip_Install_Isaaclab_Isaacsim_Rl_Trains_Cartpole`
 - ❌ `TestCliInstallCoreInUvenvCorrectness`
 
 #### Method naming (strict)

docs/_extensions/isaaclab_docs.py

@@ -216,7 +216,7 @@ def run(self) -> list[nodes.Node]:
         content = f"""\
 .. code-block:: bash
 
-   uv pip install "isaaclab[isaacsim,all]" \\
+   uv pip install "isaaclab[isaacsim,sb3,skrl,rsl-rl]" \\
      --overrides "{overrides_url}" \\
      --extra-index-url https://pypi.nvidia.com \\
      --index-strategy unsafe-best-match --prerelease=allow

source/isaaclab/changelog.d/mh-install-docs-detailed.rst

@@ -0,0 +1,7 @@
+Removed
+^^^^^^^
+
+* **Breaking:** Removed the aggregate ``all`` extra. Request the extras you need by name
+  instead, for example ``uv pip install "isaaclab[isaacsim,sb3,skrl,rsl-rl]"`` in place of
+  ``uv pip install "isaaclab[isaacsim,all]"``, or ``uv run --extra sb3 --extra skrl`` in place
+  of ``uv run --extra all``. See :ref:`installation-optional-extras` for the full extras table.

source/isaaclab/test/cli/test_uv_run_pyproject.py

@@ -61,13 +61,14 @@ def test_uv_run_exposes_centralized_feature_extras():
         "teleop",
         "rlinf",
         "tetrahedralization",
-        "all",
     }
     assert expected_extras <= set(optional_dependencies)
 
     # The Newton viewer GUI is part of the base install, so there is no ``newton`` extra.
     assert "newton" not in optional_dependencies
     assert "rtx" not in optional_dependencies
+    # Each extra is requested by name; there is no aggregate ``all`` extra.
+    assert "all" not in optional_dependencies
 
     # Concrete third-party deps live in the extras (not subpackage self-references).
     # ``ov`` installs both Omniverse backends; ``ovphysx`` / ``ovrtx`` select one.
@@ -82,14 +83,18 @@ def test_uv_run_exposes_centralized_feature_extras():
     assert any(dep.startswith("ovstage") for dep in optional_dependencies["ovrtx"])
 
 
-def test_tetrahedralization_is_explicit_and_excluded_from_all():
+def test_tetrahedralization_is_explicit_extra_only():
     """TetWild and its visualization stack are installed only when requested."""
     project = _root_pyproject()["project"]
     optional = project["optional-dependencies"]
 
     assert not any(dep.startswith("pytetwild") for dep in project["dependencies"])
     assert optional["tetrahedralization"] == ["pytetwild[all]>=0.3.0,<0.4"]
-    assert not any("tetrahedralization" in dep or dep.startswith("pytetwild") for dep in optional["all"])
+    # No other extra pulls PyTetWild in transitively.
+    for name, deps in optional.items():
+        if name == "tetrahedralization":
+            continue
+        assert not any("tetrahedralization" in dep or dep.startswith("pytetwild") for dep in deps)
 
 
 def test_version_single_source_matches_literal_pins():
@@ -190,7 +195,6 @@ def test_uv_run_isaacsim_extra_handles_dependency_conflicts():
     conflict_groups = [{entry["extra"] for entry in group} for group in pyproject["tool"]["uv"]["conflicts"]]
     for extra in ("ov", "ovphysx"):
         assert {"isaacsim", extra} in conflict_groups, f"isaacsim must declare a conflict with '{extra}'"
-    assert {"isaacsim", "all"} not in conflict_groups
     # ``test`` is no longer forked away: the coverage override reconciles it with Isaac Sim.
     assert {"isaacsim", "test"} not in conflict_groups
     # ``mimic`` is no longer forked away either: robomimic dropped its lxml constraint, so
@@ -217,7 +221,6 @@ def test_uv_run_teleop_co_resolves_with_isaacsim():
     # The historical lxml split is gone: robomimic no longer constrains lxml, so the teleop
     # and imitation-learning stacks co-resolve.
     assert {"teleop", "mimic"} not in conflict_groups
-    assert {"teleop", "all"} not in conflict_groups
 
 
 def test_uv_run_teleop_extra_bundles_isaacsim():
@@ -242,7 +245,7 @@ def test_uv_run_teleop_extra_bundles_isaacsim():
     # Only the packaging split is real; the overrides reconcile everything else.
     for extra in ("ov", "ovphysx"):
         assert {"teleop", extra} in conflict_groups, f"teleop must declare a conflict with '{extra}'"
-    for extra in ("mimic", "all", "viser", "test"):
+    for extra in ("mimic", "viser", "test"):
         assert {"teleop", extra} not in conflict_groups
     # ``--extra teleop --extra test`` must keep working so the teleop suite stays runnable.
     assert {"teleop", "test"} not in conflict_groups

source/isaaclab/test/cli/test_wheel_builder_metadata.py

@@ -96,10 +96,9 @@ def test_wheel_builder_rsl_rl_pin_matches_root_pyproject(tmp_path):
     assert core_pins == [expected_pin]
 
     optional_dependencies = generated["project"]["optional-dependencies"]
-    # RSL-RL ships in its own ``rsl-rl`` extra and in the aggregate ``all`` extra.
-    for extra_name in ("rsl-rl", "all"):
-        rsl_rl_pins = [dep for dep in optional_dependencies[extra_name] if dep.startswith("rsl-rl-lib==")]
-        assert rsl_rl_pins == [expected_pin]
+    # RSL-RL is also exposed through its own ``rsl-rl`` extra.
+    rsl_rl_pins = [dep for dep in optional_dependencies["rsl-rl"] if dep.startswith("rsl-rl-lib==")]
+    assert rsl_rl_pins == [expected_pin]
 
 
 def test_wheel_builder_keeps_tetrahedralization_explicit(tmp_path):
@@ -110,7 +109,10 @@ def test_wheel_builder_keeps_tetrahedralization_explicit(tmp_path):
 
     assert not any(dep.startswith("pytetwild") for dep in project["dependencies"])
     assert optional_dependencies["tetrahedralization"] == ["pytetwild[all]>=0.3.0,<0.4"]
-    assert not any(dep.startswith("pytetwild") for dep in optional_dependencies["all"])
+    for name, deps in optional_dependencies.items():
+        if name == "tetrahedralization":
+            continue
+        assert not any(dep.startswith("pytetwild") for dep in deps)
 
 
 def test_wheel_builder_uv_overrides_match_root_pyproject(tmp_path):

source/isaaclab/test/install_ci/misc/test_wheel_builder_smoke.py

@@ -7,7 +7,7 @@
 Setup:
     - bash tools/wheel_builder/build.sh
     - ./isaaclab.sh -u
-    - uv pip install <wheel>[all]
+    - uv pip install <wheel>[sb3,skrl,rsl-rl]
 Tests:
     - import isaaclab -> verify importable
     - from isaaclab import __version__ -> verify version matches wheel filename
@@ -19,7 +19,7 @@
     - python -m isaaclab --help -> verify CLI functional
     - import pinocchio -> verify importable
     - python -c "import importlib.util; raise SystemExit(importlib.util.find_spec('pytetwild') is not None)"
-        -> verify the all extra omits tetrahedralization dependencies
+        -> verify the RL extras omit tetrahedralization dependencies
 """
 
 from __future__ import annotations
@@ -36,6 +36,7 @@ class Test_Wheel_Builder_Smoke(UV_Mixin):
     """Test building the isaaclab wheel and installing it in a uv environment."""
 
     _wheel: str = ""
+    _extras: str = "[sb3,skrl,rsl-rl]"
 
     @classmethod
     def setup_class(cls):
@@ -67,7 +68,7 @@ def _build_and_install_wheel(self, isaaclab_root):
         cls.env_path = self.env_path
         cls.python = self.python
         cls.cli_script = self.cli_script
-        result = self.run_in_uv_env(["uv", "pip", "install", cls._wheel + "[all]"])
+        result = self.run_in_uv_env(["uv", "pip", "install", cls._wheel + cls._extras])
         assert result.returncode == 0, f"uv pip install wheel failed:\n{result.stdout}\n{result.stderr}"
 
         yield
@@ -134,13 +135,15 @@ def test_pinocchio_importable(self):
         result = self.run_in_uv_env(["python", "-c", "import pinocchio as pin; print(pin.__version__)"])
         assert result.returncode == 0, f"import pinocchio failed:\n{result.stdout}\n{result.stderr}"
 
-    def test_install_all_omits_tetrahedralization_dependencies(self):
-        """Verify the wheel's all extra does not install pytetwild."""
+    def test_install_rl_extras_omits_tetrahedralization_dependencies(self):
+        """Verify the wheel's RL extras do not install pytetwild."""
         result = self.run_in_uv_env(
             [
                 "python",
                 "-c",
                 "import importlib.util; raise SystemExit(importlib.util.find_spec('pytetwild') is not None)",
             ]
         )
-        assert result.returncode == 0, f"pytetwild should not be installed by [all]:\n{result.stdout}\n{result.stderr}"
+        assert result.returncode == 0, (
+            f"pytetwild should not be installed by {self._extras}:\n{result.stdout}\n{result.stderr}"
+        )

…isaaclab_all_isaacsim_trains_cartpole.py …_isaaclab_isaacsim_rl_trains_cartpole.pysource/isaaclab/test/install_ci/uv_pip/test_uv_pip_install_isaaclab_all_isaacsim_trains_cartpole.py renamed to source/isaaclab/test/install_ci/uv_pip/test_uv_pip_install_isaaclab_isaacsim_rl_trains_cartpole.py source/isaaclab/test/install_ci/uv_pip/test_uv_pip_install_isaaclab_all_isaacsim_trains_cartpole.py renamed to source/isaaclab/test/install_ci/uv_pip/test_uv_pip_install_isaaclab_isaacsim_rl_trains_cartpole.py

@@ -7,7 +7,7 @@
 Setup:
     - (wheel supplied by runner: tools/run_install_ci.py --build-wheel or --wheel <path>)
     - ./isaaclab.sh -u
-    - uv pip install <wheel>[all,isaacsim] --overrides uv_pip/uv-overrides.txt
+    - uv pip install <wheel>[isaacsim,sb3,skrl,rsl-rl] --overrides uv_pip/uv-overrides.txt
         --extra-index-url https://pypi.nvidia.com --index-strategy unsafe-best-match --prerelease=allow
     - uv pip install --reinstall-package torch --reinstall-package torchvision
         torch==<pinned> torchvision==<pinned> --index-url <cu128|cu130>
@@ -31,8 +31,8 @@
 
 
 @pytest.mark.install_path_uv_pip
-class Test_Uv_Pip_Install_Isaaclab_All_Isaacsim_Trains_Cartpole(UV_Mixin):
-    """Build the wheel, ``uv pip install <wheel>[all,isaacsim]``, verify cartpole training."""
+class Test_Uv_Pip_Install_Isaaclab_Isaacsim_Rl_Trains_Cartpole(UV_Mixin):
+    """Build the wheel, ``uv pip install <wheel>[isaacsim,sb3,skrl,rsl-rl]``, verify cartpole training."""
 
     @classmethod
     def setup_class(cls):
@@ -45,15 +45,15 @@ def setup_class(cls):
     @pytest.mark.slow
     @pytest.mark.gpu
     @pytest.mark.timeout(4800)
-    def test_uv_pip_install_isaaclab_all_isaacsim_trains_cartpole(
+    def test_uv_pip_install_isaaclab_isaacsim_rl_trains_cartpole(
         self, isaaclab_root, wheel, uv_overrides, cartpole_smoke_script
     ):
-        """Install the runner-supplied wheel with ``[all,isaacsim]`` via ``uv pip``, run cartpole training."""
+        """Install the runner-supplied wheel with ``[isaacsim,sb3,skrl,rsl-rl]`` via ``uv pip``, then train."""
         try:
-            # 1. Create the uv env and install the wheel with [all,isaacsim] extras.
+            # 1. Create the uv env and install the wheel with [isaacsim,sb3,skrl,rsl-rl] extras.
             self.create_uv_env(isaaclab_root)
 
-            # uv pip install "isaaclab[all,isaacsim]" --extra-index-url https://pypi.nvidia.com
+            # uv pip install "isaaclab[isaacsim,sb3,skrl,rsl-rl]" --extra-index-url https://pypi.nvidia.com
             #   --index-strategy unsafe-best-match --prerelease=allow
             # NOTE: --index-strategy unsafe-best-match re-resolves torch from PyPI (CPU build),
             #       overriding any pre-installed CUDA torch. So install isaaclab FIRST, then
@@ -63,7 +63,7 @@ def test_uv_pip_install_isaaclab_all_isaacsim_trains_cartpole(
                     "uv",
                     "pip",
                     "install",
-                    f"{wheel}[all,isaacsim]",
+                    f"{wheel}[isaacsim,sb3,skrl,rsl-rl]",
                     "--overrides",
                     str(uv_overrides),
                     "--extra-index-url",
@@ -76,7 +76,7 @@ def test_uv_pip_install_isaaclab_all_isaacsim_trains_cartpole(
                 timeout=1800,
             )
             assert result.returncode == 0, (
-                f"uv pip install {wheel}[all,isaacsim] failed:\n{result.stdout}\n{result.stderr}"
+                f"uv pip install {wheel}[isaacsim,sb3,skrl,rsl-rl] failed:\n{result.stdout}\n{result.stderr}"
             )
 
             # 2. uv pip install --reinstall-package torch --reinstall-package torchvision

source/isaaclab/test/install_ci/uv_pip/test_uv_pip_install_isaaclab_rl_tasks_imports_rl_tasks.py

@@ -7,7 +7,7 @@
 Setup:
     - (wheel supplied by runner: tools/run_install_ci.py --build-wheel or --wheel <path>)
     - ./isaaclab.sh -u
-    - uv pip install <wheel>[all]
+    - uv pip install <wheel>[sb3,skrl,rsl-rl]
 Tests:
     - python -c "import isaaclab_rl" -> verify isaaclab_rl importable
     - python -c "import isaaclab_tasks" -> verify isaaclab_tasks importable
@@ -24,9 +24,10 @@
 
 @pytest.mark.install_path_uv_pip
 class Test_Uv_Pip_Install_Isaaclab_Rl_Tasks_Imports_Rl_Tasks(UV_Mixin):
-    """``uv pip install <wheel>[all]``: verify RL imports without Isaac Sim."""
+    """``uv pip install <wheel>[sb3,skrl,rsl-rl]``: verify RL imports without Isaac Sim."""
 
     _wheel: str = ""
+    _extras: str = "[sb3,skrl,rsl-rl]"
 
     @classmethod
     def setup_class(cls):
@@ -38,18 +39,19 @@ def _install_wheel(self, isaaclab_root, wheel):
         cls = self.__class__
         cls._wheel = str(wheel)
 
-        # Create the uv env and install with the [all] extra (no isaacsim, no NVIDIA flags).
+        # Create the uv env and install the RL extras (no isaacsim, no NVIDIA flags).
         self.create_uv_env(isaaclab_root)
         cls.env_path = self.env_path
         cls.python = self.python
         cls.cli_script = self.cli_script
 
+        target = f"{cls._wheel}{cls._extras}"
         result = self.run_in_uv_env(
-            ["uv", "pip", "install", f"{cls._wheel}[all]"],
+            ["uv", "pip", "install", target],
             cwd=isaaclab_root,
             timeout=1200,
         )
-        assert result.returncode == 0, f"uv pip install {cls._wheel}[all] failed:\n{result.stdout}\n{result.stderr}"
+        assert result.returncode == 0, f"uv pip install {target} failed:\n{result.stdout}\n{result.stderr}"
 
         yield
 
@@ -60,7 +62,7 @@ def _install_wheel(self, isaaclab_root, wheel):
     @pytest.mark.slow
     @pytest.mark.timeout(1200)
     def test_install_rl_tasks_makes_isaaclab_rl_importable(self):
-        """``import isaaclab_rl`` succeeds after ``uv pip install <wheel>[all]``."""
+        """``import isaaclab_rl`` succeeds after ``uv pip install <wheel>[sb3,skrl,rsl-rl]``."""
         result = self.run_in_uv_env(["python", "-c", "import isaaclab_rl"])
         assert result.returncode == 0, f"import isaaclab_rl failed:\n{result.stdout}\n{result.stderr}"
 
@@ -69,7 +71,7 @@ def test_install_rl_tasks_makes_isaaclab_rl_importable(self):
     @pytest.mark.slow
     @pytest.mark.timeout(1200)
     def test_install_rl_tasks_makes_isaaclab_tasks_importable(self):
-        """``import isaaclab_tasks`` succeeds after ``uv pip install <wheel>[all]``."""
+        """``import isaaclab_tasks`` succeeds after ``uv pip install <wheel>[sb3,skrl,rsl-rl]``."""
         result = self.run_in_uv_env(["python", "-c", "import isaaclab_tasks"])
         assert result.returncode == 0, f"import isaaclab_tasks failed:\n{result.stdout}\n{result.stderr}"
 
@@ -78,6 +80,6 @@ def test_install_rl_tasks_makes_isaaclab_tasks_importable(self):
     @pytest.mark.slow
     @pytest.mark.timeout(1200)
     def test_install_rl_tasks_omits_isaacsim(self):
-        """``import isaacsim`` fails after ``uv pip install <wheel>[all]`` (extra not requested)."""
+        """``import isaacsim`` fails after installing the RL extras (isaacsim extra not requested)."""
         result = self.run_in_uv_env(["python", "-c", "import isaacsim"])
-        assert result.returncode != 0, "isaacsim should not be installed by [all] extra"
+        assert result.returncode != 0, f"isaacsim should not be installed by the {self._extras} extras"

source/isaaclab_newton/isaaclab_newton/renderers/newton_warp_renderer.py

@@ -34,7 +34,7 @@
 
 _PPISP_IMPORT_ERROR_MESSAGE = (
     "isaaclab_ppisp is required when CameraCfg.isp_cfg is set. "
-    "Install Isaac Lab with the 'all' extra (`pip install isaaclab[all]`) or install the "
+    "It ships with the Isaac Lab wheel (`pip install isaaclab`); otherwise install the "
     "isaaclab-ppisp extension from the Isaac Lab source checkout."
 )
 

source/isaaclab_ov/isaaclab_ov/renderers/ovrtx_renderer.py

@@ -116,7 +116,7 @@
 
 _PPISP_IMPORT_ERROR_MESSAGE = (
     "isaaclab_ppisp is required when CameraCfg.isp_cfg is set. "
-    "Install Isaac Lab with the 'all' extra (`pip install isaaclab[all]`) or install the "
+    "It ships with the Isaac Lab wheel (`pip install isaaclab`); otherwise install the "
     "isaaclab-ppisp extension from the Isaac Lab source checkout."
 )
 _READ_GPU_TRANSFORMS_ENV = "ISAAC_LAB_OVRTX_READ_GPU_TRANSFORMS"