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

WIP: Operators for non-orthogonal meshes #88

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

bendudson
Copy link
Owner

Including off-diagonal metric terms coming from non-orthogonal meshes. Starting with operators in neutral fluid model as those are likely to have the most impact.

A version of FV::Div_a_Grad_perp that includes all metric components,
including g12 (X-Y non-orthogonal) and g13 (X-Z nonorthogonal) terms.

Aim to test and then merge upstream into BOUT++
Testing, using operator that includes nonorthogonal mesh corrections
@mikekryjak mikekryjak linked an issue Dec 2, 2022 that may be closed by this pull request
@dschwoerer
Copy link
Contributor

I have written a small MMS test and have been running it for the div_a_grad_perp_nonorthog, but it does not seem to converge:

dagpno

For reference, the Delp2 operator:
delp2

I am trying to check zoidberg is correct, so the bug might be in zoidberg, but as Delp2 passes, I thought I mention it ...

The test converges if the grid is orthogonal, but fails otherwise.

@mikekryjak mikekryjak mentioned this pull request May 1, 2024
9 tasks
@mikekryjak
Copy link
Collaborator

@dschwoerer Michael Hardman started with us and will be taking a look at this, do you mind sharing your MMS test?

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

Successfully merging this pull request may close these issues.

Implement proper handling of non-orthogonal grids
3 participants