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

fixing the jax.nn.selu frontend test #9521

Merged
merged 1 commit into from
Jan 10, 2023
Merged

fixing the jax.nn.selu frontend test #9521

merged 1 commit into from
Jan 10, 2023

Conversation

hello-fri-end
Copy link
Contributor

closes #8695 .

@ivy-leaves ivy-leaves added the JAX Frontend Developing the JAX Frontend, checklist triggered by commenting add_frontend_checklist label Jan 9, 2023
iamjameskeane
iamjameskeane approved these changes Jan 10, 2023
View reviewed changes
Copy link
Contributor

@iamjameskeane iamjameskeane left a comment

Choose a reason for hiding this comment

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

LGTM :)

@iamjameskeane iamjameskeane merged commit 14de306 into ivy-llc:master Jan 10, 2023
NouranYoussef pushed a commit to NouranYoussef/ivy that referenced this pull request Feb 26, 2023
@hello-fri-end @iamjameskeane
jax.nn.selu: only allow numeric datatypes (ivy-llc#9521)
712cf23 
Co-authored-by: jkeane508 <[email protected]>
@hello-fri-end hello-fri-end deleted the fixJAXselu branch March 15, 2023 03:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iamjameskeane iamjameskeane iamjameskeane approved these changes

Assignees

@iamjameskeane iamjameskeane

Labels
JAX Frontend Developing the JAX Frontend, checklist triggered by commenting add_frontend_checklist
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix jax_nn_activations.test_jax_nn_selu
3 participants
@hello-fri-end @iamjameskeane @ivy-leaves