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

Changes for TF 1.13.1 and CUDA 10.0 #88

Open
aolney opened this issue Apr 28, 2019 · 4 comments
Open

Changes for TF 1.13.1 and CUDA 10.0 #88

aolney opened this issue Apr 28, 2019 · 4 comments

Comments

@aolney
Copy link

aolney commented Apr 28, 2019

Getting error AttributeError: module 'tensorflow.contrib.rnn' has no attribute 'core_rnn_cell'

My TF is 1.13.1 and CUDA is 10.1

Guessing this is b/c of incompatibility b/w my version of TF and the presumed 1.0.0 version. However, I do not want to downgrade my CUDA from 10 to 8 just to use TF 1.0.0.

To fix this, I:

  1. Made the minor code change suggested in this closed PR
  2. Downloaded the current seq2seq, unzipped into the dependencies directory, and installed
  3. Recompiled using build.sh as described in the README
  4. Replaced the install/g2p-seq2seq-cmudict directory with one matching the seq2seq version

This seems to be working with my configuration. Since the aforementioned PR was declined, I'm documenting this only as an issue.

@joseamair
Copy link

Im having the same issue now.
I'm going to try with your solution. Hope It works.

@joseamair
Copy link

I'd followed your instructions, and now it is working for me @aolney. Thanks for the Guidance.

@saurabhshri
Copy link
Owner

@aolney Thank you very much for looking into this. For some reason, I can't recall, we decided to not use the recent ones. But I see no reason right now why we should not support the latest dependencies.

Unfortunately, I am not able to get enough time to look into this (hence, my superlate reply to this).

Feel free to open a PR with changes that would work with latest libs, or re-open the one I closed.

Thanks again for documenting this.

@aolney
Copy link
Author

aolney commented Oct 27, 2019

Unfortunately I blew my code away later in the summer to free up disk space. So to do this would require complete recreation.

@joseamair Would you consider a PR?

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

3 participants