search

neuroailab / tfutils

Utilities for working with tensorflow
MIT License
25 stars 8 forks source link

tests issue #59

Closed qbilius closed 6 years ago

qbilius commented 7 years ago

I've slightly changed how variables and ops are named in model.py. Then test_training in tests/test.py started to fail, complaining that a particular name was not in the checkpoint. However, when I simply changed the database name, the tests started to pass, suggesting that in the first case something was being loaded from previously run tests.

yamins81 commented 7 years ago

I suspect this is because of local caching. The test should delete the local cache, which it currently doesn't, in addition to dropping the test database, which it currently does. I guess we should have real test setup and teardown.

On Tue, Mar 21, 2017 at 5:28 AM Jonas Kubilius notifications@github.com wrote:

I've slightly changed how variables and ops are named in model.py. Then test_training in tests/test.py started to fail, complaining that a particular name was not in the checkpoint. However, when I simply changed the database name, the tests started to pass, suggesting that in the first case something was being loaded from previously run tests.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/neuroailab/tfutils/issues/59, or mute the thread https://github.com/notifications/unsubscribe-auth/AAOHizLnRtGTylm3HbBTctZ1VWz5y0Y2ks5rn8J4gaJpZM4MjueK .