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

Explicitly pass values kwarg to tf.name_scope #3

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

Conversation

bananabowl
Copy link

Explicitly pass values kwarg to tf.name_scope as it is currently being treated as the default_name kwarg instead. This causes an exception to be thrown in eager mode.

Explicitly pass values kwarg to tf.name_scope as it is currently being treated as the default_name kwarg instead. This causes an exception to be thrown in eager mode.
@bananabowl
Copy link
Author

Friendly ping!

@cheind
Copy link
Owner

cheind commented May 2, 2019

sorry, I'm awaiting the build check (which failed last time for non obvious reasons). Once passed, I will merge.

@bananabowl
Copy link
Author

Looks like the build check failed for a different reason:

https://ci.appveyor.com/project/cheind/tf-matplotlib/builds/24248719/job/et0kls2215q8b0jt

Installing collected packages: tfmpl
*** Error compiling 'C:\Users\appveyor\AppData\Local\Temp\1\pip-install-ydipx5en\tfmpl\tfmpl\samples\mnist.py'...
File "C:\Users\appveyor\AppData\Local\Temp\1\pip-install-ydipx5en\tfmpl\tfmpl\samples\mnist.py", line 79
print(f'Iteration {i}')
^
SyntaxError: invalid syntax

*** Error compiling 'C:\Users\appveyor\AppData\Local\Temp\1\pip-install-ydipx5en\tfmpl\tfmpl\samples\sgd.py'...
File "C:\Users\appveyor\AppData\Local\Temp\1\pip-install-ydipx5en\tfmpl\tfmpl\samples\sgd.py", line 76
xys = [tf.get_variable(f'xy_{o[1]}', 2, tf.float32, initializer=start) for o in optimizers]
^
SyntaxError: invalid syntax

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