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

Need to add area correction factors for fluxes in the nuopc cap #69

Open
billsacks opened this issue Nov 10, 2021 · 2 comments
Open

Need to add area correction factors for fluxes in the nuopc cap #69

billsacks opened this issue Nov 10, 2021 · 2 comments
Labels

Comments

@billsacks
Copy link
Member

(For some background, see ESCOMP/CMEPS#84).

For fluxes, we generally need a correction factor between the model's internal areas and the grid cell areas on the mesh file that is used to create the mappings between components. For glc, it's tricky to figure out which fields need these area corrections and which ones don't because of the non-standard way in which some fields are mapped. I think (but am not 100% sure) that we should be doing area corrections for the fluxes sent from glc to other components: the heat flux sent to land, and the runoff fluxes sent to the ocean (or theoretically to the ice model).

I'm pretty sure we do not want to do these area corrections for the other glc -> lnd fields. (In section 3.4 of https://docs.google.com/document/d/144PZ63gwpz875U4EYEI7NuxnF__V955QJOxV9LcEAL0/edit# , I showed why we do not want to do area correction for these (or at least why we don't need to do area correction... I'm not 100% sure that it is a problem to do area correction, but I think it would be).

For lnd -> glc fluxes (currently just smb/qice), with the way we are doing the smb renormalization, I'm pretty sure we do not want to do these area corrections: see this comment:

https://github.com/ESCOMP/CMEPS/blob/5dc55a8913c1687bef673d508d87c300a46c2754/mediator/med_phases_prep_glc_mod.F90#L1011-L1025

@mvertens @whlipscomb @Katetc

@billsacks
Copy link
Member Author

@mvertens @whlipscomb @KateC in reviewing some open issues, this one came to my attention as something needed for CESM3.

@billsacks
Copy link
Member Author

Sorry, meant to tag @Katetc there.

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

No branches or pull requests

1 participant