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

Improving cable_radiation.F90 #267

Open
penguian opened this issue Sep 2, 2020 · 2 comments
Open

Improving cable_radiation.F90 #267

penguian opened this issue Sep 2, 2020 · 2 comments

Comments

@penguian
Copy link
Collaborator

penguian commented Sep 2, 2020

keyword_nogit owner:[email protected] type_model improvement | by [email protected]


cable_radiation.F90:
: cbl_init_radiation.F90
: cbl_radiation.F90
: cbl_rhoch.F90
: cbl_sinbet.F90
: cbl_spitter.F90

In JAC, the surface albedo is required before CABLE has more generally been
initialized. Closer inspection shows that we can compute the surface albedo given
only a few veg and soil parameters - So long as we move to accessing
whether a cell is sunlit or not given the zenith angle rather than downward SW,
which in a circular fashion precluding calculation of the surface albedo has not
yet been determined, requiring surface albedo in its determination.
Closer inspection also reveals that this pathway to get the surface albedo is
very un-neccessarily convoluted and hard to follow with non-descriptive varible
names and a haphazzard order of operation.
See Ticket #267 as well

The first step to achieving the ultimate goal of calcculating the surface albedo
as efficiently as possible will be to unclutter the existing radiation_module as it
is required by "albedo" code; by first separating the single cable_radiation.F90
into subroutines per pe modules, files.

The "init_radiation" routine in particular is re-written to carify what is actually
needed and when.


Issue migrated from trac:267 at 2023-11-27 11:34:31 +1100

@penguian
Copy link
Collaborator Author

penguian commented Feb 2, 2023

@[email protected] commented


Close?

@penguian
Copy link
Collaborator Author

penguian commented Feb 7, 2023

@[email protected] set keywords to nogit

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

2 participants