google-research / jax3d

Apache License 2.0
729 stars 94 forks source link

[LSC] change uses of jax.random.KeyArray and jax.random.PRNGKeyArray to jax.Array #193

Closed copybara-service[bot] closed 10 months ago

copybara-service[bot] commented 10 months ago

[LSC] change uses of jax.random.KeyArray and jax.random.PRNGKeyArray to jax.Array

This change replaces uses of jax.random.KeyArray and jax.random.PRNGKeyArray in the context of type annotations with jax.Array (or chex.PRNGKey where chex is used). These are the correct annotation for JAX PRNG keys moving forward.

The purpose of this change is to remove references to KeyArray and PRNGKeyArray, which are deprecated (https://github.com/google/jax/pull/17594) and will soon be removed from JAX. The design and thought process behind this is described in https://jax.readthedocs.io/en/latest/jep/9263-typed-keys.html.

Note that KeyArray and PRNGKeyArray have always been aliased to Any, so the new type annotation is far more specific than the old one.