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

Polar Mitigations are not compatible with splitting phi #138

Open
bprather opened this issue Mar 7, 2025 · 0 comments
Open

Polar Mitigations are not compatible with splitting phi #138

bprather opened this issue Mar 7, 2025 · 0 comments

Comments

@bprather
Copy link
Contributor

bprather commented Mar 7, 2025

KHARMA commonly mitigates instability around the pole by averaging the EMF at the pole, B3 around the pole, or U3 around the pole. These averages were written when I was primarily using transmitting polar boundary conditions, and considered transmitting conditions to be the primary stability concern. Since transmitting polar boundary conditions currently require the mesh have only a single block in $\phi$ at the pole, the mitigations were written with that assumption as well.

Some folks have started using mitigations in runs with reflecting polar boundaries, to stabilize the combination of Kastaun inverter and face-centered B at the polar boundary. At the least, KHARMA should check for these mitigations if $\phi$ is split and exit with an error. Ideally, the necessary reductions could be implemented by inferring the mesh structure about the pole from Parthenon's mesh location & numbering scheme. Averages could then be computed over the whole range, and resulting values applied uniformly to prevent issues.

@bprather bprather changed the title Polar Mitigations are not compatible with splitting $\phi$ Polar Mitigations are not compatible with splitting phi Mar 7, 2025
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