-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
adding roots function for jax frontend #23481
Conversation
Thanks for contributing to Ivy! 😊👏 |
@Ookamice please help me understand this error. Because when I use some random inputs to this function to verify it with jax.roots() it provides the the same results, fun_name = 'roots', args = ([Array([-1., -1., -1., -1., -1.], dtype=float32)],)
E TypeError: roots requires ndarray or scalar arguments, got <class 'list'> at position 0. /opt/fw/jax/jax/_src/numpy/util.py:328: TypeError |
ivy_tests/test_ivy/test_frontends/test_jax/test_numpy/test_mathematical_functions.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.
PR Compliance Checks
Thank you for your Pull Request! We have run several checks on this pull request in order to make sure it's suitable for merging into this project. The results are listed in the following section.
Conventional Commit PR Title
In order to be considered for merging, the pull request title must match the specification in conventional commits. You can edit the title in order for this check to pass.
Most often, our PR titles are something like one of these:
- docs: correct typo in README
- feat: implement dark mode"
- fix: correct remove button behavior
Linting Errors
- Found type "null", must be one of "feat","fix","docs","style","refactor","perf","test","build","ci","chore","revert"
- No subject found
How can I work with dtype errors. During all the tests I am getting the right answers but not the right dtypes. When ground truth is |
Hello! @ZoeCD all the tests are passing in my system. Could you please help me understand what shall I do next. |
Hello @DecFox Please help me in merging this PR all the tests are passing from my end |
Hello @hirwa-nshuti could you please help me merge this function it has been here since a month. |
atol=1e-05, | ||
rtol=1e-03, | ||
) | ||
ret, frontend_ret = call() |
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.
unless test_values=False
, the test_frontend_function doesn't return the values. In addition this call()
should be made outside the the definition of call
. Tbh we don't even need to define it as a call. But thats fine.
Once this is updated pls request my review. Thnx
This PR has been labelled as stale because it has been inactive for more than 7 days. If you would like to continue working on this PR, then please add another comment or this PR will be closed in 7 days. |
This PR has been closed because it has been marked as stale for more than 7 days with no activity. |
PR Description
Related Issue
Close #23179
Checklist
Socials: