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.

Sign up for GitHub

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

Jump to bottom

PEP8 formatting #309

Open

Zhigal wants to merge 1 commit into datamllab:master from Zhigal:fix-formatting-and-other-issues

Contributor

Zhigal commented Jan 26, 2024

Sorry for a HUGE pr...

In this PR:

PEP8 formatting applied:
- docstring converted to """ from '''
- emply lines added where needed
- self replaced with cls for class methods
proofreading
mutable arguments for some methods were replaced with immutable
unused imports removed
None comparison replaces with is None check
unnecessary parenthesis removed
array initialization optimized for this cases:

arr = []
arr.append(...)

The logic is still the same, no changes here

Zhigal force-pushed the fix-formatting-and-other-issues branch from 5d674c9 to d7ff690 Compare

January 26, 2024 12:34


          formatting applied

d48a2d4

Zhigal force-pushed the fix-formatting-and-other-issues branch from d7ff690 to d48a2d4 Compare

January 26, 2024 12:39

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment