Skip to content

Add chex as a runtime dependency#924

Merged
jimmyt857 merged 3 commits intoPhysical-Intelligence:mainfrom
kstonekuan:fix/add-chex-runtime-dependency
Apr 15, 2026
Merged

Add chex as a runtime dependency#924
jimmyt857 merged 3 commits intoPhysical-Intelligence:mainfrom
kstonekuan:fix/add-chex-runtime-dependency

Conversation

@kstonekuan
Copy link
Copy Markdown
Contributor

Summary

This adds chex==0.1.90 to openpi runtime dependencies.

Why

openpi.models.utils.fsq_tokenizer imports chex at runtime, but chex is not currently declared in pyproject.toml.

In a production inference environment, this caused imports to fail with:

ModuleNotFoundError: No module named 'chex'

Changes

  • add chex==0.1.90 to pyproject.toml

Notes

I used 0.1.90 because it is compatible with the current openpi JAX line (jax==0.5.3).

The newer chex>=0.1.91 release requires jax>=0.7.0, so leaving chex unconstrained would risk pulling in an incompatible JAX stack.

@kstonekuan kstonekuan marked this pull request as draft April 15, 2026 07:21
@kstonekuan kstonekuan marked this pull request as ready for review April 15, 2026 08:07
@kstonekuan
Copy link
Copy Markdown
Contributor Author

Hey @jimmyt857 , struggled with the CI due to the uv lockfile for a bit but this is ready for review now, thanks!

Copy link
Copy Markdown
Contributor

jimmyt857 commented Apr 15, 2026

Merge activity

  • Apr 15, 3:16 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Apr 15, 3:16 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 a190e00 into Physical-Intelligence:main Apr 15, 2026
2 checks passed
@kstonekuan kstonekuan deleted the fix/add-chex-runtime-dependency branch April 15, 2026 15:29
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