autonomio / talos

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

Proposal for preserving state of the experiment. #518

Closed mikkokotila closed 2 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

Docs

Tests


pep8speaks commented 3 years ago

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

Line 216:31: E261 at least two spaces before inline comment Line 291:1: W293 blank line contains whitespace Line 291:1: W391 blank line at end of file

Line 6:80: E501 line too long (80 > 79 characters) Line 15:80: E501 line too long (84 > 79 characters) Line 36:80: E501 line too long (93 > 79 characters)

Line 17:25: E128 continuation line under-indented for visual indent Line 21:33: E127 continuation line over-indented for visual indent Line 22:33: E127 continuation line over-indented for visual indent Line 42:80: E501 line too long (91 > 79 characters) Line 46:80: E501 line too long (83 > 79 characters)

Line 18:16: E231 missing whitespace after ':' Line 18:18: E251 unexpected spaces around keyword / parameter equals Line 54:19: E231 missing whitespace after ':' Line 54:21: E251 unexpected spaces around keyword / parameter equals Line 64:80: E501 line too long (80 > 79 characters) Line 91:19: E231 missing whitespace after ':' Line 91:21: E251 unexpected spaces around keyword / parameter equals Line 101:80: E501 line too long (82 > 79 characters)

mikkokotila commented 3 years ago

What do you think, what is the simplest way in which we can achieve this same? Ideally, we would not have to add anything except a point at each round, on which a local file is updated.