-
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
add reciprocal inplace function in paddle and passed all tests #22824
Conversation
Thanks for contributing to Ivy! 😊👏 |
|
@rishabgit It passes lint check check too. please do take a look so that we can merge it. |
@rishabgit please review as It is ready to merge from my side. please do let me know if I need to change anything. |
Hi, your comment syntax wasn't right in the ToDo board and was picked up someone else before, but they hadn't started a PR yet, so I closed their issue and assigned your PR manually. Please use the correct syntax as mentioned in the docs in the future 😅 Your PR looks good to me otherwise! Thanks for adding this 😄 |
@rishabgit Thanks for merging the PR. Could you help me with this linking of the Issues in the To-do list? I have all three methods: - [ ] #Issue_number, #12345, and this one #12345. Neither of these links the issue. Previously I have linked with this method - [ ] #Issue_number and this works fine all the time. I also linked this one using this method, but I don't why it hasn't been linked. Please do let me know If I have any mistakes while linking so that I improve it and avoid problems. |
Close #22818
PR Description
Related Issue
Close #
Checklist
Socials: