autonomio / talos

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

Adding new tests #398

Closed mikkokotila closed 4 years ago

mikkokotila commented 4 years ago

You want to make a PR to Talos

Thanks so much :) First, please take a moment to carefully check through the below items:

Sanity

Docs

Tests


coveralls commented 4 years ago

Pull Request Test Coverage Report for Build 607


Changes Missing Coverage Covered Lines Changed/Added Lines %
talos/reducers/local_strategy.py 4 6 66.67%
<!-- Total: 9 11 81.82% -->
Totals Coverage Status
Change from base Build 603: 4.4%
Covered Lines: 1336
Relevant Lines: 1490

💛 - Coveralls
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 41:80: E501 line too long (89 > 79 characters)

Comment last updated at 2019-09-15 12:23:53 UTC