ESCOMP / ESCOMP/CAM

Cleanup - replacing log-gamma function with F2008 intrinsic in WACCMX code

Open
#1,080 0 comments 0 reactions 1 assignee Claimed by @briandobbins View on GitHub
Dominant language
No language data
Stars
91
Forks
183
Avg merge
6d 2m
Merged PRs (30d)
9

Description

### Issue Type

Code Clean-up

### Issue Description

We had an older, outdated way of computing a log-gamma value. This switches it to use the F2008 intrinsic.

I ran one test with the change and it passed, but I've held off from all tests until Francis or someone else signs off, since this is WACCMX code. Mathematically, the function and intrinsic should be the same, but I could envision there being small numerical differences resulting from this.

### Will this change answers?

I Don't Know

### Will you be implementing this yourself?

Yes

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.