Skip to content
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

Reformat_lines #12990

Closed
kisan121 opened this issue Mar 23, 2023 · 0 comments
Closed

Reformat_lines #12990

kisan121 opened this issue Mar 23, 2023 · 0 comments
Labels
Sub Task a sub task which is stemming from a ToDo list issue

Comments

@kisan121
Copy link

As explained in the function reformatting open task, each function in the layers submodule should be updated to follow the implementation instructions given in the Deep Dive section of the docs. The updates should be applied for both the Ivy API and Backend API. The Deep Dive is the best general resource for learning in detail how functions should be implemented. Many of the functions are not implemented as they should be. You will need to make changes to the current implementations, but you do not need to go through all section of the Deep Dive. Specifically, you should make the following changes, where appropriate:

remove all lambda and direct bindings for backend functions, with each function instead defined using def.
update the function arguments and type hints.
add the correct docstrings
add thorough docstring examples
_

Please keep in mind that the proper way to link an issue to this list is to comment "- [ ] #issue_number" while the issue's title only includes the name of the function you've chosen.

_

#4769
#8980
#2073
#9639
#2103
#1116
#975
#9635
#1804
#7390
#1793
#9320

@kisan121 kisan121 added the Sub Task a sub task which is stemming from a ToDo list issue label Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Sub Task a sub task which is stemming from a ToDo list issue
Projects
None yet
Development

No branches or pull requests

2 participants