Dario Panici
Dario Panici
> 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...
>For a future PR: The variable we call `keys` here and in the other util functions below is the same as what we call `names` in the `compute` functions above...
I think `FourierPlanarCurve` should be possibly straightforward to make into a C0 curve. Say we have 2 C0 points, and we have two curves defined by 2 sets of Fourier...
`FixSumCoilCurrents` will only fix the sum of the currents of the coils that have unique params in the coilset. An intended use for this objective is to fix currents in...
Vectorize over `knots` and `intervals` instead of enforcing `knots` and `intervals` , but we need to make `knots` and `intervals` part of `params` possibly (in the compute function definition) See...
If I want to say optimize a modular coilset in the presence of a background `ToroidalField`, currently we would have to do so with a `SumMagneticField` object, passing in the...
It is sometimes nice to see just contour plots instead of filled contour plots, we could have a keyword like `filled` or something that toggles between the two for `plot_2d`
_Originally posted by @dpanici in https://github.com/PlasmaControl/DESC/pull/1153#discussion_r1712110839_
DESC requires JAX now to run (due to `interpax` dependency), so the numpy backend is not really used or relevant anymore. We should remove it then to reduce amount of...
When there are lots of objectives, it is hard to take a glance at the output of an optimization and see how the objectives each changed before/after the optimization. For...