Fix for reuse that caused issues

[kmyi]

reuse variable was being unused. This actually caused some issues when the program was run, causing tensorflow to crash.

Again, this fix may be incorrect. Please merge with caution.

Kwang

[kmyi]

Even with 1ecb075 I still get the following error

ValueError: Variable discriminator/conv_1/weights does not exist, or was not created with tf.get_variable(). Did you mean to set reuse=None in VarScope?

Perhaps you should set reuse=False on the first one?

Kwang

[carpedm20]

Are you using the most recent tf? I've never suffered from it even after I updated tf just before.

[kmyi]

Are you using the most recent tf? I've never suffered from it even after I updated tf just before.

Yeap, pip installed tensorflow tensorflow-gpu

tf.__version__ tells me 0.12.1

[carpedm20]

How about try nightly build like:

• https://ci.tensorflow.org/view/Nightly/job/nightly-matrix-linux-gpu/TF_BUILD_IS_OPT=OPT,TF_BUILD_IS_PIP=PIP,TF_BUILD_PYTHON_VERSION=PYTHON2,label=gpu-linux/lastSuccessfulBuild/artifact/pip_test/whl/tensorflow_gpu-0.12.1-cp27-none-linux_x86_64.whl
• https://ci.tensorflow.org/view/Nightly/job/nightly-matrix-mac-gpu/TF_BUILD_IS_OPT=OPT,TF_BUILD_IS_PIP=PIP,TF_BUILD_PYTHON_VERSION=PYTHON2,label=gpu-mac/lastSuccessfulBuild/artifact/pip_test/whl/tensorflow_gpu-0.12.1-py2-none-any.whl

These links can be found README.md of https://github.com/tensorflow/tensorflow and I always use one of these because pip code is always behind away.

[kmyi]

ooooh... nightly is quite scary though...

What about this fix? Should be correct right? It seems it's just a matter of which get's initialized first...

[carpedm20]

Oh, now I see the problem.

self.D_y, self.D_y_logits = \
      self._build_discrim(self.normalized_y, name="D_y")

which doesn't use reuse=True should be on time. So it should be like:

  self.D_y, self.D_y_logits = \
      self._build_discrim(self.normalized_y, name="D_y")
  self.D_R_x, self.D_R_x_logits = \
      self._build_discrim(self.R_x, name="D_R_x", reuse=True)
  self.D_R_x_history, self.D_R_x_history_logits = \
      self._build_discrim(self.R_x_history,
                          name="D_R_x_history", reuse=True)

Give me a PR again with this. I don't know why it went down..

[kmyi]

I squashed and change the PR :-) plz merge

[kmyi]

Give me a PR again with this. I don't know why it went down..

Maybe emacs or vim magic?

[carpedm20]

@kmyid Thanks again!