Skip to content

Adding figures of Bock and Lauer (2024) - #3526

Merged
schlunma merged 162 commits into
mainfrom
clouds_ecs_eval
Jan 10, 2025
Merged

Adding figures of Bock and Lauer (2024)#3526
schlunma merged 162 commits into
mainfrom
clouds_ecs_eval

Conversation

@LisaBock

@LisaBock LisaBock commented Feb 9, 2024

Copy link
Copy Markdown
Member

Description

In this PR we add the figures of Bock and Lauer, 2024

Fig. 3: Geographical maps of cloud properties
Fig. 4: Geographical maps of net cloud radiative effect
Fig. 6: Zonal averages of cloud properties and their relative projected changes
Fig. 7: Boxplots of relative changes in projected cloud properties

Note: In Figure 3 only the total cloud fraction is included and liquid and ice water path are commented out. The reason is a different standard name of these two variables in CMIP5 and CMIP6 which doesn't allow a computation of the multi model mean by the preprocessor. Iris is working on a solution of the problem: SciTools/iris#5313 (review)


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 recipe/diagnostic

@LisaBock

Copy link
Copy Markdown
Member Author

Thank you so much @schlunma for the review!

I tried my best regarding the codacy issues. Could we leave it like that?

Comment thread esmvaltool/diag_scripts/clouds/clouds_ecs_groups_zonal.py Outdated
LisaBock and others added 2 commits January 7, 2025 16:44
Co-authored-by: Manuel Schlund <32543114+schlunma@users.noreply.github.com>
@LisaBock

LisaBock commented Jan 7, 2025

Copy link
Copy Markdown
Member Author

Thanks @schlunma for reviewing!
I changed the line about the ancestor files in the other two diagnostic scripts as well. Hopefully, it's all fine now.

@schlunma

schlunma commented Jan 9, 2025

Copy link
Copy Markdown
Contributor

I still get the "invalid ancestor" error from clouds/clouds_ecs_groups_maps.py:

log.txt

@schlunma

schlunma commented Jan 9, 2025

Copy link
Copy Markdown
Contributor

Ah, and there is a WARNING Could not write HTML report: Some tags in the recipe could not be resolved: syntax error in line 13: premature end of file which prevents the index.html file to be written (which we need to get a nice overview of the results).

I am not quite sure what the problem is, I guess it's either a recipe problem (this warning appears in all 3 recipes) or a syntax error in the bibtex file you added (esmvaltool/references/bock24acp.bibtex).

@LisaBock

LisaBock commented Jan 9, 2025

Copy link
Copy Markdown
Member Author

Hi @schlunma , both issues should be solved now. In my tests, there are no warnings anymore.

Comment thread esmvaltool/diag_scripts/clouds/clouds_ecs_groups_maps.py Outdated
Co-authored-by: Manuel Schlund <32543114+schlunma@users.noreply.github.com>

@schlunma schlunma left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Lisa, looks good now! All recipes ran successful and their output looks good 🚀

@schlunma
schlunma merged commit 818569f into main Jan 10, 2025
@schlunma
schlunma deleted the clouds_ecs_eval branch January 10, 2025 13:57
@k-a-webb

Copy link
Copy Markdown

Is there a shape file available for recipe_bock24acp_fig7_boxplots.yml that should have been added to ESMValCore?

shapefile: shapefiles/sc_regions.shp

I would have expected to find it here?
https://github.com/ESMValGroup/ESMValCore/tree/main/esmvalcore/preprocessor/shapefiles

@axel-lauer

Copy link
Copy Markdown
Contributor

The shapefile can be found on Levante in our data pool at ESMValTool2/AUX/shapefiles. I don't think there is a policy yet where shapefiles for recipes should be put.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adding figures of Bock and Lauer (2024)

6 participants