ESCOMP / ESCOMP/CMEPS

Calling shr_wv_sat_init() from mediator rather than driver?

Đang mở
#653 5 bình luận 0 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
Fortran
Star
35
Fork
106
Merge trung bình
6 ngày 12 giờ
Pull request đã merge (30 ngày)
5

Mô tả

The [`shr_wv_sat_init()`](https://github.com/ESCOMP/CMEPS/blob/a0343bcb7c6016960facd6d7bb124223013446bd/cesm/driver/esm.F90#L526) call is required to to use [`shr_wv_sat_qsat_liquid()`](https://github.com/ESCOMP/CMEPS/blob/a0343bcb7c6016960facd6d7bb124223013446bd/cesm/flux_atmocn/flux_atmocn_Large.F90#L219) routine and similar ones to calculate the saturation mixing ratio from the tables but the current implementation assumes that the CMEPS flux calculation will be only used by CESM and its driver. Since we are coupling MPAS and MOM6 outside of the CESM framework (at this point) through ESMX driver, the [`shr_wv_sat_qsat_liquid()`](https://github.com/ESCOMP/CMEPS/blob/a0343bcb7c6016960facd6d7bb124223013446bd/cesm/flux_atmocn/flux_atmocn_Large.F90#L219) call returns all zero and flux calculation is messed up. As I see those codes are under driver and it makes sense but I think moving those outside of the driver code and init through the `med.F90` rather then `esm.F90` would make it more generic and it could be also used by UFS WM (I did not check their driver has this init call or not yet) and other projects like ours.

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.