Skip to content

PYLINT.md

Latest commit

 

History

History
162 lines (139 loc) · 9.72 KB

PYLINT.md

File metadata and controls

162 lines (139 loc) · 9.72 KB

Pylint Report

Before Fixing Issues

Module: main

Unfortunatly we first did the pylint report and then the other things like restructuring the code. This way, a lot of improvements are not visible anymore. But we solved 28 pylint issues. The changes were made by seva, so one can look up the code after a commit from seva. We didnt change much in DataFrameAnalyzer, so the changes we made there (with pylint) are still present. So we hope this is enough :)

  • C0301: Line too long (107/100) (line-too-long)
  • C0305: Trailing newlines (trailing-newlines) (fixed ✅)
  • C0114: Missing module docstring (missing-module-docstring) (fixed ✅)
  • C0103: Constant name "file_path" doesn't conform to UPPER_CASE naming style (invalid-name)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • W0621: Redefining name 'file_path' from outer scope (line 5) (redefined-outer-name)
  • W0621: Redefining name 'df' from outer scope (line 54) (redefined-outer-name)
  • C0103: Variable name "df" doesn't conform to snake_case naming style (invalid-name)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • C0103: Argument name "df" doesn't conform to snake_case naming style (invalid-name)
  • W0621: Redefining name 'df' from outer scope (line 54) (redefined-outer-name)
  • W0621: Redefining name 'df_modified' from outer scope (line 55) (redefined-outer-name)
  • W0611: Unused numpy imported as np (unused-import) (fixed ✅)

Your code has been rated at 6.29/10

Module: DataframeAnalyzer

  • C0301: Line too long (118/100) (line-too-long)
  • C0301: Line too long (180/100) (line-too-long)
  • C0301: Line too long (127/100) (line-too-long)
  • C0301: Line too long (120/100) (line-too-long)
  • C0301: Line too long (106/100) (line-too-long)
  • C0303: Trailing whitespace (trailing-whitespace) (fixed ✅)
  • C0301: Line too long (134/100) (line-too-long)
  • C0303: Trailing whitespace (trailing-whitespace) (fixed ✅)
  • C0301: Line too long (102/100) (line-too-long)
  • C0301: Line too long (104/100) (line-too-long)
  • C0301: Line too long (109/100) (line-too-long)
  • C0301: Line too long (101/100) (line-too-long)
  • C0301: Line too long (132/100) (line-too-long)
  • C0301: Line too long (118/100) (line-too-long)
  • C0301: Line too long (108/100) (line-too-long)
  • C0301: Line too long (120/100) (line-too-long)
  • C0301: Line too long (118/100) (line-too-long)
  • C0301: Line too long (124/100) (line-too-long)
  • C0303: Trailing whitespace (trailing-whitespace) (fixed ✅)
  • C0303: Trailing whitespace (trailing-whitespace) (fixed ✅)
  • C0305: Trailing newlines (trailing-newlines) (fixed ✅)
  • C0114: Missing module docstring (missing-module-docstring) (fixed ✅)
  • C0103: Module name "DataframeAnalyzer" doesn't conform to snake_case naming style (invalid-name)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • C0103: Argument name "df" doesn't conform to snake_case naming style (invalid-name)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • C0103: Argument name "df" doesn't conform to snake_case naming style (invalid-name)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • C0103: Variable name "df" doesn't conform to snake_case naming style (invalid-name)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • C0103: Variable name "df" doesn't conform to snake_case naming style (invalid-name)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • W0611: Unused pandas imported as pd (unused-import)

Your code has been rated at 3.51/10

Module: UserInteractWithDataset

  • C0303: Trailing whitespace (trailing-whitespace) (fixed ✅)
  • C0301: Line too long (115/100) (line-too-long)
  • C0301: Line too long (103/100) (line-too-long)
  • C0301: Line too long (117/100) (line-too-long)
  • C0301: Line too long (109/100) (line-too-long)
  • C0301: Line too long (111/100) (line-too-long)
  • C0114: Missing module docstring (missing-module-docstring) (fixed ✅)
  • C0103: Module name "UserInteractWithDataset" doesn't conform to snake_case naming style (invalid-name)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • C0103: Variable name "df" doesn't conform to snake_case naming style (invalid-name)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • C0103: Variable name "df" doesn't conform to snake_case naming style (invalid-name)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • R0914: Too many local variables (19/15) (too-many-locals)
  • C0103: Variable name "dataframeAnalyzer" doesn't conform to snake_case naming style (invalid-name)
  • R1724: Unnecessary "else" after "continue", remove the "else" and de-indent the code inside it (no-else-continue)
  • C0103: Variable name "df" doesn't conform to snake_case naming style (invalid-name)
  • R1724: Unnecessary "else" after "continue", remove the "else" and de-indent the code inside it (no-else-continue)
  • C0103: Variable name "df" doesn't conform to snake_case naming style (invalid-name)
  • R0912: Too many branches (24/12) (too-many-branches)
  • R0915: Too many statements (82/50) (too-many-statements)
  • W0612: Unused variable 'dataframe_csv' (unused-variable)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)
  • W0622: Redefining built-in 'list' (redefined-builtin) (fixed ✅)
  • C0116: Missing function or method docstring (missing-function-docstring) (fixed ✅)

Your code has been rated at 7.42/10

After Fixing Many Issues

Module: main

  • C0301: Line too long (111/100) (line-too-long)
  • C0103: Constant name "file_path" doesn't conform to UPPER_CASE naming style (invalid-name)
  • W0621: Redefining name 'file_path' from outer scope (line 8) (redefined-outer-name)
  • W0621: Redefining name 'df' from outer scope (line 66) (redefined-outer-name)
  • C0103: Variable name "df" doesn't conform to snake_case naming style (invalid-name)
  • C0103: Argument name "df" doesn't conform to snake_case naming style (invalid-name)
  • W0621: Redefining name 'df' from outer scope (line 66) (redefined-outer-name)
  • W0621: Redefining name 'df_modified' from outer scope (line 69) (redefined-outer-name)

Your code has been rated at 7.65/10 (previous run: 7.35/10, +0.29)

Module: DataframeAnalyzer

  • C0301: Line too long (116/100) (line-too-long)
  • C0301: Line too long (147/100) (line-too-long)
  • C0301: Line too long (137/100) (line-too-long)
  • C0301: Line too long (134/100) (line-too-long)
  • C0301: Line too long (101/100) (line-too-long)
  • C0301: Line too long (115/100) (line-too-long)
  • C0301: Line too long (101/100) (line-too-long)
  • C0301: Line too long (107/100) (line-too-long)
  • C0301: Line too long (115/100) (line-too-long)
  • C0301: Line too long (120/100) (line-too-long)
  • C0301: Line too long (115/100) (line-too-long)
  • C0103: Module name "DataframeAnalyzer" doesn't conform to snake_case naming style (invalid-name)
  • C0103: Argument name "df" doesn't conform to snake_case naming style (invalid-name)
  • C0103: Argument name "df" doesn't conform to snake_case naming style (invalid-name)
  • C0103: Variable name "df" doesn't conform to snake_case naming style (invalid-name)
  • C0103: Variable name "df" doesn't conform to snake_case naming style (invalid-name)
  • W0611: Unused pandas imported as pd (unused-import)

Your code has been rated at 7.02/10 (previous run: 4.29/10, +2.73)

Module: UserInteractWithDataset

  • C0301: Line too long (115/100) (line-too-long)
  • C0301: Line too long (101/100) (line-too-long)
  • C0301: Line too long (117/100) (line-too-long)
  • C0301: Line too long (109/100) (line-too-long)
  • C0301: Line too long (111/100) (line-too-long)
  • C0103: Module name "UserInteractWithDataset" doesn't conform to snake_case naming style (invalid-name)
  • C0103: Variable name "df" doesn't conform to snake_case naming style (invalid-name)
  • C0103: Variable name "df" doesn't conform to snake_case naming style (invalid-name)
  • R0914: Too many local variables (19/15) (too-many-locals)
  • C0103: Variable name "dataframeAnalyzer" doesn't conform to snake_case naming style (invalid-name)
  • R1724: Unnecessary "else" after "continue", remove the "else" and de-indent the code inside it (no-else-continue)
  • C0103: Variable name "df" doesn't conform to snake_case naming style (invalid-name)
  • R1724: Unnecessary "else" after "continue", remove the "else" and de-indent the code inside it (no-else-continue)
  • C0103: Variable name "df" doesn't conform to snake_case naming style (invalid-name)
  • R0912: Too many branches (24/12) (too-many-branches)
  • R0915: Too many statements (82/50) (too-many-statements)
  • W0612: Unused variable 'dataframe_csv' (unused-variable)

Your code has been rated at 8.25/10 (previous run: 7.42/10, +0.82)

In total, I fixed 28 issues.

I didn't fully understand the requirement of adding the code. However, the modified code is in our src file. It would take up too much space to add the code here.