Pin packaging>=20.0 to fix Theia CI import failure

[AntoineRichard]

Description

Theia observation tests on CI fail when transformers is imported:

ValueError: Unable to compare versions for packaging>=20.0:
need=20.0 found=None. This is unusual. Consider reinstalling
packaging.

transformers calls importlib.metadata.version("packaging") at import time. On Isaac Sim's bundled Python, packaging is on sys.path but the dist-info metadata is missing, so the lookup returns None. Because setup.py listed packaging without a version floor, pip considered the broken copy satisfied and skipped reinstall.

Add >=20.0 (matching transformers' own require_version call) so pip re-resolves packaging and reinstalls it with valid metadata.

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
List any dependencies that are required for this change.

Checklist

• I have read and understood the contribution guidelines
• I have run the pre-commit checks with ./isaaclab.sh --format
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• I have updated the changelog and the corresponding version in the extension's config/extension.toml file
• I have added my name to the CONTRIBUTORS.md or my name already exists there

[isaaclab-review-bot]

🤖 Isaac Lab Review Bot

Summary

This PR pins packaging>=20.0 in setup.py to fix a CI failure where transformers import fails due to Isaac Sim's bundled packaging library lacking proper dist-info metadata. The fix is minimal, targeted, and correctly documented. The version floor matches the constraint that transformers itself checks at runtime.

Architecture Impact

Self-contained. This change only affects dependency resolution at install time. No runtime code paths are modified. The packaging library is a pure Python utility for version parsing — adding a version floor has no impact on Isaac Lab's tensor pipelines, physics stepping, or RL training loops.

Implementation Verdict

Ship it

Test Coverage

This is a dependency/CI fix, not a logic change. The "test" is whether CI passes after the change — specifically the Theia observation tests that import transformers. No unit test is needed or appropriate for a setup.py dependency pin. The PR description indicates this fixes an existing CI failure, which will serve as implicit validation.

CI Status

No CI checks available yet. The PR should be validated by confirming the Theia-based observation tests pass after merge.

Findings

🔵 Improvement: source/isaaclab/setup.py:54-55 — Comment accuracy
The inline comment is excellent and explains the why clearly. However, the comment says "forces pip to reinstall" which is slightly imprecise — pip will reinstall only if the existing installation doesn't satisfy the constraint. This is minor and the current wording is acceptable for practical purposes.

🔵 Improvement: source/isaaclab/docs/CHANGELOG.rst:3 — Changelog entry is thorough
The changelog entry at lines 3-14 correctly documents the root cause (missing dist-info metadata in Isaac Sim's bundled packaging), the symptom (ValueError on transformers import), and the fix. This is well-written documentation.

🔵 Note: source/isaaclab/config/extension.toml:4 — Version bump is correct
Version incremented from 4.6.11 to 4.6.12 following semver for a patch-level fix. The extension.toml and CHANGELOG.rst are in sync.

No critical or warning-level issues identified. The change is correct, minimal, and well-documented.

[greptile-apps]

Greptile Summary

Pins packaging>=20.0 in setup.py to fix a CI import failure where Isaac Sim's bundled Python ships packaging without dist-info metadata, causing transformers to raise a ValueError at import time. The patch version is bumped to 4.6.12 and the changelog is updated accordingly.

Confidence Score: 5/5

Safe to merge — minimal, targeted dependency floor that resolves a well-diagnosed CI failure with no functional risk.

All three changed files are trivially correct: the version specifier floor matches transformers' own runtime check, the version bump and changelog are consistent, and no logic changes are introduced.

No files require special attention.

Important Files Changed

Filename	Overview
source/isaaclab/setup.py	Adds packaging>=20.0 floor to force pip to reinstall the metadata-less Isaac Sim bundled copy; change is minimal and correctly targeted.
source/isaaclab/config/extension.toml	Patch version bump from 4.6.11 → 4.6.12, consistent with the changelog entry.
source/isaaclab/docs/CHANGELOG.rst	Adds 4.6.12 changelog entry accurately describing the packaging pin fix.

Sequence Diagram

sequenceDiagram
    participant pip
    participant setup.py
    participant IsaacSimPython as Isaac Sim Python
    participant transformers

    pip->>setup.py: resolve install_requires
    setup.py-->>pip: packaging>=20.0
    pip->>IsaacSimPython: check installed packaging version
    IsaacSimPython-->>pip: metadata-less copy (dist-info missing)
    pip->>pip: version constraint not satisfied → reinstall
    pip-->>IsaacSimPython: packaging with valid dist-info
    IsaacSimPython->>transformers: import AutoModel
    transformers->>IsaacSimPython: importlib.metadata.version("packaging")
    IsaacSimPython-->>transformers: "24.x" (valid)
    transformers-->>IsaacSimPython: import succeeds

Loading

_{Reviews (1): Last reviewed commit: "Pin packaging>=20.0 to fix Theia CI impo..." | Re-trigger Greptile}