-
Notifications
You must be signed in to change notification settings - Fork 217
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
start to use Perflint to improved performance #1063
base: main
Are you sure you want to change the base?
Conversation
ce4c579
to
559a08d
Compare
559a08d
to
cce73da
Compare
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.
TL;DR: Partially useful changes
After looking at all your changes, I conclude that they don't affect performance much unless you're chasing nanoseconds. In general, of course, some changes should really be made to the code base. And some that might reduce execution time by n amount of time are not worth it, as they make the code heavier to read. I would consider to commit partial changes.
IMHO
like if else expression ? making it more difficult to read ? |
Not only statements, complicated list/dict comprehension. I didn't say that all changes are useless or over complicated due to reduce execution time, I fully agree with you that some solutions in the current code have some flaws that you want to correct. Don't take it as disrespect 😁 |
Yeah dont worry, its hard to determine a balance between readability and improving |
added option to ruff to guide developers to use more performant constructs in python