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

Add LAI to the static information. #1255

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

tanyasmirnova
Copy link

@tanyasmirnova tanyasmirnova commented Dec 19, 2024

This PR adds monthly climatology for leaf area index to the static information in core_init_atmosphere.
It also introduces separate alarms for updating three surface variables: albedo, lai and greenness, in core_atmosphere.
The alarms will be created only if config_albedo_update, config_lai_update and config_greeness_update /= 'none'. The default values for these flags is 24 hours.

core_init_atmosphere/mpas_init_atm_static.F added horizontal
interpolation of the monthly climatological LAI.
	- replaced config_sfc_albedo with config_albedo_update  (this
name is more descriptive of its meaning)
	- added config_lai_update
	- added new variable lai12m: monthly climatology for leaf area
index
	- use separate alarms for albedo, lai and greenness updating,
          controlled by config_albedo_update, config_lai_update and config_greeness_update
        - set alarms only if config_albedo_update, config_lai_update and config_greeness_update
          are not equal to 'none'
        - instead of one physics_update_surface call to
          physics_update_surface_albedo, physics_update_surface_lai and
          physics_update_surface_greeness. This is needed because of some
          specifics for each variable.
	instead of one physics_update_surface do three subroutines:
        physics_update_surface_albedo, physics_update_surface_lai and
        physics_update_surface_gvf
<var name="snoalb"/>
<var name="albedo12m"/>
<var name="greenfrac"/>
<var name="lai12m"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

The fields isltyp, ..., greenfrac are added in the "input" and "restart" streams through inclusion of the "invariant" stream (see line 532 and line 581/593). So I think what we want is to add lai12m to the "invariant" stream.

Copy link
Author

Choose a reason for hiding this comment

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

@mgduda Michael, I am sorry I don't understand this comment. I tried to follow albedo12m. Could you please give me more information or make the change as you see needed?

units="-"
description="The climatological monthly leaf area index dataset to use (case 7 only)"
possible_values="`MODIS'"/>

Copy link
Contributor

Choose a reason for hiding this comment

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

Just a suggestion, but we could move the definition of config_lai_data up around line 177 so that it's nearer to the definition of config_vegfrac_data and generally within the set of options for selecting which datasets to use.

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.

2 participants