Skip to content

WIP: FV::Div_a_Grad_perp operator#2823

Open
bendudson wants to merge 1 commit into
nextfrom
next-div-a-grad-perp
Open

WIP: FV::Div_a_Grad_perp operator#2823
bendudson wants to merge 1 commit into
nextfrom
next-div-a-grad-perp

Conversation

@bendudson

Copy link
Copy Markdown
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
Copy Markdown
Contributor

Failing test should be fixed by #2824

@mikekryjak

Copy link
Copy Markdown
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