Skip to content

refactor(robots): port robots to latest upstream RLinf APIs - #216

Open
jx-qiu wants to merge 10 commits into
RLinf:mainfrom
jx-qiu:refactor/rlinf-main
Open

jx-qiu wants to merge 10 commits into
RLinf:mainfrom
jx-qiu:refactor/rlinf-main

Conversation

@jx-qiu

@jx-qiu jx-qiu commented Sep 23, 2026 •

Copy link
Copy Markdown
Collaborator

This pull request updates the RPent documentation, configuration, and dependency management to reflect the migration to RLinf's main branch and unified model loader, and improves configuration clarity for Franka robot setups. The changes also clean up references to legacy RLinf branches and backends, and update code to match the new RLinf structure.

Verified working end-to-end on Libero-pro.

Signed-off-by: Jiaxing Qiu <jx.qiu@outlook.com>
@jx-qiu jx-qiu changed the title refactor(rlinf): port robots to latest upstream APIs refactor(robots): port robots to latest upstream RLinf APIs Sep 23, 2026
@read-the-docs-community

read-the-docs-community Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

@read-the-docs-community

read-the-docs-community Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

Signed-off-by: Jiaxing Qiu <jx.qiu@outlook.com>
Signed-off-by: Jiaxing Qiu <jx.qiu@outlook.com>
… of direct git reference

Signed-off-by: Jiaxing Qiu <jx.qiu@outlook.com>
@jx-qiu
jx-qiu marked this pull request as ready for review September 23, 2026 08:22
Copilot AI lite review requested due to automatic review settings September 23, 2026 08:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Comment thread pyproject.toml
Comment on lines 107 to +108
franka = [
"rlinf[franka] @ git+https://github.com/RLinf/RLinf.git@bde6c918642abf9a4776cb1d5fabcc5087dfe195",
"rpent[rlinf]",

@wilburx813 wilburx813 Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will replacing rlinf[franka] with rpent[rlinf] drop franka-specific dependency?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. Franka dependencies are complex and even with rpent[rlinf], some dependencies still require manual install. So me and @qurakchin decided to (1) let users clone RLinf and install franka deps using RLinf's requirement/install.sh, and (2) install RPent's additional deps on top of the venv created by RLinf. Let me update the documentation to reflect this workflow.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After syncing with @andylin-hao I got that upstream RLinf switched from franka-ros to Franky, which makes dep installation much easier (and almost fully pyproject-compliant). Let me update the code and docs to reflect this change.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Implemented in 2511e83.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants