CliMA / SurfaceFluxes.jl

A package for computing surface fluxes between the atmosphere, ocean and land models.
https://clima.github.io/SurfaceFluxes.jl/dev/
Apache License 2.0
16 stars 3 forks source link

Add physical tests for surface fluxes #73

Open szy21 opened 1 year ago

szy21 commented 1 year ago

Purpose

Add physical tests for surface fluxes based on Monin-Obukhov similarity theory

Cost/Benefits/Risks

cost: Time of a few (2) atmos team members benefits: Surface fluxes needs some substantial refactoring, mostly for EDMF and also for the land. Having physical tests can ensure we don't break things (as we did a few times before!) risks: We are designing our own tests as there are not many existing tests that we can follow

People and Personnel

(@ the personnel required/requested to implement the software solution)

Components

see task breakdown

Inputs

N/A

Results and deliverables

Tests that are run in ci

Task Breakdown And Tentative Due Date

Proposed Delivery Date

Nov 18, 2022 (We can prioritize it if needed by EDMF)

tapios commented 1 year ago

LGTM. However, please note that the momentum flux can have either sign (depending on sign of velocity);

szy21 commented 1 year ago

Thanks for adding in some of the tests. @akshaysridhar @LenkaNovak What is the timeline for the rest of the tests? No hurry, I just want to keep the delivery date up-to-date. I know that instead of following the paper you plan to use era data - please feel free to change the SDI too!

LenkaNovak commented 1 year ago

Akshay already has a qualitative comparison with ERA-5, but we're currently exploring whether we can easily extend this to use surface data instead of data on a pressure level, for an even better agreement. I'll defer to @akshaysridhar for the timelines, since he's leading the effort.