removed use of np.prod to avoid using limited size integers solving #244

autonomio / talos

Hyperparameter Experiments with TensorFlow and Keras

https://autonom.io

MIT License

1.62k stars 268 forks source link

removed use of np.prod to avoid using limited size integers solving #244 #245

Closed JohanMollevik closed 5 years ago

pep8speaks commented 5 years ago

Hello @JohanMollevik! Thanks for submitting the PR.

In the file talos/parameters/ParamGrid.py, following are the PEP8 issues :

Line 27:26: E225 missing whitespace around operator Line 29:30: E225 missing whitespace around operator Line 34:80: E501 line too long (85 > 79 characters)

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 394

3 of 3 (100.0%) changed or added relevant lines in 1 file are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.4%) to 90.617%

Files with Coverage Reduction	New Missed Lines	%
talos/reducers/reduce_run.py	1	90.0%
talos/reducers/correlation.py	4	73.68%
<!--	Total:	5	-->

Totals
Change from base Build 393:	-0.4%
Covered Lines:	1101
Relevant Lines:	1215

💛 - Coveralls

mikkokotila commented 5 years ago

Great! Thanks :)