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

potential issue with neutral diffusion diagnostics when multiple iterations per timestep occur #1557

Open
klindsay28 opened this issue Feb 7, 2022 · 3 comments
Labels

Comments

@klindsay28
Copy link
Contributor

The post_data calls for neutral diffusion diagnostics occur within the subroutine neutral_diffusion. This subroutine is called from subroutine tracer_hordiff inside a sub-cycling iteration loop that is there to avoid violating diffusive CFL limits. If there are multiple iterations because of CFL limits, then there are multiple calls to post_data per timestep for these diagnostics.

Note that advection diagnostics are handled differently. While advection can be sub-cycled, the advection diagnostics are accumulated across these iterations, and the post_data calls are outside of the iteration loop.

The potentially multiple calls per timestep to post_data for neutral diffusion diagnostics result in multiple calls per timestep to FMS's send_data. Does FMS handle multiple send_data calls per timestep for individual diagnostics?

The same question applies to diagnostics for lateral boundary mixing, whose implementation closely follows the neutral diffusion implementation.

@Hallberg-NOAA
Copy link
Collaborator

No, FMS does not handle multiple post_data calls per timestep gracefully, so it is up to the MOM6 code to accumulate the net transports across multiple iterations. The construct noted with this issue is a bug and should be fixed.

@adcroft adcroft added the bug label Mar 14, 2022
@Hallberg-NOAA
Copy link
Collaborator

@klindsay28, are there ongoing efforts to fix this via the dev/ncar fork, or should we try to organize someone to address this bug?

@klindsay28
Copy link
Contributor Author

@Hallberg-NOAA, I am working on this on the dev/ncar fork.
Thanks for asking.

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

No branches or pull requests

3 participants