-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Issues using Sonnet #261
Comments
This is the specific spot the error is occurring at: https://github.com/deepmind/dnc/blob/master/dnc/addressing.py#L58 Downloading an older version of Sonnet (if it will still work, that is) would also work, if that's a possibility, though I can't find a way to install an older version. Also, I realized that it's |
I unfortunately cannot downgrade Sonnet, so that is not an option. There is an issue with tensorflow-probability 0.8.0 (required by Sonnet 1.36) requiring cloudpickle==1.1.1, while Gymnasium, which I am using to simulate and train with, requires cloudpickle>=1.2.0. |
Hi @Denzeduous , indeed the first error is from using the wrong Sonnet package 😄 With regards to not being able to downgrade, I'm afraid that DNC requires TF1 and Sonnet <2. I wonder what wold happen if you installed the old version of cloudpickle to satisfy TFP during install, and then force upgraded it to 1.2.0. |
Hello,
I'm having an issue attempting to run a project dependent on Sonnet (more specifically, the DeepMind DNC implementation). I'm getting the error:
Thinking that maybe I'm missing a package, I did the next obvious thing and ran
pip install graphs
(which I felt is not the solution, thatgraphs
must be an internal package of sorts, but I still gave it a shot) and got the errorImportError: cannot import name 'Sonnet' from 'graphs'
I thought that maybe the version on pip is outdated, and it seems like it is. I then attempted to download the source code and run
pip install -e
on the extracted folder. However, after that, I get:Does anyone have any idea what I could do to solve these issues? Is there a way I could modify the DNC code to work with modern Sonnet? Is there an alternative to AbstractModule in this use case?
The text was updated successfully, but these errors were encountered: