fixes recovery issue mentioned in #534 - Githubissues

autonomio / talos

Hyperparameter Experiments with TensorFlow and Keras

https://autonom.io

MIT License

1.62k stars 270 forks source link

fixes recovery issue mentioned in #534 #546

Closed mikkokotila closed 3 years ago

mikkokotila commented 3 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

[x] I'm aware of the implications of the proposed changes
[x] Code is PEP8
[x] I'm making the PR to master

Docs

[ ] Docs are updated
[ ] Docs version is correct (index.html and _coverpage.md)

Tests

[ ] Changes have gone through actual use testing
[x] All local tests have passed (run ./test.sh in /talos)
[x] Tests have been updated to reflect the changes

fixes #534

pep8speaks commented 3 years ago

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

In the file talos/utils/recover_best_model.py:

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

Comment last updated at 2021-05-25 18:09:32 UTC