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

size_policy error #175

Closed
dhruvbalwada opened this issue Oct 21, 2019 · 7 comments
Closed

size_policy error #175

dhruvbalwada opened this issue Oct 21, 2019 · 7 comments

Comments

@dhruvbalwada
Copy link
Contributor

We noticed today that the following error shows up when trying to use llcreader
TypeError: request() got an unexpected keyword argument 'size_policy'

The error can be reproduced quite simply as :

model = llcreader.ECCOPortalLLC4320Model()
model
ds_sst = model.get_dataset(varnames=['Theta'], k_levels=[0], type='latlon')
@rabernat
Copy link
Member

This was fixed in #170. Update xmitgcm and fsspec.

@dhruvbalwada
Copy link
Contributor Author

Just tried the updates on ocean pangeo, but still getting the same error.

Got the latest release of xmitgcm from github, and for fsspec pip install indicated that ocean pangeo was already on latest version.

@dhruvbalwada
Copy link
Contributor Author

dhruvbalwada commented Oct 21, 2019

It is possible that the xmitgcm installation is not working. We do not get the error if we are executing the python commands for reading llc data while being inside the xmitgcm folder.

@rabernat
Copy link
Member

rabernat commented Oct 21, 2019

The following works for me on ocean.pangeo.io:

import xmitgcm
from xmitgcm import llcreader
import fsspec
print(xmitgcm.__version__, fsspec.__version__)
# --> 0.4.1 0.5.2
model = llcreader.ECCOPortalLLC4320Model()
ds_sst = model.get_dataset(varnames=['Theta'], k_levels=[0], type='latlon')
ds_sst

edited to remove unnecessary NUMPY_EXPERIMENTAL_ARRAY_FUNCTION stuff...

@dhruvbalwada
Copy link
Contributor Author

Still getting the same error (but it shows up when trying to plot something) :
Screen Shot 2019-10-21 at 12 17 38 PM

@rabernat
Copy link
Member

rabernat commented Oct 21, 2019

Ok good catch. ocean has xmitgcm v0.4.1. The changes introduced in #170 have not been released yet.

To get them, pip install git+https://github.com/MITgcm/xmitgcm.git. Then restart your kernel. That will fix your issue.

We should do a new release soon.

@dhruvbalwada
Copy link
Contributor Author

Thanks, that worked now.
For some reason when I had cloned the latest version from github and ran setup.py it had not worked, maybe some update to the setup file is needed? Or maybe I was doing something stupid.

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