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: FV::Div_a_Grad_perp operator #2823

Open
wants to merge 1 commit into
base: next
Choose a base branch
from
Open

Conversation

bendudson
Copy link
Contributor

Going through this operator (again). Reviewing how metric elements are interpolated to cell edges.

To do:

  • Tests
  • Documentation

Near the X-point the metrics g23, g_23 and g_22 all become
large with a 1/Bpol dependence. The combination that appears
in the perpendicular diffusion operator, g23 * g_23 / g_22,
remains well behaved. This attempts to evaluate combinations of
metric elements at the same location, rather than interpolating
and then combining.
@dschwoerer
Copy link
Contributor

Failing test should be fixed by #2824

@mikekryjak
Copy link
Contributor

Thanks for this @bendudson. I've put this through our puff test. For everyone else, we are facing a very tough to debug problem where our neutrals are spontaneously increasing in temperature and decreasing in density around the X-point. We have ruled out a lot of things and our current suspect is the operator. We came up with a test where we take a simplified model with a fixed plasma background of constant temperature and density and impose a neutral source near the X-point. We then observe how the diffusive transport behaves, particularly how the density falloff evolves with distance from the puff source.

I am seeing no difference with this new operator. There is a bifurcation near the source which is identical in both cases. The bifurcation can be seen near X=0 in the RHS plot we see several points of similar distance but different density. It may be that this is due to cell size after all, i.e. because the cell value represents the average across the entire cell, meaning that a larger cell covering a greater distance from the puff source would have a lower density value (I think?).

Old results:
image

New results:
image

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.

3 participants