Skip to content

Remove conflicting tree dependency from openpi-client#937

Merged
jimmyt857 merged 1 commit into
Physical-Intelligence:mainfrom
Hebbian-Robotics:fix-openpi-client-tree-dependency
May 5, 2026
Merged

Remove conflicting tree dependency from openpi-client#937
jimmyt857 merged 1 commit into
Physical-Intelligence:mainfrom
Hebbian-Robotics:fix-openpi-client-tree-dependency

Conversation

@kstonekuan

Copy link
Copy Markdown
Contributor

Summary

This removes the conflicting tree>=0.2.4 dependency from packages/openpi-client.

Why

openpi-client imports tree for tree.map_structure, but that import is provided by dm-tree, which is already declared as a dependency.

The separate PyPI package named tree can conflict with dm-tree because both use the tree import name. This was causing openpi-client imports to resolve to the wrong package depending on install order/environment.

Changes

  • remove tree>=0.2.4 from packages/openpi-client/pyproject.toml
  • update uv.lock (removes the transitive svgwrite lock entry)

Validation

  • ran uv lock --check
  • ran uv run --project packages/openpi-client --with pytest pytest packages/openpi-client
  • ran uv run --no-sync ruff check --fix packages/openpi-client
  • ran uv run --no-sync ruff format packages/openpi-client

Notes

This keeps dm-tree unchanged because it is the package that provides the tree.map_structure API used by openpi-client.

jimmyt857 commented May 5, 2026

Copy link
Copy Markdown
Contributor

Merge activity

  • May 5, 4:07 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 5, 4:07 PM UTC: Graphite couldn't merge this PR because it failed for an unknown reason (Fast-forward merges are not supported for forked repositories. Please create a branch in the target repository in order to merge).

@jimmyt857
jimmyt857 merged commit c23745b into Physical-Intelligence:main May 5, 2026
2 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.

2 participants