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

CCAPS - use time centred rho in step 4's grav forcing #69

Closed
JOThurgood opened this issue Dec 4, 2018 · 5 comments
Closed

CCAPS - use time centred rho in step 4's grav forcing #69

JOThurgood opened this issue Dec 4, 2018 · 5 comments

Comments

@JOThurgood
Copy link
Owner

In step 4 should probabally calc a time centred rho (from 0.5 * (rho^n + rho^n+1 ) to use in calc of U*

Currently uses rho^n+1 (as advect_dens is called prior, which updates the rho array to the new time level).

Should test and then (if good) implement in CCAPS, CCAPS_vardens and CCAPS_atm_const

@JOThurgood
Copy link
Owner Author

Now done in CCAPS_atm_const - carry over to others when get a chance.

@JOThurgood
Copy link
Owner Author

Now done in CCAPS_atm_const - carry over to others when get a chance.

Did you mess it up - looks like used rho0(iy) (i.e. hydrostatic rho at the start of the step) rather than just the rho + perturbations or perturbations (whatever is appropriate)

Check this!!!!

@JOThurgood
Copy link
Owner Author

This might create problems with corrections (which use the end rho?)

@JOThurgood
Copy link
Owner Author

Now done in CCAPS_atm_const - carry over to others when get a chance.

Did you mess it up - looks like used rho0(iy) (i.e. hydrostatic rho at the start of the step) rather than just the rho + perturbations or perturbations (whatever is appropriate)

Check this!!!!

Must have been tired - it looks fine.... At least, for the choice on the coefficient in-front of the gravitational source.

There is a question over what factor of beta/rho (in atmosphere case, or 1/rho otherwise) to use in front of the lagged pressure gradient. Since grad p is lagged, should you use a similarly lagged density? Or should we use the time-centred density? What currently happens is the density at n+1 is used.

@JOThurgood
Copy link
Owner Author

This is essentially issue #115 but for a specific part of the code. There are other places where it might be appropriate to use rho(n+1/2) etc. Closing, see that issue

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

1 participant