added cleanup of keras model to avoid a GPY memmory related crash in CNTK (fixes #206)

autonomio / talos

Hyperparameter Experiments with TensorFlow and Keras

https://autonom.io

MIT License

1.62k stars 268 forks source link

added cleanup of keras model to avoid a GPY memmory related crash in CNTK (fixes #206) #247

Closed JohanMollevik closed 5 years ago

JohanMollevik commented 5 years ago

Fix for https://github.com/autonomio/talos/issues/206

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:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2019-03-05 10:12:35 UTC

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 423

3 of 3 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 90.32%

Totals
Change from base Build 422:	-0.06%
Covered Lines:	1129
Relevant Lines:	1250

💛 - Coveralls

mikkokotila commented 5 years ago

Thanks!

Check out the preview for v.0.6 where this is handled already. Shall we move to the same in v.0.5... in the meantime?

JohanMollevik commented 5 years ago

No need for my sake, as long as it will find its way into master eventually I'm happy

mikkokotila commented 5 years ago

@JohanMollevik ok great, it will. I stay with testing this week, and I think we can merge v0.6.0 with daily dev by early next week and then it will make its way fast. Closing here.