-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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.
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
added jax numpy logic_or #10316
added jax numpy logic_or #10316
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @xc720! Can you please check my requested changes and apply them before we can start checking the implementaion? Thanks 🙂
ivy_tests/test_ivy/test_frontends/test_jax/test_jax_numpy_logic.py
Outdated
Show resolved
Hide resolved
ivy_tests/test_ivy/test_frontends/test_jax/test_jax_numpy_logic.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @xc720! logical_not is already implemented and added to the repository. Your issue tracks the logical_or implemmentation. Can you remove your function and implement logical_or as suggested by your initial issue?
Your tracked issue #10312 references implementation of logical_or |
Yes, I don't know why it is changed somehow, I will implement it real quick |
Hello, I think now it is implemented, please have a look. |
Looking good, can you finally run the test function locally and post the result here? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
added jax numpy logic_or
Closes #10312