Skip to content
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

Adjoint runs fail with salt/tempVertAdvScheme = 33 #39

Closed
emmomp opened this issue Feb 3, 2020 · 3 comments
Closed

Adjoint runs fail with salt/tempVertAdvScheme = 33 #39

emmomp opened this issue Feb 3, 2020 · 3 comments

Comments

@emmomp
Copy link

emmomp commented Feb 3, 2020

Adjoint runs fail during the forward portion of the run (i.e. during calculation of the cost function) with CALC_R_STAR errors when I tried a run with salt/tempAdvScheme=33 and didn't specify the VertAdvScheme (set to 3 in ECCO setup). Not sure if this is expected behaviour. Cost function is a simple global temperature sum.

Background: I've spun up a version of the LLC grid forced by CORE2 normal year fields (setup verifications/global_oce_llc90/) which has various differences in the data namelists when compared to the ECCO standard setup. Adjoint runs fail with current options. By comparison with the ECCO parameters and several test runs I determined that it is the differing vertical advection schemes causing the failure. Specifying vertAdvScheme=3 leads to a successful run.

@emmomp
Copy link
Author

emmomp commented Feb 3, 2020

p.s. also if this is the wrong place to raise this issue please let me know!

@gaelforget
Copy link
Member

p.s. also if this is the wrong place to raise this issue please let me know!

This and the @MITgcm issue tracker seem like the best place to get help on this. I cross referenced it in MITgcm/MITgcm#329 in case someone on that side might be able to jump in and might not follow this org

@emmomp
Copy link
Author

emmomp commented Feb 10, 2020

Apologies - this turns out not to be an Adjoint issue. Any forward run fails similarly with salt/tempAdvScheme=33 and salt/tempImplVertAdv=.TRUE. It runs successfully with vertAdvScheme=3. This may be a feature of the implicit vertical advection, there is no documentation I could find. Opened as MITgcm/MITgcm#331

@emmomp emmomp closed this as completed Feb 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants