ESCOMP / ESCOMP/CTSM

Add dewpoint function to QSat

Open
#3,694 1 comment 0 reactions 0 assignees View on GitHub
b4b code health
Dominant language
Fortran
Stars
352
Forks
361
Avg merge
2d 21h
Merged PRs (30d)
7

Description

We should make dewpoint a little function that's part of QSat. That isolates it's complexity in one place, allows for the possibility of unit testing, or testing specific inputs that might fail, and it's a move to better modularization. Right now dewpoint isn't used anywhere else in the code, but it's a basic enough thing that it might be.

This is likely a refactoring that we should do later. But, I did want to bring it up.

_Originally posted by @ekluzek in https://github.com/ESCOMP/CTSM/pull/3688#discussion_r2692750610_

Definition of Done:
- [x] Make dewpoint into a function part of BargroundFluxes
- [ ] Add unit testing for it
- [ ] Move to QSat

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.