Skip to content
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

Fix places that were passing jnp.array where a shape was expected. #1258

Merged
merged 1 commit into from
Apr 22, 2021
Merged

Fix places that were passing jnp.array where a shape was expected. #1258

merged 1 commit into from
Apr 22, 2021

Conversation

pschuh
Copy link
Contributor

@pschuh pschuh commented Apr 20, 2021

For reference: jax-ml/jax#6400

@google-cla google-cla bot added the cla: yes label Apr 20, 2021
@andsteing andsteing requested a review from levskaya April 20, 2021 05:44
@andsteing andsteing added the Priority: P2 - no schedule Best effort response and resolution. We have no plan to work on this at the moment. label Apr 20, 2021
levskaya
levskaya approved these changes Apr 20, 2021
View reviewed changes
Copy link
Collaborator

@levskaya levskaya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@copybara-service copybara-service bot merged commit 279f80b into google:master Apr 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@levskaya levskaya levskaya approved these changes

Assignees

@levskaya levskaya

Labels
cla: yes Priority: P2 - no schedule Best effort response and resolution. We have no plan to work on this at the moment. pull ready
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pschuh @levskaya @jheek @andsteing