[RL] Update RSL-RL to 5.5.1 - #7825
Conversation
There was a problem hiding this comment.
Isaac Lab Review Bot
The dependency-only update consistently pins rsl-rl-lib 5.5.1 in the base dependencies and rsl-rl extra, with corresponding lockfile updates. The removed direct onnxscript declaration remains satisfied transitively by RSL-RL.
- Design and architecture: Dependency ownership is simplified without introducing version skew: both RSL-RL declarations use 5.5.1, and
onnxscriptis delegated to the RSL-RL package that requires it. The lockfile confirms that dependency remains in the resolved graph. - API: No public Python API, CLI argument, or configuration field changes. Project metadata and the generated lockfile agree for the base and optional-extra dependency paths, and no source-package changelog or documentation update is required.
- Implementation: The updated lock entry for
rsl-rl-lib5.5.1 retainsonnxscripttransitively. Although RSL-RL no longer brings intorchvision, the project continues to declare it directly. The remaining tradeoff is that the minimum supportedonnxscriptversion is now controlled by RSL-RL rather than an Isaac Lab constraint.
No blocking issues. No inline issue met the actionable-evidence threshold; the assessment above records the review feedback.
Automated review; human maintainers own approval decisions.
|
|
run-ci |
Description
Updates the
rsl-rl-libpin from 5.4.1 to 5.5.1 and removes the redundant directonnxscriptdependency, which RSL-RL already provides.This is the dependency-only portion of #7784; no compatibility shims, tests, or runtime code are changed.
Type of change
Release backport
developScreenshots
Not applicable.
Validation
uv lock --checkuv run isaaclab -fuv run python -m pytest source/isaaclab_rl/test/test_rsl_rl_cfg_deprecation.py -q(39 passed)Checklist
uv run isaaclab -fCONTRIBUTORS.md