autonomio / talos

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

Fix for #258 allow using list type hyperparameters #259

Closed JohanMollevik closed 5 years ago

pep8speaks commented 5 years ago

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

Line 64:10: E225 missing whitespace around operator Line 66:14: E225 missing whitespace around operator

Comment last updated at 2019-03-06 08:57:00 UTC
coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 426


Changes Missing Coverage Covered Lines Changed/Added Lines %
talos/utils/results.py 3 4 75.0%
<!-- Total: 9 10 90.0% -->
Totals Coverage Status
Change from base Build 424: 0.4%
Covered Lines: 1135
Relevant Lines: 1255

💛 - Coveralls
mikkokotila commented 5 years ago

Thanks for making the PR. This was never merged as the push is away from all kinds of custom formatting codes as much as it is possible.