Skip to content
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

Mark global_step_init as used to prevent spurious warning msg #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gar1t
Copy link
Contributor

@gar1t gar1t commented Nov 29, 2018

Without this, TF displays a warning message that looks like an
error (contains traceback):

E1121 12:34:00.888499 139860141586240 tf_logging.py:105] ==================================
Object was never used (type <class 'tensorflow.python.framework.ops.Operation'>):
<tf.Operation 'init' type=NoOp>
If you want to mark it as used call its "mark_used()" method.
It was originally created here:
  ...

This behavior occurs on TF 1.12.0 but reports suggest that these
warnings started appearing as early as TF 1.2.0.

Without this, TF displays a warning message that looks like an
error (contains traceback):

```
E1121 12:34:00.888499 139860141586240 tf_logging.py:105] ==================================
Object was never used (type <class 'tensorflow.python.framework.ops.Operation'>):
<tf.Operation 'init' type=NoOp>
If you want to mark it as used call its "mark_used()" method.
It was originally created here:
  ...
```

This behavior occurs on TF 1.12.0 but reports suggest that these
warnings started appearing as early as TF 1.2.0.
@craffel
Copy link
Contributor

craffel commented Jan 30, 2019

Hmm, I think a better solution is for global_step_init to be created in train_model.py instead. What do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants