Skip to content

fix(franka): constrain PyArrow for NumPy 1 compatibility (v0.1) - #278

Merged
wilburx813 merged 3 commits into
RLinf:mainfrom
jx-qiu:nieyi/fix-franka-pyarrow
Oct 14, 2026
Merged

wilburx813 merged 3 commits into
RLinf:mainfrom
jx-qiu:nieyi/fix-franka-pyarrow

Conversation

@NewYork424

@NewYork424 NewYork424 commented Oct 13, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Constrain the Franka extra to pyarrow<26.0.0, matching its existing numpy<2 requirement. PyArrow 26 drops NumPy 1.x support; importing it caused Ray initialization and the dual-Franka environment server to fail.

The upper bound excludes this confirmed incompatibility without pinning all installations to one patch release. Other dependencies still determine applicable lower bounds; older PyArrow releases have not all been tested.

Validation

  • Isolated Python 3.12: NumPy 1.26.4 + PyArrow 25.0.1 import and NumPy-to-Arrow conversion passed.
  • Parsed the requirement and verified that 25.0.1 is allowed and 26.0.0 excluded; git diff --check passed.
  • The operator confirmed successful two-node deployment and Task 6 execution with NumPy 1.26.4 + PyArrow 25.0.1. No new hardware or planner run was performed for this update.

Only the Franka dependency constraint is changed. Full unit tests and pre-commit were not run locally; there are no Python implementation changes.

Copilot AI balanced review requested due to automatic review settings October 13, 2026 16:41

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.

@wilburx813

Copy link
Copy Markdown
Collaborator

Hi @NewYork424 ,please try using something like pyarrow>10,<26, instead of pinning to a single version

@wilburx813 wilburx813 changed the title fix(franka): pin PyArrow for NumPy 1 compatibility fix(franka): pin PyArrow for NumPy 1 compatibility(v0.1) Oct 14, 2026
@panhe1818
panhe1818 requested a review from Sonorous281 October 14, 2026 07:11
@read-the-docs-community

read-the-docs-community Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Documentation build overview

📚 RPent | 🛠️ Build #35136075 | 📁 Comparing 0a55157 against latest (ef70215)

  🔍 Preview build  

3 files changed
± rst_source/guides/cli.html
± rst_source/guides/memory.html
± rst_source/simulators/robocasa.html

@read-the-docs-community

read-the-docs-community Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Documentation build overview

📚 RPent-cn | 🛠️ Build #35136076 | 📁 Comparing 0a55157 against latest (ef70215)

  🔍 Preview build  

3 files changed
± rst_source/guides/cli.html
± rst_source/guides/memory.html
± rst_source/simulators/robocasa.html

@NewYork424 NewYork424 changed the title fix(franka): pin PyArrow for NumPy 1 compatibility(v0.1) fix(franka): constrain PyArrow for NumPy 1 compatibility (v0.1) Oct 14, 2026
@wilburx813
wilburx813 merged commit a8485e2 into RLinf:main Oct 14, 2026
7 checks passed
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.

5 participants