Skip to content
Code Quality (Tests, Linting, Coverage)

#446: simplified subclustering and further improved instrumentation #618

Sign in to view logs

#446: simplified subclustering and further improved instrumentation

#446: simplified subclustering and further improved instrumentation #618

Triggered via push October 18, 2023 13:55
@ppebayppebay
pushed cd7599a
446-resolve-with-subclustering-improvements
Status Failure
Total duration 4m 17s
Artifacts –
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

code-quality.yml

on: push
Matrix: code-quality
Fit to window
Zoom out
Zoom in

Annotations

2 errors, 10 warnings, and 10 notices
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsStatistics.py#L368
Undefined variable 'key_tuples' (undefined-variable)
code-quality (ubuntu-latest, 3.8)
Process completed with exit code 255.
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsCentralizedPrefixOptimizerAlgorithm.py#L152
Lambda may not be necessary (unnecessary-lambda)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsCriterionBase.py#L66
Dangerous default value [] as argument (dangerous-default-value)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsCriterionBase.py#L76
Unnecessary pass statement (unnecessary-pass)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsCriterionBase.py#L79
Dangerous default value [] as argument (dangerous-default-value)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsClusteringTransferStrategy.py#L99
Parameter 'phase' has been renamed to 'known_peers' in overridden 'ClusteringTransferStrategy.execute' method (arguments-renamed)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsClusteringTransferStrategy.py#L99
Parameter 'known_peers' has been renamed to 'phase' in overridden 'ClusteringTransferStrategy.execute' method (arguments-renamed)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsClusteringTransferStrategy.py#L180
Redeclared variable 'r_dst' in assignment (redeclared-assigned-name)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsRecursiveTransferStrategy.py#L63
Parameter 'phase' has been renamed to 'known_peers' in overridden 'RecursiveTransferStrategy.execute' method (arguments-renamed)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsRecursiveTransferStrategy.py#L63
Parameter 'known_peers' has been renamed to 'phase' in overridden 'RecursiveTransferStrategy.execute' method (arguments-renamed)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsAlgorithmBase.py#L100
Unreachable code (unreachable)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataWriter.py#L1
Missing module docstring (missing-module-docstring)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataWriter.py#L50
Method could be a function (no-self-use)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataWriter.py#L113
Use a generator instead 'all(isinstance(p, Phase) for p in phases.values())' (use-a-generator)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataWriter.py#L121
Variable name "p" doesn't conform to snake_case naming style (invalid-name)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataWriter.py#L122
Variable name "r" doesn't conform to snake_case naming style (invalid-name)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataWriter.py#L12
Too few public methods (1/2) (too-few-public-methods)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsConfigurationValidator.py#L56
Use a generator instead 'all(isinstance(y, float) for y in x.values())' (use-a-generator)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsConfigurationValidator.py#L100
Use a generator instead 'all(isinstance(y, int) for y in x)' (use-a-generator)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsConfigurationValidator.py#L203
Unnecessary "else" after "return" (no-else-return)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsConfigurationValidator.py#L212
Too many branches (13/12) (too-many-branches)