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

Cleanup upw_frac in AdvectionSrc code #1522

Closed
ewquon opened this issue Mar 21, 2024 · 6 comments
Closed

Cleanup upw_frac in AdvectionSrc code #1522

ewquon opened this issue Mar 21, 2024 · 6 comments
Assignees
Labels
cleanup Cleanup code or comments

Comments

@ewquon
Copy link
Contributor

ewquon commented Mar 21, 2024

From erf_slow_rhs_pre, horiz_upw_frac and vert_upw_frac are passed:

  1. to AdvectionSrcForMom
  2. to AdvectionSrcForMomVert_*
  3. to AdvectionSrcForMomWrapper_* (where the InterpType_H and InterpType_V structs are instantiated)
  4. to AdvectionSrcFor*Mom_*
  5. which calls interp_*_h.InterpolateIn* and interp_*_v.InterpolateIn* with upw_frac_h and upw_frac_v, respectively, as arguments.

All the hand offs for step 4 and 5 can be avoided by just instantiating the interp structs in step 3 with the upwind fraction as an argument.

@ewquon ewquon added the cleanup Cleanup code or comments label Mar 21, 2024
@ewquon ewquon self-assigned this Mar 21, 2024
@asalmgren
Copy link
Collaborator

This sounds like a good change

@asalmgren
Copy link
Collaborator

@ewquon -- is this still something we want to do? Or has it been superseded by other changes to advection?

@ewquon
Copy link
Contributor Author

ewquon commented Jan 20, 2025

@asalmgren I'll take another look. While I'm at it, I'll also want to think about how to make the blending more general for Matt's work.

@asalmgren
Copy link
Collaborator

@ewquon -- does your latest PR resolve this issue?

@ewquon
Copy link
Contributor Author

ewquon commented Feb 5, 2025 via email

@asalmgren
Copy link
Collaborator

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Cleanup code or comments
Projects
None yet
Development

No branches or pull requests

2 participants