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.

Sign up for GitHub

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

Jump to bottom

Error encountered when serializing moving_average_variables #1123

Closed
bernardopires opened this issue Feb 16, 2016 · 4 comments
Closed

Error encountered when serializing moving_average_variables #1123

bernardopires opened this issue Feb 16, 2016 · 4 comments
Assignees
@mrry
Labels
type:bug Bug

Comments

@bernardopires
Copy link
Contributor

I compiled Tensorflow from source. Running cifar10_train.py outputs the following warning message:

WARNING:tensorflow:Error encountered when serializing moving_average_variables.
Type is unsupported, or the types of the items don't match field type in CollectionDef.
unbound method to_proto() must be called with Variable instance as first argument (got Tensor instance instead)
@vrv
Copy link

vrv commented Feb 17, 2016

Also @shlens in case he knows.

@ZhengBitFusion
Copy link

Having the same issue. And the training command hangs. In fact, even the mnist example on setup page hangs too.

@vrv vrv added the bug label Feb 17, 2016
@mrry mrry assigned mrry and unassigned sherrym Feb 17, 2016
@mrry
Copy link
Contributor

mrry commented Feb 17, 2016

Thanks for pointing out this warning! It's benign (for the training process), but unfortunate to have this in an example model. I have a patch in the pipeline that will fix the root cause for the warning.

@vrv vrv closed this as completed in 42f06d8 Feb 17, 2016
vrv pushed a commit that referenced this issue Feb 17, 2016
@mrry
Fixes issues with the tf.train.Saver in the 0.7.0 release.
3f1d5d5 
1. The cifar10_train.py example model was emitting warnings, because of
   non-Variable objects in the `tf.moving_average_variables()`
   collection. This change fixes that by only adding `Variable`-typed
   objects to that collection in `moving_averages.py` (which better
   agrees with the definition in `tf.GraphKeys.MOVING_AVERAGES_VARIABLES`).

2. Saver.save() now calls `tf.gfile.MakeDirs(os.path.dirname(save_path))`,
   which fails if `save_path` does not contain a directory component.
   This change fixes the implementation of `tf.gfile.MakeDirs('')` to be a
   no-op (which better matches the internal library that it is shadowing).

Fixes #1123. Fixes #1135.
Change: 114895020
@trinity77
Copy link

Issue Cleanup

@aselle aselle added type:bug Bug and removed bug labels Feb 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mrry mrry

Labels
type:bug Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@mrry @aselle @vrv @bernardopires @trinity77 @sherrym @ZhengBitFusion