Fix for https://github.com/autonomio/talos/issues/263

autonomio / talos

Hyperparameter Experiments with TensorFlow and Keras

https://autonom.io

MIT License

1.62k stars 268 forks source link

Fix for https://github.com/autonomio/talos/issues/263 #265

Closed JohanMollevik closed 5 years ago

pep8speaks commented 5 years ago

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

In the file talos/metrics/entropy.py:

Line 42:61: E225 missing whitespace around operator Line 42:80: E501 line too long (90 > 79 characters) Line 48:63: E225 missing whitespace around operator Line 48:80: E501 line too long (93 > 79 characters)

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 427

2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.056%

Totals
Change from base Build 424:	0.0%
Covered Lines:	1123
Relevant Lines:	1247

💛 - Coveralls

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 427

2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.056%

Totals
Change from base Build 424:	0.0%
Covered Lines:	1123
Relevant Lines:	1247

💛 - Coveralls

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 427

2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.056%

Totals
Change from base Build 424:	0.0%
Covered Lines:	1123
Relevant Lines:	1247

💛 - Coveralls

mikkokotila commented 5 years ago

Related with #268

mikkokotila commented 5 years ago

This had been since then fixed, so closing here.