Conversation
|
I think there might be changes coming as they are migrating their website today (cf. https://rda.ucar.edu/news/rda-gdex-relaunch/). I will have a look tomorrow! |
jlenh
left a comment
There was a problem hiding this comment.
Thanks Lisa, the new link works perfectly!
I think there are few things to remove still from the old code that are not relevant anymore:
- Removing the temporary files is not needed anymore
ESMValTool/esmvaltool/cmorizers/data/downloaders/datasets/jra_55.py
Lines 123 to 126 in c31a664
- The use of the
download_optionsparameter is not needed anymore
ESMValTool/esmvaltool/cmorizers/data/downloaders/datasets/jra_55.py
Lines 110 to 112 in c31a664
|
@jlenh I still have problems with the downloaded data, as the formatter is not working with them. The files are too small. Did you experience the same issue? |
The formatter raises |
|
I had another look and I cannot open the newly downloaded GRIB files as I always get the following error: On the other hand, I think I couldn't run the formatter on the already downloaded data on Levante because of the index files that are written when opening the GRIB files which cannot be done on my side as I do not have writing permission in the ESMValTool data folders. |
|
Made it work using the following address taken from the web downloader https://osdf-director.osg-htc.org/ncar/gdex/d628001 but I doubt that it is ideal as it always redirects to different addresses. I have also sent a message to the data host at NSF/NCAR Geoscience Data Exchange (GDEX) for further recommendations. |
|
Hello, this pull request has been marked with the If you won't be able to finish this in time, don't worry - just unassign the milestone |
I would be in favor of proceeding with this option in order to make the data downloadable again, keeping in mind that it would need an update later on (by opening a new issue probably?) as it does not use a supported download address. Haven't gotten an answer from the GDEX support yet. Thoughts from @ESMValGroup/technical-lead-development-team so that this can potentially be included in the 2.14 release? (I haven't pushed the commit with the modified download URL yet, waiting for feedback first) Here is an example of the downloading output using the aforementioned address: |
jlenh
left a comment
There was a problem hiding this comment.
Changing the link as discussed, so it's good to go now!
|
@schlunma If you want to test this quickly before merging, let me know! Otherwise I will merge later today 😄 |
|
Thanks @jlenh, fine for me if everything works fine and files on Levante have been updated (if necessary) 🚀 |
I tested again for a few years and it works fine. I just bumped the |
|
I can copy them over for you if you point me to your path! |
Description
The downloading works now without RDA login. And the link to the data changed.
Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.
New or updated data reformatting script