[Code health] Remove Re-definition found for builtin function
See original GitHub issue🚀 Feature
As reported by deepsource in here we abuse from using built-in input
function in our functionality.
Motivation
We target to have a clean and healthy source code free of risk.
Pitch
Replace variable names whether it makes sense e.g. for image based functionality input
-> image
; in losses input
-> pred
(or similar)
Consider also to contribute to Kornia universe projects 😃
- Tutorials: our repository containing the tutorials.
Issue Analytics
- State:
- Created 2 years ago
- Comments:5 (3 by maintainers)
Top Results From Across the Web
python: builtin function redefinition, with different arguments
To replace the 'print' builtin, I can write my python function. python 2.6 : at runtime, I get syntax error. solution from __future__...
Read more >Redefinition of function - ROOT Forum
Hello, Is there any way to redefine previously defined function in ROOT environment? Let's say I input function : root[0].
Read more >GNU M4 1.4.19 macro processor
5.1 Defining a macro. The normal way to define or redefine macros is to use the builtin define : Builtin: define ( name...
Read more >Code-Dependent: Pros and Cons of the Algorithm Age
The non-scientific canvassing found that 38% of these particular ... Our algorithms are now redefining what we think, how we think and what ......
Read more >How do you fix a redefinition error in C programming? - Quora
you do that by removing the redefinition. ... cases in C. I dislike this approach, as complex functions can end up with dozens...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
I do agree with the tool… It is not considered Pythonic to shadow a builtin. It does not matter if the current code does not use
input()
because a future contributor might add it and then be highly confused by the results.@sesam I’d go module by module and possibly starting with test files