Skip to content

Pin RSL-RL dependency to a fixed commit - #35

Merged
patrickhaoy merged 1 commit into
mainfrom
pat/pin-rsl-rl
Sep 15, 2026
Merged

patrickhaoy merged 1 commit into
mainfrom
pat/pin-rsl-rl

Conversation

@patrickhaoy

Copy link
Copy Markdown
Collaborator

Description

uwlab_rl[rsl-rl] currently installs RSL-RL from the moving main branch. A new RSL-RL push can therefore change what the same UWLab revision installs before UWLab has adopted the corresponding API changes.

Pin the dependency to e7cd3c77bdb3c94753612f208c725e1add38a655, the current RSL-RL main revision. This preserves the runtime code currently selected by @main; the latest RSL-RL commit only added Greptile configuration. Future RSL-RL upgrades should update this pin together with compatible UWLab changes. The all extra inherits the same pin.

Bump uwlab_rl from 0.1.3 to 0.1.4 and record the installation fix in its changelog.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Validation

  • Parsed setup.py and the dependency using a PEP 508 requirement parser; verified the full SHA resolves to a commit in RSL-RL.
  • Evaluated the extras definitions and verified rsl-rl and all contain the same pinned requirement.
  • Verified the extension version matches the changelog and ran git diff --check.
  • Ran configured pre-commit hooks on all three changed files. The pinned pyupgrade release crashes under local Python 3.14; reran that hook successfully with Python 3.11 using a temporary config with identical hook versions and arguments. No repository tooling changes.
  • Simulator/training tests were not run; this changes installation metadata only.

Checklist

  • Followed the repository's contribution and review conventions.
  • Ran applicable pre-commit checks on the changed files (see validation).
  • Updated the changelog and corresponding extension version.
  • My name already exists in CONTRIBUTORS.md.

@greptile-apps

greptile-apps Bot commented Sep 15, 2026

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

The PR appears safe to merge, with the dependency pin and release metadata applied consistently.

Summary

This PR makes RSL-RL installations reproducible by replacing the moving main reference with a full commit pin.

  • Pins the rsl-rl extra, inherited by the all extra, to commit e7cd3c77bdb3c94753612f208c725e1add38a655.
  • Bumps the uwlab_rl extension and package version from 0.1.3 to 0.1.4.
  • Adds a matching changelog entry explaining the installation fix.

Reviews (1) · Last reviewed commit: "Pin RSL-RL dependency to a fixed commit"

@patrickhaoy
patrickhaoy merged commit 333e287 into main Sep 15, 2026
4 of 9 checks passed
@patrickhaoy
patrickhaoy deleted the pat/pin-rsl-rl branch September 15, 2026 06:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant