autonomio / talos

Hyperparameter Experiments with TensorFlow and Keras
https://autonom.io
MIT License
1.62k stars 269 forks source link

preparation test for merging with v.0.5 master to v.0.6.2 #364

Closed mikkokotila closed 4 years ago

mikkokotila commented 4 years ago
pep8speaks commented 4 years ago

Hello @mikkokotila! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 6:1: E402 module level import not at top of file Line 7:1: E402 module level import not at top of file Line 15:1: E402 module level import not at top of file Line 16:1: E402 module level import not at top of file

Line 122:80: E501 line too long (80 > 79 characters)

Line 202:80: E501 line too long (81 > 79 characters)

Line 1:1: E302 expected 2 blank lines, found 0 Line 100:80: E501 line too long (84 > 79 characters) Line 103:80: E501 line too long (82 > 79 characters)

Line 77:9: E722 do not use bare 'except' Line 183:80: E501 line too long (81 > 79 characters) Line 205:80: E501 line too long (80 > 79 characters) Line 220:80: E501 line too long (82 > 79 characters)

Line 21:80: E501 line too long (83 > 79 characters)

Line 30:80: E501 line too long (85 > 79 characters)

Line 16:80: E501 line too long (83 > 79 characters)

Line 41:80: E501 line too long (86 > 79 characters)

Line 35:19: E712 comparison to True should be 'if cond is True:' or 'if cond:'

Line 29:80: E501 line too long (87 > 79 characters)

Line 53:80: E501 line too long (80 > 79 characters)

Line 57:9: E722 do not use bare 'except'

Line 21:80: E501 line too long (127 > 79 characters) Line 45:80: E501 line too long (92 > 79 characters) Line 59:80: E501 line too long (92 > 79 characters) Line 84:80: E501 line too long (92 > 79 characters) Line 101:80: E501 line too long (92 > 79 characters) Line 119:80: E501 line too long (92 > 79 characters)

Line 7:80: E501 line too long (80 > 79 characters) Line 48:80: E501 line too long (80 > 79 characters)

Line 36:80: E501 line too long (96 > 79 characters)

Line 66:80: E501 line too long (97 > 79 characters) Line 151:5: E303 too many blank lines (2)

Line 11:18: E231 missing whitespace after ':' Line 20:1: E302 expected 2 blank lines, found 1 Line 26:80: E501 line too long (107 > 79 characters) Line 38:80: E501 line too long (85 > 79 characters) Line 39:53: E127 continuation line over-indented for visual indent Line 40:53: E127 continuation line over-indented for visual indent Line 40:80: E501 line too long (83 > 79 characters) Line 41:53: E127 continuation line over-indented for visual indent Line 42:53: E127 continuation line over-indented for visual indent Line 43:53: E127 continuation line over-indented for visual indent Line 54:1: E101 indentation contains mixed spaces and tabs Line 54:1: W191 indentation contains tabs Line 54:21: E128 continuation line under-indented for visual indent Line 59:1: E101 indentation contains mixed spaces and tabs

Line 23:80: E501 line too long (137 > 79 characters) Line 26:80: E501 line too long (118 > 79 characters)

Comment last updated at 2019-08-25 11:34:06 UTC
coveralls commented 4 years ago

Pull Request Test Coverage Report for Build 569


Changes Missing Coverage Covered Lines Changed/Added Lines %
talos/model/early_stopper.py 2 3 66.67%
talos/reducers/limit_by_metric.py 7 8 87.5%
talos/reducers/sample_reducer.py 9 10 90.0%
talos/commands/evaluate.py 15 17 88.24%
talos/logging/results.py 18 20 90.0%
talos/scan/scan_finish.py 13 15 86.67%
talos/utils/experiment_log_callback.py 40 42 95.24%
talos/utils/torch_history.py 3 5 60.0%
talos/metrics/entropy.py 15 18 83.33%
talos/commands/deploy.py 9 13 69.23%
<!-- Total: 837 1007 83.12% -->
Files with Coverage Reduction New Missed Lines %
talos/commands/evaluate.py 1 91.43%
talos/reducers/correlation.py 1 94.12%
talos/reducers/sample_reducer.py 1 69.7%
talos/model/normalizers.py 2 83.33%
talos/model/network_shape.py 5 82.14%
talos/utils/connection_check.py 5 0.0%
talos/commands/predict.py 10 21.05%
<!-- Total: 25 -->
Totals Coverage Status
Change from base Build 521: -5.0%
Covered Lines: 1304
Relevant Lines: 1527

💛 - Coveralls