Extended ESRL cmorizer#1937
Conversation
|
@axel-lauer Wanna do me the honor of a review, since you looked at Sabrinas original implementation for just the main station data? Remaining codacy error is the use of unsecure ftp for data download, copied this from the CT2019 cmorizer script. Feel free to change if you have a better idea. |
|
Is there a reason why the new CMORizer only produces only ASK, GLOBAL and MLO? What happened to BRW, SMO, SPO? |
|
I just put them there as example, the cmorizer can handle all base stations (BRW, SMO, SPO, MLO), the global average from their website, as well as all the single flask sites in the ESRL network (https://www.esrl.noaa.gov/gmd/dv/site/?program=ccgg). So I used one station for each example. I wasn't sure if it should take all the possible stations as default input. |
|
The old CMORizer would output BRW, MLO, SMO, SPO. I would therefore suggest to additionally output BRW, SMO, SPO with the new version. |
|
I've added support for an "all" keyword put in as default, which will process all available datasets - so this also includes brw, spo, mlo and smo -, and added a flag for downloading additional files, in case someone only wants to format all datasets locally available. I hope that addresses your concern sufficiently! |
axel-lauer
left a comment
There was a problem hiding this comment.
This looks good. I tested the CMORizer and it does exactly what it is supposed to do. The only thing left to do would be to try to fix (some of) the Codacy issues. @bettina-gier could you please take a look? Other than that, this is ready for merging.
|
Thanks for testing Axel! |
axel-lauer
left a comment
There was a problem hiding this comment.
Thanks for addressing the Codacy issues.
Required for #1915
Checklist for technical review
yamllintto check that your YAML files do not contain mistakesNew or updated data reformatting script:
Automated checks pass, status can be seen below the pull request:
Detailslink to find out why.Detailslink to see it.Checklist for scientific review
New or updated data reformatting script:
If you need help with any of the items on the checklists above, please do not hesitate to ask by commenting in the issue or pull request.
Old NCL ESRL cmorizer only worked for baseline observatories. New cmorizer rewritten in python, includes capability to process surface flask monthly data, as well as global mean data from ESRL website. Additionally writes altitude to the netcdf files, gets this information from the ESRL website table, no external file required. Kept the "OBS" datatable so recipes using data from the old cmorizer will not need to be changed. Will download data if not present in RAW folder.
Changed recipe_check_obs.yml to include one of each case. No change to documentation necessary as ESRL cmorizer already noted there.
Required for #1915