Dario Panici
Dario Panici
I think I will do it separate for now. That branch still has some things that have to be fixed it seems and it requires this to be done anyways
I have a PR I think addressing this now. Basically, - For "desc.geometry.surface.FourierRZToroidalSurface" , the radial derivative is undefined (is a constant rho surface so has no notion of how...
Put in docstring of `FixSumCoilCurrents`
Worth thinking of what the center/normals need to be (ideally we want the normals to be allowed to be different, I think the centers probably have to be the same?
@dpanici @kianorr I think this only works if the normal vectors are the same (i.e. the coil is planar), am I remembering this right? is this constraint of connected curves...
Also for grid resolution, we should make sure the default plot grid for computing always has sufficient resolution for the given equilibrium/object
Add decorator to __init__ (and build too) then running on GPU and comparing the profile of time with vs without the decorator
I think the point here is, even for a well behaved omni field, the max |B| point in $(\theta_B, \zeta_B)$ space might be misaligned with that of the equilibrium when...
Could we add a check to the Omnigenity objective to check if the initial eq's B max is at theta_booz=0? @ddudt
Add ability to shift B values after transformation to match where eq's Bmax is