-
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
feat: Add frontend jax.numpy.fft.ifftn #28550
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.
PR Compliance Checks Passed!
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.
Hi @Medo072
Thank you very much for the PR.
Currently the test_jax_numpy_ifftn seems to fail with torch and numpy backends. You can check the ci logs here https://github.com/unifyai/ivy/actions/runs/8235689979/job/22521391264?pr=28550 under combined test result by searching for test_jax_numpy_ifftn
. Alternatively you can run the tests locally with --backend torch
and --num-examples 200
to catch the failures. Hope that helps. Let me know if something is unclear. :)
there was an issue in the _x_and_ifftn function in test_layers.py as Jax only accept axes of max size = 3 but the function doesn't handle this case, so, I added a new function to handle this case in test_layers.py. |
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.
looks good and passing the test. Thank you very much @Medo072 for the effort put in this contribution :)
PR Description
Related Issue
Closes #28549
Checklist
Socials