Fix post-merge image build broken by SimReady pkg

[xyao-nv]

Summary

Fix post-merge image build broken by SimReady pkg pip

Detailed description

• Add the simready-search dependency #999 listed simready-search==0.2.1, but pypi.nvidia.com carries only 2026.4.2. The pin was a leftover, so the post-merge image build died at that layer.
• Fix: dropped the re-pin and used the version in [dev] as the single source of truth.

[arena-review-bot]

🤖 Isaac Lab-Arena Review Bot

Summary

This PR drops a stale simready-search==0.2.1 pin from the Dockerfile's hand-applied --force-reinstall --no-deps step, which was breaking the post-merge image build because that version does not exist on pypi.nvidia.com (only 2026.4.2 is published). simready-search is already installed by the preceding pip install -e .[dev] (declared as >=2026.4.2 in pyproject.toml), so removing the re-pin leaves pyproject.toml as the single source of truth. Verified against pyproject.toml: simready-search is not part of override-dependencies, and the remaining force-reinstalled packages (boto3/botocore/s3transfer/requests) still match the AWS/HTTP stack overrides — the fix is correct and minimal.

Test Coverage

Not applicable — this is a Docker build fix; the post-merge image build itself is the validating check.

Verdict

Ship it

[greptile-apps]

Greptile Summary

This PR fixes the base image build by removing an obsolete, unavailable simready-search==0.2.1 reinstall and relying on the version declared by the project’s dev extra.

• Keeps pyproject.toml as the single source of truth for simready-search.
• Retains the explicit AWS and requests dependency overrides required by Isaac Sim.

Confidence Score: 5/5

The PR appears safe to merge, with the obsolete package reinstall removed while the package remains installed through the existing development extra.

The editable development installation already declares simready-search at the available version, and the subsequent command continues to enforce the Isaac Sim-compatible boto3, botocore, s3transfer, and requests versions.

Important Files Changed

Filename	Overview
docker/Dockerfile.isaaclab_arena	Removes a redundant invalid package pin while preserving the Docker image’s intentional dependency compatibility overrides.

_{Reviews (1): Last reviewed commit: "Fix post-merge image build broken by a s..." | Re-trigger Greptile}