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

generator = Generator(model) in demo produces error #5

Open
denisfitz57 opened this issue Sep 23, 2016 · 6 comments
Open

generator = Generator(model) in demo produces error #5

denisfitz57 opened this issue Sep 23, 2016 · 6 comments

Comments

@denisfitz57
Copy link

when running the demo using tensorflow .10, python 3.5 (anaconda), commit 20485a2 I get the following :

Make Generator.

TypeError Traceback (most recent call last)
in ()
----> 1 generator = Generator(model)

/home/denis/fast-wavenet/wavenet/models.py in init(self, model, batch_size, input_size)
99 count += 1
100
--> 101 outputs = _output_linear(h)
102
103 out_ops = [tf.argmax(tf.nn.softmax(outputs), 1)]

/home/denis/fast-wavenet/wavenet/layers.py in _output_linear(h, name)
170
171 def _output_linear(h, name=''):
--> 172 with tf.variable_scope(name, reuse=True):
173 w = tf.get_variable('w')[0, :, :]
174 b = tf.get_variable('b')

/home/denis/anaconda3/lib/python3.5/contextlib.py in enter(self)
57 def enter(self):
58 try:
---> 59 return next(self.gen)
60 except StopIteration:
61 raise RuntimeError("generator didn't yield") from None

/home/denis/anaconda3/lib/python3.5/site-packages/tensorflow/python/ops/variable_scope.py in variable_scope(name_or_scope, default_name, values, initializer, regularizer, caching_device, partitioner, custom_getter, reuse, dtype)
1350 """
1351 if default_name is None and not name_or_scope:
-> 1352 raise TypeError("If default_name is None then name_or_scope is required")
1353 if values is None:
1354 values = []

TypeError: If default_name is None then name_or_scope is required

@tomlepaine
Copy link
Owner

Do you still have this error with the newest updates? Also I haven't tested this with python 3. Can you try using it with python 2.7?

@jkyl
Copy link

jkyl commented Oct 11, 2016

I encounter an identical error with the latest version of fast-wavenet and python2.7. Will update if I find a fix.

@giovannic
Copy link

This is likely a tensorflow backward-compatibility issue: tensorflow/tensorflow#4576

@bhtucker
Copy link

bhtucker commented Jan 3, 2017

I also have this issue.

I don't know TF that well but this looks pretty solvable by simply giving a name to the variable scope where the output tensor is created. Not sure what else needs to be in that scope, though.

Would happily pair with someone to make a workaround!

@bhtucker
Copy link

bhtucker commented Jan 3, 2017

I've put up a PR that works on Python 2.7 and TF 0.11.0.

I'll test it this evening on Python 3.5, but this fix makes sense with the TF issue @giovannic posted, so there shouldn't be any language version differences.

@ishandutta2007
Copy link

ishandutta2007 commented Nov 5, 2018

@bhtucker @denisfitz57 @giovannic @jkyl if you have succeeded in training it and have the model with you can you please share the model.

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

No branches or pull requests

6 participants