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

Tensorflow implementation not compiling? #2

Open
maksay opened this issue May 30, 2017 · 2 comments
Open

Tensorflow implementation not compiling? #2

maksay opened this issue May 30, 2017 · 2 comments

Comments

@maksay
Copy link

maksay commented May 30, 2017

As I missing something, or both zoneout_tensorflow and zoneout_seq2seq feature non-existing variables and wrong function parameter names?

@ASalvail
Copy link

ASalvail commented Jun 2, 2017

Indeed, state_part_zoneout_prob is referenced in call but isn't initialized anywhere in the float version.
However, if you happen to have your zoneout probabilities in a numpy array and the state_size is a tuple, the computation will actually go through without problems. The code should still be fixed, but at least there is a working path.

@ASalvail
Copy link

ASalvail commented Jun 2, 2017

Be careful however since doing this is actually converting your tuple into a 2-dim numpy array. It works, but honnestly, it shouldn't.

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

2 participants