Chris Smiet
Chris Smiet
It would be amazing to have all of the QUASR configurations accessible with just one command. I implemented a simple interface using the 'requests' library, that downloads any QUASR field....
The publish to pypi action in our CI is broken, PyPi requires a verified email address nowadays. See the logs in the [action](https://github.com/PrincetonUniversity/SPEC/actions/runs/9677267706/job/26698520100). @zhucaoxiang you made this action, do you...
In issue #449 a change was made to the resolution of F2py to support python2 builds and prefer the versioned `f2pyx` over an unversioned. https://github.com/scikit-build/scikit-build/blob/5b20908709070e6329348aabbb889bacc25ca892/skbuild/resources/cmake/FindF2PY.cmake#L69 This made sense at the...
I see that the `FortranModule` class is implemented with `__metaclass__=Singleton`, such that only one instance can be created. This creates some issues, if on the python side I want to...
Added documentation on how `simsopt` interfaces with the MHD codes `SPEC` and `VMEC`.
surface change_resolution method causes segmentation fault on subsequent evaluation of CPP methods
When a `Surface` object's resolution is changed by the `change_resolution` method its `sopp.Curve` methods will access unallocated memory and cause segmentation faults. MWE: ```python from simsopt.geo import SurfaceRZFourier NFP =...
There are many cross-imports that could lead to difficulties for future maintainability. This is especially in the `simsopt.field` and `simsopt.mhd` modules. The CoilSet class that I wrote is especially egregious...
The QfmSurface class is extremely useful for generating surfaces for magnetic fields, but it stumbles on moderately shaped surfaces. I believe this is due to the lack of enforcement of...
I wanted to add a toroidal field to a configuration, because I just wanted to jiggle the rotational transform a bit without too much ado. That would be nice with...
Noticed that the ` test_meansquaredforces_taylor_test` sporadically fails. These were added for the coil force PR #450, can you have a look @smhurwitz ? See for example [this test]( https://github.com/hiddenSymmetries/simsopt/actions/runs/11435741765/job/31811730030?pr=438) Error...