Dario Panici
Dario Panici
@unalmis I merged master, gonna see which tests fail but I think there needs to be some extra logic added to `"A"` and `"A(z)"` to prevent `ZernikeRZToroidalSection` objects from getting...
couple omega to theta pest somehow?
- lower L of omega basis but keep M,N large enough to match boundary - penalty on omega amplitude potentially (but would rather not have it be an explicit penalty)
@dpanici double check this. by computing volume of torus or something, try lower L nodes, recheck old notes
> @dpanici double check this. by computing volume of torus or something, try lower L nodes, recheck old notes or just integrate R
Rory is right, here is finding the average R by integrating over a quad grid for DSHAPE, we hit the floor at L=M=14 but the default 0D grid is at...
@dpanici do this, make nodes created with L/2 not L
> I think the following files also have to be updated to stay self-consistent with this change: > > * input.HELIOTRON_desc_no_continuation > * input.HELIOTRON_desc_no_continuation_output.h5 > * basic_equilibrium.ipynb > * advanced_equilibrium_continuation.ipynb...
> > Feels really weird that going from m=8 to m=9 makes such a big difference? > > @dpanici are you sure this PR is necessary? The notebooks on master...
> Feels really weird that going from m=8 to m=9 makes such a big difference? I also agree but this is the result I see, if the point of the...