New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Bert Example: Add needed vmap to embeddings #608

Merged

patrick-kidger merged 1 commit into patrick-kidger:main from haydn-jones:main

Nov 30, 2023

Contributor

haydn-jones commented Nov 30, 2023

Running the Bert example fails with

ValueError: `eqx.nn.Embedding()(x)` should be called with a scalar index `x`. Use `jax.vmap` if you would like to index with multiple values.

just added the vmap in.


          Add needed vmap to embeddings

707bfc0

patrick-kidger merged commit ca995ee into patrick-kidger:main

2 checks passed

Owner

patrick-kidger commented Nov 30, 2023 •

edited

Loading

LGTM! Merged - thank you for the fix. :)

(This was changed to be more strict a few releases ago.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet