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

Set chech_for_unused_update_ops to False by default. #23

Open
Gsunshine opened this issue Mar 31, 2020 · 2 comments
Open

Set chech_for_unused_update_ops to False by default. #23

Gsunshine opened this issue Mar 31, 2020 · 2 comments
Labels
question Further information is requested

Comments

@Gsunshine
Copy link

Gsunshine commented Mar 31, 2020 •
edited
Loading

A bug is reported in SAGAN that check_for_unused_update_ops=True raise a ValueError. I find it suppose all ema buffers in bn to be unused update ops.

However there is no API in current SAGAN example to disable this check since it has ben encapsulated. So I set the kwarg in original tfgan implementation https://github.com/tensorflow/gan/blob/master/tensorflow_gan/python/train.py#L974 to False. The code runs normally then.

My tf version is 2.0.0 and tfgan is the current version.
@Gsunshine
Copy link
Author

Update: The bug is reported on 1080Ti with build-in GPU estimator. When I run the code on TPUs, everything is fine.

Hope this issue could help you!

@RenYurui RenYurui mentioned this issue Apr 24, 2020
Closed
@joel-shor
Copy link
Contributor

Do you experience this with TF 1.X?

@joel-shor joel-shor added the question Further information is requested label Jul 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question Further information is requested
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@joel-shor @Gsunshine