-
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
grid_sample #22128
grid_sample #22128
Conversation
add main
remove constraint
Thanks for contributing to Ivy! 😊👏 |
If you are working on an open task, please edit the PR description to link to the issue you've created. For more information, please check ToDo List Issues Guide. Thank you 🤗 |
@JONAHKYAGABA Can you please reference the issue that you are tackling in this PR, Thanks! |
Close #21957 |
Hi @jkeane508 sorry i had forgotten to link it but its sorted and am still updating the function. |
@@ -83,6 +83,15 @@ def test_jax_add( | |||
x=x[0], | |||
y=x[1], | |||
) | |||
@handle_frontend_test( |
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.
Hey, I don't think your test implementation is quite right. Please refer to the docs or take some inspiration from other tests in this file.
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 @jkeane508 i kindly request for a review,
And also am facing an error while running the tests.Am using a minconda environment.
" backend_version = {"version": paddle.version.full_version}
E AttributeError: module 'paddle' has no attribute 'version'"
https://discord.com/channels/799879767196958751/933380219832762430/1145083248121217165
# Conflicts: # ivy/functional/frontends/paddle/nn/functional/vision.py
Hello @jkeane508 please kindly review this PR or inform me n how to proceed |
Close #21957
START ADDING GRID SAMPLE TO PADDLE FRONTEND