Closed mikkokotila closed 5 years ago
Hello @mikkokotila! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
talos/commands/reporting.py
:talos/logging/logging_run.py
:Line 21:80: E501 line too long (83 > 79 characters)
talos/metrics/entropy.py
:Line 30:80: E501 line too long (85 > 79 characters)
talos/scan/scan_round.py
:talos/templates/models.py
:Line 7:80: E501 line too long (80 > 79 characters) Line 48:80: E501 line too long (80 > 79 characters)
test/core_tests/test_scan.py
:Line 183:80: E501 line too long (81 > 79 characters)
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
talos/reducers/sample_reducer.py | 9 | 10 | 90.0% | ||
talos/utils/init.py | 1 | 2 | 50.0% | ||
talos/logging/results.py | 18 | 20 | 90.0% | ||
talos/scan/scan_round.py | 17 | 19 | 89.47% | ||
talos/metrics/entropy.py | 15 | 18 | 83.33% | ||
talos/reducers/reduce_run.py | 19 | 23 | 82.61% | ||
talos/commands/evaluate.py | 5 | 10 | 50.0% | ||
talos/commands/kerasmodel.py | 16 | 22 | 72.73% | ||
talos/reducers/correlation.py | 25 | 31 | 80.65% | ||
talos/reducers/limit_by_metric.py | 1 | 9 | 11.11% | ||
<!-- | Total: | 382 | 493 | 77.48% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
talos/commands/evaluate.py | 1 | 69.7% | ||
talos/reducers/correlation.py | 1 | 79.41% | ||
talos/utils/connection_check.py | 5 | 0.0% | ||
<!-- | Total: | 7 | --> |
Totals | |
---|---|
Change from base Build 450: | -5.2% |
Covered Lines: | 1086 |
Relevant Lines: | 1274 |
See commit details for more information.