-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add a conversion back to CAM variables before calling the net. #21
base: CAM-ML
Are you sure you want to change the base?
Conversation
There remains a question of what |
As per my comment above the code here works as is, but I think could be clarified/simplified. We currently pass I think we only need one now that there is no subcycling inside the parameterisation.
Currently we use the The most recent commit 3a7eb8c reduces to a single temperature input to the scheme. |
@paogorman notes that this choice seems correct, but actually we could consider removing the second conversion as it could in-fact hurt performance* compared to some field campaigns, so consider removing? Paul will confirm this and we will then either merge this, or remove the second conversion. *because we are forcing the CAM thermodynamics towards the SAM thermodynamic setup. |
My recollection is that it didn't help, but I can't find it in my notes.
In any event, it would be best to check with/without this conversion using
the newest version of the code for TOGA and probably an aquaplanet too.
…On Wed, Sep 11, 2024 at 10:43 AM Jack Atkinson ***@***.***> wrote:
@paogorman <https://github.com/paogorman> notes that this choice seems
correct, but actually we could consider removing the second conversion as
it could in-fact hurt performance compared to some field campaigns, so
consider removing?
Paul will confirm this and we will then either merge this, or remove the
second conversion.
—
Reply to this email directly, view it on GitHub
<#21 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ARPLW64JI335KZ72XF7QYOTZWBJJBAVCNFSM6AAAAABKPHEL5KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBTHA4DCMRXGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
…r to reduce numerical error in calculation of deltas.
…interpolated to CAM space.
3a7eb8c
to
19c4218
Compare
I have just rebased this back on top of CAL-ML which contains the new NN and moist-dry conversion. Next step will be to run a comparison to see if it has any effect on the Toga II case. |
Add a conversion back to CAM variables before calling the net in order to reduce numerical error in calculation of deltas.
As discussed in #16 this will reduce numerical inaccuracies that could result from the variable conversion schemes and propagate into the tendency calculation.
Closes #16