Closed zbeekman closed 2 years ago
Merging #404 (22c9357) into master (b6540f2) will increase coverage by
0.84%
. The diff coverage is70.08%
.
@@ Coverage Diff @@
## master #404 +/- ##
==========================================
+ Coverage 68.38% 69.23% +0.84%
==========================================
Files 92 98 +6
Lines 7992 8619 +627
Branches 1374 1692 +318
==========================================
+ Hits 5465 5967 +502
- Misses 2030 2138 +108
- Partials 497 514 +17
Flag | Coverage Δ | |
---|---|---|
CI | 69.23% <70.08%> (+0.84%) |
:arrow_up: |
unittests | 69.23% <70.08%> (+0.84%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
packages/taucmdr/cf/__init__.py | 100.00% <ø> (ø) |
|
packages/taucmdr/cf/compiler/caf.py | 100.00% <ø> (ø) |
|
packages/taucmdr/cf/compiler/cuda.py | 100.00% <ø> (ø) |
|
packages/taucmdr/cf/compiler/host.py | 100.00% <ø> (ø) |
|
packages/taucmdr/cf/compiler/mpi.py | 100.00% <ø> (ø) |
|
packages/taucmdr/cf/compiler/python.py | 100.00% <ø> (ø) |
|
packages/taucmdr/cf/compiler/shmem.py | 100.00% <ø> (ø) |
|
packages/taucmdr/cf/platforms.py | 81.37% <0.00%> (ø) |
|
...kages/taucmdr/cli/commands/application/__init__.py | 100.00% <ø> (ø) |
|
packages/taucmdr/cli/commands/application/copy.py | 100.00% <ø> (ø) |
|
... and 100 more |
8719 statements analysed.
type | number | old number | difference | %documented | %badname |
---|---|---|---|---|---|
module | 99 | NC | NC | 100.00 | 0.00 |
class | 129 | NC | NC | 99.22 | 0.00 |
method | 695 | NC | NC | 98.71 | 0.86 |
function | 114 | NC | NC | 97.37 | 0.88 |
type | number | % | previous | difference |
---|---|---|---|---|
code | 10598 | 51.39 | NC | NC |
docstring | 5389 | 26.13 | NC | NC |
comment | 2945 | 14.28 | NC | NC |
empty | 1690 | 8.20 | NC | NC |
now | previous | difference | |
---|---|---|---|
nb duplicated lines | 0 | NC | NC |
percent duplicated lines | 0.000 | NC | NC |
type | number | previous | difference |
---|---|---|---|
convention | 105 | NC | NC |
refactor | 82 | NC | NC |
warning | 100 | NC | NC |
error | 0 | NC | NC |
message id | occurrences |
---|---|
import-outside-toplevel | 80 |
raise-missing-from | 60 |
no-else-return | 32 |
unspecified-encoding | 12 |
missing-function-docstring | 12 |
no-else-raise | 11 |
consider-using-with | 10 |
inconsistent-return-statements | 8 |
invalid-name | 7 |
arguments-differ | 7 |
useless-super-delegation | 3 |
no-else-continue | 3 |
logging-fstring-interpolation | 3 |
line-too-long | 3 |
fixme | 3 |
duplicate-code | 3 |
consider-using-in | 3 |
use-implicit-booleaness-not-len | 2 |
unused-variable | 2 |
unused-argument | 2 |
too-many-nested-blocks | 2 |
too-many-locals | 2 |
invalid-overridden-method | 2 |
use-list-literal | 1 |
use-dict-literal | 1 |
use-a-generator | 1 |
unused-import | 1 |
too-many-public-methods | 1 |
simplifiable-if-expression | 1 |
redefined-builtin | 1 |
raising-format-tuple | 1 |
no-else-break | 1 |
missing-class-docstring | 1 |
logging-not-lazy | 1 |
consider-using-dict-comprehension | 1 |
condition-evals-to-constant | 1 |
broad-except | 1 |
bad-indentation | 1 |
Your code has been rated at 9.21/10
************* Module taucmdr.error /home/runner/work/taucmdr/taucmdr/packages/taucmdr/error.py:75:4: C0116: Missing function or method docstring (missing-function-docstring) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/error.py:122:20: R1734: Consider using [] instead of list() (use-list-literal) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/error.py:168:8: C0415: Import outside toplevel (taucmdr.cli.commands.project.create.COMMAND) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/error.py:169:8: C0415: Import outside toplevel (taucmdr.cli.commands.project.select.COMMAND) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/error.py:170:8: C0415: Import outside toplevel (taucmdr.cli.commands.project.list.COMMAND) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/error.py:182:8: C0415: Import outside toplevel (taucmdr.cli.commands.select.COMMAND) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/error.py:183:8: C0415: Import outside toplevel (taucmdr.cli.commands.experiment.create.COMMAND) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/error.py:184:8: C0415: Import outside toplevel (taucmdr.cli.commands.dashboard.COMMAND) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/error.py:185:8: C0415: Import outside toplevel (taucmdr.cli.commands.project.list.COMMAND) (import-outside-toplevel) ************* Module taucmdr /home/runner/work/taucmdr/taucmdr/packages/taucmdr/__init__.py:103:4: C0415: Import outside toplevel (platform) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/__init__.py:104:4: C0415: Import outside toplevel (socket) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/__init__.py:105:4: C0415: Import outside toplevel (datetime.datetime) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/__init__.py:106:4: C0415: Import outside toplevel (taucmdr.logger) (import-outside-toplevel) ************* Module taucmdr.progress /home/runner/work/taucmdr/taucmdr/packages/taucmdr/progress.py:45:9: W1514: Using open without explicitly specifying an encoding (unspecified-encoding) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/progress.py:130:8: C0415: Import outside toplevel (taucmdr.util) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/progress.py:142:8: C0415: Import outside toplevel (taucmdr.util) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/progress.py:177:4: C0116: Missing function or method docstring (missing-function-docstring) ************* Module taucmdr.model.compiler /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/compiler.py:204:8: R1720: Unnecessary "else" after "raise" (no-else-raise) ************* Module taucmdr.model.experiment /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:50:4: C0415: Import outside toplevel (taucmdr.model.target.Target) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:51:4: C0415: Import outside toplevel (taucmdr.model.application.Application) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:52:4: C0415: Import outside toplevel (taucmdr.model.measurement.Measurement) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:108:4: W0221: Number of parameters was 2 in 'Controller.one' and is now 3 in overridden 'ExperimentController.one' method (arguments-differ) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:112:4: W0221: Number of parameters was 1 in 'Controller.all' and is now 2 in overridden 'ExperimentController.all' method (arguments-differ) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:116:4: W0221: Number of parameters was 2 in 'Controller.count' and is now 1 in overridden 'ExperimentController.count' method (arguments-differ) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:122:4: W0221: Number of parameters was 2 in 'Controller.search' and is now 3 in overridden 'ExperimentController.search' method (arguments-differ) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:126:4: W0221: Number of parameters was 3 in 'Controller.exists' and is now 2 in overridden 'ExperimentController.exists' method (arguments-differ) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:133:4: W0235: Useless super delegation in method 'create' (useless-super-delegation) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:136:4: W0235: Useless super delegation in method 'update' (useless-super-delegation) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:139:4: W0235: Useless super delegation in method 'unset' (useless-super-delegation) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:142:4: W0221: Number of parameters was 3 in 'Controller.delete' and is now 2 in overridden 'ExperimentController.delete' method (arguments-differ) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:178:8: R1720: Unnecessary "elif" after "raise" (no-else-raise) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:197:12: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:242:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:272:8: C0415: Import outside toplevel (taucmdr.cf.software.tau_installation.TauInstallation) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:433:4: C0116: Missing function or method docstring (missing-function-docstring) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:434:8: C0415: Import outside toplevel (taucmdr.cf.software.tau_installation.TauInstallation) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:440:11: R1714: Consider merging these comparisons with "in" to "rewrite_package in ('maqao', 'pebil')" (consider-using-in) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/experiment.py:532:8: R1705: Unnecessary "else" after "return" (no-else-return) ************* Module taucmdr.logger /home/runner/work/taucmdr/taucmdr/packages/taucmdr/logger.py:112:8: C0415: Import outside toplevel (ctypes.windll, ctypes.create_string_buffer) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/logger.py:120:8: C0415: Import outside toplevel (struct) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/logger.py:138:8: C0415: Import outside toplevel (subprocess) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/logger.py:139:15: R1732: Consider using 'with' for resource-allocating operations (consider-using-with) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/logger.py:143:15: R1732: Consider using 'with' for resource-allocating operations (consider-using-with) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/logger.py:167:12: C0415: Import outside toplevel (fcntl) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/logger.py:168:12: C0415: Import outside toplevel (termios) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/logger.py:169:12: C0415: Import outside toplevel (struct) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/logger.py:272:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) ************* Module taucmdr.model.measurement /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/measurement.py:53:4: C0415: Import outside toplevel (taucmdr.model.project.Project) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/measurement.py:54:4: C0415: Import outside toplevel (taucmdr.model.target.Target) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/measurement.py:55:4: C0415: Import outside toplevel (taucmdr.model.application.Application) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/measurement.py:56:4: C0415: Import outside toplevel (taucmdr.cf.platforms.HOST_OS, taucmdr.cf.platforms.DARWIN, taucmdr.cf.platforms.IBM_CNK) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/measurement.py:57:4: C0415: Import outside toplevel (taucmdr.cf.compiler.mpi.MPI_CC, taucmdr.cf.compiler.mpi.MPI_CXX, taucmdr.cf.compiler.mpi.MPI_FC) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/measurement.py:58:4: C0415: Import outside toplevel (taucmdr.cf.compiler.shmem.SHMEM_CC, taucmdr.cf.compiler.shmem.SHMEM_CXX, taucmdr.cf.compiler.shmem.SHMEM_FC) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/measurement.py:302:44: C1802: Do not use `len(SEQUENCE)` without comparison to determine if a sequence is empty (use-implicit-booleaness-not-len) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/measurement.py:511:8: C0415: Import outside toplevel (taucmdr.error.ImmutableRecordError) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/measurement.py:512:8: C0415: Import outside toplevel (taucmdr.model.experiment.Experiment) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/measurement.py:544:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/measurement.py:556:8: C0415: Import outside toplevel (taucmdr.error.ImmutableRecordError) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/measurement.py:557:8: C0415: Import outside toplevel (taucmdr.model.experiment.Experiment) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/measurement.py:568:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/measurement.py:587:8: C0415: Import outside toplevel (taucmdr.model.project.Project) (import-outside-toplevel) ************* Module taucmdr.model.project /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/project.py:47:4: C0415: Import outside toplevel (taucmdr.model.target.Target) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/project.py:48:4: C0415: Import outside toplevel (taucmdr.model.application.Application) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/project.py:49:4: C0415: Import outside toplevel (taucmdr.model.measurement.Measurement) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/project.py:50:4: C0415: Import outside toplevel (taucmdr.model.experiment.Experiment) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/project.py:97:4: W0221: Number of parameters was 3 in 'Controller.delete' and is now 2 in overridden 'ProjectController.delete' method (arguments-differ) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/project.py:129:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/project.py:142:8: C0415: Import outside toplevel (taucmdr.model.experiment.Experiment) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/project.py:143:8: C0415: Import outside toplevel (taucmdr.model.compiler.Compiler) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/project.py:149:8: R1702: Too many nested blocks (8/5) (too-many-nested-blocks) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/project.py:205:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) ************* Module taucmdr.util /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:80:12: W0622: Redefining built-in 'dir' (redefined-builtin) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:82:12: W0612: Unused variable 'src' (unused-variable) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:110:11: R1732: Consider using 'with' for resource-allocating operations (consider-using-with) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:144:0: R1710: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:269:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:263:49: W0612: Unused variable 'progress_bar' (unused-variable) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:293:19: R1732: Consider using 'with' for resource-allocating operations (consider-using-with) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:331:8: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:329:14: R1732: Consider using 'with' for resource-allocating operations (consider-using-with) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:378:15: C0103: Variable name "v" doesn't conform to '[a-z_][a-z0-9_]{2,30}$' pattern (invalid-name) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:379:12: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:391:12: C0103: Variable name "v" doesn't conform to '[a-z_][a-z0-9_]{2,30}$' pattern (invalid-name) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:497:4: R1705: Unnecessary "else" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:507:21: W1514: Using open without explicitly specifying an encoding (unspecified-encoding) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:507:21: R1732: Consider using 'with' for resource-allocating operations (consider-using-with) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:561:15: R1732: Consider using 'with' for resource-allocating operations (consider-using-with) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:633:15: R1732: Consider using 'with' for resource-allocating operations (consider-using-with) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:683:8: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/util.py:779:11: R1732: Consider using 'with' for resource-allocating operations (consider-using-with) ************* Module taucmdr.cf.objects /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/objects.py:66:0: W0613: Unused argument 'args' (unused-argument) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/objects.py:66:0: W0613: Unused argument 'kwargs' (unused-argument) ************* Module taucmdr.model.application /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/application.py:49:0: C0116: Missing function or method docstring (missing-function-docstring) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/application.py:50:4: C0415: Import outside toplevel (taucmdr.model.project.Project) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/application.py:51:4: C0415: Import outside toplevel (taucmdr.model.target.Target) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/application.py:52:4: C0415: Import outside toplevel (taucmdr.model.measurement.Measurement) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/application.py:53:4: C0415: Import outside toplevel (taucmdr.cf.platforms.DARWIN, taucmdr.cf.platforms.HOST_OS, taucmdr.cf.platforms.CRAY_CNL) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/application.py:179:8: C0415: Import outside toplevel (taucmdr.error.ImmutableRecordError) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/application.py:180:8: C0415: Import outside toplevel (taucmdr.model.experiment.Experiment) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/application.py:198:8: C0415: Import outside toplevel (taucmdr.error.ImmutableRecordError) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/application.py:199:8: C0415: Import outside toplevel (taucmdr.model.experiment.Experiment) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/application.py:210:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/application.py:220:8: C0415: Import outside toplevel (taucmdr.model.project.Project) (import-outside-toplevel) ************* Module taucmdr.cf.platforms /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/platforms.py:77:17: W1514: Using open without explicitly specifying an encoding (unspecified-encoding) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/platforms.py:79:20: R1724: Unnecessary "elif" after "continue" (no-else-continue) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/platforms.py:125:16: C0415: Import outside toplevel (platform) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/platforms.py:130:20: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/platforms.py:179:16: C0415: Import outside toplevel (platform) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/platforms.py:184:20: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/platforms.py:234:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) ************* Module taucmdr.model.trial /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/trial.py:58:4: C0415: Import outside toplevel (taucmdr.model.experiment.Experiment) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/trial.py:159:8: R1720: Unnecessary "else" after "raise" (no-else-raise) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/trial.py:382:8: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/trial.py:352:4: R1710: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/trial.py:412:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/trial.py:525:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/trial.py:530:4: R0914: Too many local variables (23/20) (too-many-locals) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/trial.py:566:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/trial.py:588:24: W0715: Exception arguments suggest string formatting might be intended (raising-format-tuple) ************* Module taucmdr.cf.software.sqlite3_installation /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/sqlite3_installation.py:50:0: C0301: Line too long (121/120) (line-too-long) ************* Module taucmdr.cf.software.binutils_installation /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/binutils_installation.py:62:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/binutils_installation.py:68:8: C0415: Import outside toplevel (taucmdr.cf.platforms.DARWIN, taucmdr.cf.platforms.IBM_BGP, taucmdr.cf.platforms.IBM_BGQ, taucmdr.cf.platforms.INTEL_KNC) (import-outside-toplevel) ************* Module taucmdr.cf.software.scorep_installation /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/scorep_installation.py:178:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/scorep_installation.py:184:12: R1724: Unnecessary "elif" after "continue" (no-else-continue) ************* Module taucmdr.cf.software.papi_installation /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/papi_installation.py:62:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/papi_installation.py:76:8: C0103: Variable name "cc" doesn't conform to '[a-z_][a-z0-9_]{2,30}$' pattern (invalid-name) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/papi_installation.py:122:16: R1723: Unnecessary "elif" after "break" (no-else-break) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/papi_installation.py:146:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/papi_installation.py:167:15: R1714: Consider merging these comparisons with "in" to "event_type in ('PRESET', 'NATIVE')" (consider-using-in) ************* Module taucmdr.cf.software.pdt_installation /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/pdt_installation.py:191:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) ************* Module taucmdr.cf.software.installation /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/installation.py:67:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/installation.py:157:20: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/installation.py:322:8: R1720: Unnecessary "else" after "raise" (no-else-raise) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/installation.py:355:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/installation.py:399:8: C0415: Import outside toplevel (taucmdr.cf.software) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/installation.py:420:16: R1720: Unnecessary "elif" after "raise" (no-else-raise) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/installation.py:421:20: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/installation.py:607:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) ************* Module taucmdr.cf.software.libunwind_installation /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/libunwind_installation.py:68:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) ************* Module taucmdr.cf.storage.levels /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/levels.py:64:0: C0116: Missing function or method docstring (missing-function-docstring) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/levels.py:73:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/levels.py:43:0: W0611: Unused import os (unused-import) ************* Module taucmdr.model.target /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:726:9: W0511: FIXME: not implemented (fixme) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:88:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:118:4: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:149:13: W1514: Using open without explicitly specifying an encoding (unspecified-encoding) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:159:0: R0914: Too many local variables (23/20) (too-many-locals) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:168:4: C0415: Import outside toplevel (taucmdr.model.project.Project) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:169:4: C0415: Import outside toplevel (taucmdr.cli.arguments.ParsePackagePathAction) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:170:4: C0415: Import outside toplevel (taucmdr.cf.compiler.host.CC, taucmdr.cf.compiler.host.CXX, taucmdr.cf.compiler.host.FC, taucmdr.cf.compiler.host.UPC, taucmdr.cf.compiler.host.INTEL, taucmdr.cf.compiler.host.PGI, taucmdr.cf.compiler.host.GNU) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:171:4: C0415: Import outside toplevel (taucmdr.cf.compiler.mpi.MPI_CC, taucmdr.cf.compiler.mpi.MPI_CXX, taucmdr.cf.compiler.mpi.MPI_FC, taucmdr.cf.compiler.mpi.INTEL) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:172:4: C0415: Import outside toplevel (taucmdr.cf.compiler.shmem.SHMEM_CC, taucmdr.cf.compiler.shmem.SHMEM_CXX, taucmdr.cf.compiler.shmem.SHMEM_FC) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:173:4: C0415: Import outside toplevel (taucmdr.cf.compiler.cuda.CUDA_CXX, taucmdr.cf.compiler.cuda.CUDA_FC) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:174:4: C0415: Import outside toplevel (taucmdr.cf.compiler.caf.CAF_FC) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:175:4: C0415: Import outside toplevel (taucmdr.cf.compiler.python.PY) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:532:8: C0415: Import outside toplevel (taucmdr.error.ImmutableRecordError) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:533:8: C0415: Import outside toplevel (taucmdr.model.experiment.Experiment) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:562:8: C0415: Import outside toplevel (taucmdr.error.ImmutableRecordError) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:563:8: C0415: Import outside toplevel (taucmdr.model.experiment.Experiment) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:574:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:591:8: C0415: Import outside toplevel (taucmdr.model.project.Project) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/model/target.py:723:4: R1710: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements) ************* Module taucmdr.cf.storage.sqlite3_project /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_project.py:59:8: C0415: Import outside toplevel (taucmdr.cf.storage.levels.USER_STORAGE) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_project.py:65:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_project.py:70:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_project.py:73:17: W1514: Using open without explicitly specifying an encoding (unspecified-encoding) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_project.py:105:8: C0415: Import outside toplevel (taucmdr.cf.storage.levels.USER_STORAGE, taucmdr.cf.storage.levels.SYSTEM_STORAGE) (import-outside-toplevel) ************* Module taucmdr.cf.storage.project /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/project.py:79:8: C0415: Import outside toplevel (taucmdr.cf.storage.levels.USER_STORAGE) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/project.py:85:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/project.py:90:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/project.py:93:17: W1514: Using open without explicitly specifying an encoding (unspecified-encoding) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/project.py:125:8: C0415: Import outside toplevel (taucmdr.cf.storage.levels.USER_STORAGE, taucmdr.cf.storage.levels.SYSTEM_STORAGE) (import-outside-toplevel) ************* Module taucmdr.cf.storage.local_file /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/local_file.py:72:27: W1514: Using open without explicitly specifying an encoding (unspecified-encoding) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/local_file.py:72:27: R1732: Consider using 'with' for resource-allocating operations (consider-using-with) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/local_file.py:80:8: R1720: Unnecessary "else" after "raise" (no-else-raise) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/local_file.py:168:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/local_file.py:186:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/local_file.py:200:4: W0236: Method 'prefix' was expected to be 'method', found it instead as 'property' (invalid-overridden-method) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/local_file.py:228:4: R1710: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/local_file.py:294:8: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/local_file.py:334:8: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/local_file.py:374:8: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/local_file.py:406:8: R1705: Unnecessary "elif" after "return" (no-else-return) ************* Module taucmdr.cf.storage.sqlite3_file /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_file.py:536:9: W0511: TODO This could be made more efficient if we make this function less general. (fixme) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_file.py:679:13: W0511: TODO Change this to use a single operation (fixme) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_file.py:109:16: W1203: Use lazy % formatting in logging functions (logging-fstring-interpolation) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_file.py:133:12: W1203: Use lazy % formatting in logging functions (logging-fstring-interpolation) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_file.py:273:4: C0116: Missing function or method docstring (missing-function-docstring) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_file.py:292:4: C0116: Missing function or method docstring (missing-function-docstring) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_file.py:384:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_file.py:416:12: R1705: Unnecessary "else" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_file.py:465:8: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_file.py:498:8: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_file.py:539:8: R1720: Unnecessary "elif" after "raise" (no-else-raise) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_file.py:570:8: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/sqlite3_file.py:672:8: R1705: Unnecessary "elif" after "return" (no-else-return) ************* Module taucmdr.cf.storage.storage_dispatch /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/storage_dispatch.py:330:0: W0311: Bad indentation. Found 10 spaces, expected 12 (bad-indentation) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/storage_dispatch.py:93:8: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/storage/storage_dispatch.py:156:4: W0236: Method 'prefix' was expected to be 'method', found it instead as 'property' (invalid-overridden-method) ************* Module taucmdr.mvc.controller /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/controller.py:46:8: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/controller.py:54:0: C0116: Missing function or method docstring (missing-function-docstring) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/controller.py:66:12: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/controller.py:221:8: R1705: Unnecessary "else" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/controller.py:232:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/controller.py:271:19: R1727: Boolean condition ''model' or 'collection' in self.model.attributes[attr]' will always evaluate to ''model'' (condition-evals-to-constant) ************* Module taucmdr.cf.compiler /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/compiler/__init__.py:139:8: C0415: Import outside toplevel (taucmdr.cf.platforms.HOST_TAU_MAGIC) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/compiler/__init__.py:316:28: R1705: Unnecessary "else" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/compiler/__init__.py:298:8: R1702: Too many nested blocks (6/5) (too-many-nested-blocks) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/compiler/__init__.py:368:8: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/compiler/__init__.py:552:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/compiler/__init__.py:596:16: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/compiler/__init__.py:660:4: C0116: Missing function or method docstring (missing-function-docstring) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/compiler/__init__.py:697:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/compiler/__init__.py:702:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/compiler/__init__.py:730:13: W1514: Using open without explicitly specifying an encoding (unspecified-encoding) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/compiler/__init__.py:804:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) ************* Module taucmdr.cf.software.tau_installation /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:428:0: C0301: Line too long (136/120) (line-too-long) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:431:46: C1802: Do not use `len(SEQUENCE)` without comparison to determine if a sequence is empty (use-implicit-booleaness-not-len) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:477:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:518:8: R1705: Unnecessary "else" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:542:39: R1719: The if expression can be replaced with 'bool(test)' (simplifiable-if-expression) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:571:13: W1514: Using open without explicitly specifying an encoding (unspecified-encoding) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:573:16: R1724: Unnecessary "elif" after "continue" (no-else-continue) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:802:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:830:16: C0103: Variable name "g" doesn't conform to '[a-z_][a-z0-9_]{2,30}$' pattern (invalid-name) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:834:20: C0103: Variable name "g" doesn't conform to '[a-z_][a-z0-9_]{2,30}$' pattern (invalid-name) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:927:4: C0116: Missing function or method docstring (missing-function-docstring) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:978:16: R1720: Unnecessary "elif" after "raise" (no-else-raise) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:979:20: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:996:20: W1203: Use lazy % formatting in logging functions (logging-fstring-interpolation) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:999:27: W0703: Catching too general exception Exception (broad-except) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1004:24: R1720: Unnecessary "else" after "raise" (no-else-raise) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1004:31: R1729: Use a generator instead 'all(os.path.exists(path[1]) for path in err.args[0])' (use-a-generator) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:954:4: R1710: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1144:16: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1209:15: R1717: Consider using a dictionary comprehension (consider-using-dict-comprehension) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1470:20: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1479:13: W1514: Using open without explicitly specifying an encoding (unspecified-encoding) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1479:45: W1514: Using open without explicitly specifying an encoding (unspecified-encoding) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1591:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1600:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1625:8: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1662:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1736:4: R1710: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1885:4: C0116: Missing function or method docstring (missing-function-docstring) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1910:8: C0103: Variable name "p" doesn't conform to '[a-z_][a-z0-9_]{2,30}$' pattern (invalid-name) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:1911:8: C0103: Variable name "m" doesn't conform to '[a-z_][a-z0-9_]{2,30}$' pattern (invalid-name) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cf/software/tau_installation.py:167:0: R0904: Too many public methods (27/25) (too-many-public-methods) ************* Module taucmdr.cli /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/__init__.py:128:8: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/__init__.py:239:4: R1705: Unnecessary "elif" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/__init__.py:230:0: R1710: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/__init__.py:271:8: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) ************* Module taucmdr.mvc.model /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/model.py:44:40: R1735: Consider using {} instead of dict() (use-dict-literal) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/model.py:53:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/model.py:73:4: C0116: Missing function or method docstring (missing-function-docstring) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/model.py:84:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/model.py:172:8: R1705: Unnecessary "else" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/model.py:213:16: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/model.py:225:20: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/model.py:272:24: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/model.py:287:28: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/model.py:297:24: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/mvc/model.py:603:8: C0415: Import outside toplevel (taucmdr.cli.arguments.ArgumentsNamespace) (import-outside-toplevel) ************* Module taucmdr.cli.commands.help /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/help.py:164:12: R1720: Unnecessary "else" after "raise" (no-else-raise) ************* Module taucmdr.cli.commands.rewrite /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/rewrite.py:67:8: R1720: Unnecessary "elif" after "raise" (no-else-raise) ************* Module taucmdr.cli.cli_view /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/cli_view.py:171:12: C0415: Import outside toplevel (taucmdr.cli.commands.project.edit.COMMAND) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/cli_view.py:548:8: R1705: Unnecessary "elif" after "return" (no-else-return) ************* Module taucmdr.cli.commands.__main__ /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/__main__.py:136:8: C0415: Import outside toplevel (taucmdr.model.project.Project) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/__main__.py:151:8: R1705: Unnecessary "else" after "return" (no-else-return) ************* Module taucmdr.cli.commands.dashboard /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/dashboard.py:68:16: C0415: Import outside toplevel (taucmdr.cli.commands.project.create.COMMAND) (import-outside-toplevel) ************* Module taucmdr.cli.arguments /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/arguments.py:475:0: C0301: Line too long (123/120) (line-too-long) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/arguments.py:145:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/arguments.py:280:8: R1705: Unnecessary "else" after "return" (no-else-return) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/arguments.py:480:20: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/arguments.py:507:12: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/arguments.py:528:8: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from) ************* Module taucmdr.cli.commands.trial.list /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/trial/list.py:63:11: R1714: Consider merging these comparisons with "in" to "key in ('environment', 'output')" (consider-using-in) ************* Module taucmdr.cli.commands.trial.renumber /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/trial/renumber.py:80:8: W1201: Use lazy % formatting in logging functions (logging-not-lazy) ************* Module taucmdr.cli.commands.select /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/select.py:117:4: R1710: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements) ************* Module taucmdr.cli.commands.project.edit /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/project/edit.py:163:8: C0415: Import outside toplevel (taucmdr.cli.commands.project.list.COMMAND) (import-outside-toplevel) ************* Module taucmdr.cli.commands.target.edit /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/target/edit.py:105:8: C0415: Import outside toplevel (taucmdr.cli.commands.target.copy.COMMAND) (import-outside-toplevel) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/target/edit.py:106:8: C0415: Import outside toplevel (taucmdr.cli.commands.experiment.delete.COMMAND) (import-outside-toplevel) ************* Module taucmdr.cli.commands.experiment.edit /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/experiment/edit.py:38:0: C0115: Missing class docstring (missing-class-docstring) ************* Module taucmdr.cli.commands.target.create /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/target/create.py:108:13: W1514: Using open without explicitly specifying an encoding (unspecified-encoding) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/target/create.py:1:0: R0801: Similar lines in 2 files ==taucmdr.cli.commands.target.copy:[43:79] ==taucmdr.cli.commands.target.edit:[45:81] @staticmethod def _compiler_flag_action_call(family_attr): def call(self, parser, namespace, value, *args, **kwargs): try: delattr(namespace, family_attr) except AttributeError: pass return self.__action_call__(parser, namespace, value, *args, **kwargs) return call @staticmethod def _family_flag_action(kbase, family_attr): class Action(arguments.Action): # pylint: disable=too-few-public-methods def __call__(self, parser, namespace, value, *args, **kwargs): try: delattr(namespace, family_attr) except AttributeError: pass family = InstalledCompilerFamily(kbase.families[value]) for comp in family: setattr(namespace, comp.info.role.keyword, comp.absolute_path) return Action def _configure_argument_group(self, group, kbase, family_flag, family_attr): # Add the compiler family flag. If the knowledgebase keyword isn't all-caps then show in lower case. keyword = kbase.keyword if keyword.upper() != keyword: keyword = keyword.lower() group.add_argument(family_flag, help=("select all %(kw)s compilers automatically from the given family, " "ignored if at least one %(kw)s compiler is specified") % {'kw': keyword}, metavar='', dest=family_attr, default=arguments.SUPPRESS, choices=kbase.family_names(), (duplicate-code) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/target/create.py:1:0: R0801: Similar lines in 2 files ==taucmdr.cf.storage.project:[84:133] ==taucmdr.cf.storage.sqlite3_project:[64:113] raise StorageError("Cannot create project in home directory. " "Use '-@ user' option for user level storage.") try: util.mkdirp(project_prefix) except Exception as err: raise StorageError("Failed to access %s filesystem prefix '%s': %s" % (self.name, project_prefix, err)) # Exclude project storage directory from git with open(os.path.join(self.prefix, '.gitignore'), 'w+') as fout: fout.write('/*\n') LOGGER.debug("Initialized %s filesystem prefix '%s'", self.name, project_prefix) def destroy(self, *args, **kwargs): """Disconnects the database and filesystem and recursively deletes the filesystem. Args: *args: Passed through to :any:`disconnect_filesystem`. **kwargs: Keyword arguments for :any:`disconnect_filesystem` or :any:`shutil.rmtree`. """ self.disconnect_filesystem(*args, **kwargs) ignore_errors = kwargs.pop('ignore_errors', False) onerror = kwargs.pop('onerror', None) if self._prefix: util.rmtree(self._prefix, ignore_errors=ignore_errors, onerror=onerror) self._prefix = None @property def prefix(self): """Searches the current directory and its parents for a TAU Commander project directory. This method **does not** create or modify files. If the project directory cannot be found then an error is raised. It's up to the caller to determine how the error should be handled. Returns: str: The project directory, i.e. this storage container's filesystem prefix. Raises: ProjectStorageError: Neither the current directory nor any of its parent directories contain a TAU Commander project directory. """ from taucmdr.cf.storage.levels import USER_STORAGE, SYSTEM_STORAGE if self._prefix: return self._prefix cwd = os.getcwd() if self._force_cwd: # Only check current working directory for project directory prefix = os.path.realpath(os.path.join(cwd, PROJECT_DIR)) if os.path.isdir(prefix): for exclude_storage in USER_STORAGE, SYSTEM_STORAGE: (duplicate-code) /home/runner/work/taucmdr/taucmdr/packages/taucmdr/cli/commands/target/create.py:1:0: R0801: Similar lines in 2 files ==taucmdr.cli.commands.target.create:[52:76] ==taucmdr.cli.commands.target.edit:[45:69] @staticmethod def _compiler_flag_action_call(family_attr): def call(self, parser, namespace, value, *args, **kwargs): try: delattr(namespace, family_attr) except AttributeError: pass return self.__action_call__(parser, namespace, value, *args, **kwargs) return call @staticmethod def _family_flag_action(kbase, family_attr): class Action(arguments.Action): # pylint: disable=too-few-public-methods def __call__(self, parser, namespace, value, *args, **kwargs): try: delattr(namespace, family_attr) except AttributeError: pass family = InstalledCompilerFamily(kbase.families[value]) for comp in family: setattr(namespace, comp.info.role.keyword, comp.absolute_path) return Action (duplicate-code)
Update to python3 version on unstable ahead of release