Skip to content

Commit

Permalink
Replace uses of jnp.array in types with jnp.ndarray.
Browse files Browse the repository at this point in the history
`jnp.array` is a function, not a type:
https://jax.readthedocs.io/en/latest/_autosummary/jax.numpy.array.html
so it never makes sense to use `jnp.array` in a type annotation. Presumably the intent was to write `jnp.ndarray` aka `jax.Array`.

PiperOrigin-RevId: 559400993
Change-Id: Ib2629da9b8794131f0dff1f3e149ba4896bea527
  • Loading branch information
hawkinsp authored and copybara-github committed Aug 23, 2023
1 parent f61a18b commit 8bcdcae
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lightweight_mmm/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -290,8 +290,8 @@ def media_mix_model(
custom_priors: MutableMapping[str, Prior],
transform_kwargs: Optional[MutableMapping[str, Any]] = None,
weekday_seasonality: bool = False,
extra_features: Optional[jnp.array] = None
) -> None:
extra_features: Optional[jnp.ndarray] = None,
) -> None:
"""Media mix model.
Args:
Expand Down

0 comments on commit 8bcdcae

Please sign in to comment.