Add measure for gpu power draw - Githubissues

autonomio / talos

Hyperparameter Experiments with TensorFlow and Keras

https://autonom.io

MIT License

1.62k stars 270 forks source link

Add measure for gpu power draw #416

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

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

Docs

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

Tests

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

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:

In the file talos/utils/power_draw_append.py:

Line 15:80: E501 line too long (81 > 79 characters)

In the file talos/utils/power_draw_callback.py:

Line 27:80: E501 line too long (95 > 79 characters)

Comment last updated at 2019-09-25 08:34:48 UTC