-
Notifications
You must be signed in to change notification settings - Fork 45.5k
Revert "tf_upgrade_v2 on resnet and utils folders. (#6154)" #6162
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
This reverts commit d6b2b83.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, Goldie-- can we then create a branch that is synced to the prior commit (right before this rollback) for v2 for now?
@tfboyd , so that he is informed. |
|
Why it was decided to stop tagging this feels crazy. |
@karmel I want to test the following
|
Keep in mind the README says this. The master branch of the models are in development, and they target the nightly binaries built from the master branch of TensorFlow. We aim to keep them backwards compatible with the latest release when possible (currently TensorFlow 1.5), but we cannot always guarantee compatibility. |
@tfboyd -- I think we resolved offline, but sounds like this is a problem with an older version, and the current Models HEAD works with TF 1.x HEAD, in which case a rollback/branch is not necessary. Sorry for the thrash on this, @goldiegadde |
AttributeError: module 'tensorflow.python.estimator.api.estimator' has no attribute 'SessionRunHook' |
This reverts commit d6b2b83.