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
reformatting task coding challenge #9404
Conversation
added pre-commit hook
removing linting erros
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
Thanks for contributing to ivy😊
An overview of this task is that these functions won't need to have array and container methods so you can remove them.
And as the functions are private, this means they won't be rendered in docs we can write simple docstring explaining functions with no need for putting examples.
Thanks🙂
removed array and container implementations
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
All looks good so far just a few changes to be made😊
Thanks
removed decorator
Thank you for your patience. I have removed the decorator. Please review the work at your convenience. |
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
Check below comment🙂
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!
Thanks for contributing😊
Please consider this my PR for the task 'vairable_data' in the 'gradients' submodule.
The task number #9283 and the ToDo list is #776
close #9283