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

Example recipe is too specific: will not work with DRS: BADC or DKRZ #614

Closed
valeriupredoi opened this issue Sep 20, 2018 · 10 comments
Closed
Assignees

Comments

@valeriupredoi
Copy link
Contributor

- {dataset: ERA-Interim, project: OBS, tier: 3, type: reanaly, version: 1, start_year: 2000, end_year: 2002}

This has to be a general purpose example recipe and the fact that ERA-Interim is listed under the dataset section makes it fail when running it with drs set to e.g. BADC; suggest removing the ERA-Interim data request

@bouweandela
Copy link
Member

If it fails because drs is set to BADC, that is a problem with _data_finder.py, not with the recipe.

I agree that having an example that uses observational data that is difficult to obtain is not ideal though, maybe we should remove the NCEP dataset as well.

@mattiarighi
Copy link
Contributor

I agree. The easiest solution if you want to have observational data in the example recipes is to use the obs4mips data, which are freely available and already cmorized.

Check this recipe for example.

But we can also have examples using only models, that's fine as well.

@mattiarighi
Copy link
Contributor

@valeriupredoi can you take care of this and then close this issue? Thanks! 👍

@valeriupredoi
Copy link
Contributor Author

aye! @bouweandela @mattiarighi

@fserva
Copy link
Contributor

fserva commented Oct 3, 2018

I am also trying to run the python example. However, there is a fatal error when self._check_rank()
is called, causing File "ESMValTool/esmvaltool/cmor/check.py", line 125, in report_errors raise CMORCheckError(msg) esmvaltool.cmor.check.CMORCheckError: There were errors in variable ta: ta: does not match coordinate rank in cube:. Do you think it is safe to simply comment/remove this check?
In that case, there is a plotting error with pcolormesh
result = self._pcolormesh_patched(*args, **kwargs) File "/home/serva/miniconda3/envs/esmvaltool/lib/python3.6/site-packages/cartopy/mpl/geoaxes.py", line 1469, in _pcolormesh_patched if not self._hold: AttributeError: 'GeoAxesSubplot' object has no attribute '_hold'
A related discussion is here: SciTools/cartopy#1120
Any suggestion? Many thanks

@mattiarighi
Copy link
Contributor

The issue with _check_rank has been solved by #627. You probably need to merge the latest version2_development into your branch.

@valeriupredoi can you help with the other issue?

@valeriupredoi
Copy link
Contributor Author

I am unable to reproduce the (second) issue; I have, however, removed the OBS entries in the recipe and tested - it works fine. Can you give it a test, please? #633 it is surely not related to the OBS files, but a check run of the recipe is still needed so we can push the PR. Can you tell me what matplotlib version you working with pls? 2.2.2 (one I have) is not complaining of anything

@fserva
Copy link
Contributor

fserva commented Oct 4, 2018

The environment is using matplotlib 3.0.0, and basemap 1.2.0. I will try merging the latest version2_development and removing OBS from the recipe. Thanks

@fserva
Copy link
Contributor

fserva commented Oct 4, 2018

Ok, after pulling the latest version2_development the rank error is gone. Thanks. The only error I have appears to be related with the _hold at the plotting stage.

@valeriupredoi
Copy link
Contributor Author

yes, I can confirm that with matplotlib=3.0.0 I get the same error as you. It looks like the patch (code patch, not the graphical one) is not applied in the latest version. basemap can be whatever version, no effect on the issue at hand. I can suggest downgrading to matplotlib=2.2.2, that will avert this issue. Note that to do that you can not just conda install matplotlib=2.2.2 but rather need to downgrade the overarching package which is ncl (which is 6.5.0 for you) so need to do `conda install -c conda-forge ncl=6.4.0'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants