-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Stellarator symmetry available for testing #46
Comments
Compiling at MPCDF on Draco I found one compilation bug
|
@lazersos Good catch. I changed that part after my test and didn't see the compilation error. I have pushed a fix. |
OK compiles now on Draco at MPCDF. Thanks. |
This really needs to work with coil_type=3 for it to be useful for me. |
The symmetry should work with cil_type =3, but the toroidal flux doesn't, as the vector potential of an infinite wire is infinite. |
There's an error check which prevents using cil_type=3. |
Yes, because I realize that right now using the vector potential can only handle the Fourier coils. You can mute then if you really want the inaccurate evaluation of toroidal flux. |
@lazersos I think I misunderstood you. I thought the error check you were talking about was in |
I am going to merge the branch |
For a long time, FOCUS ignores periodicity and stellarator symmetry, as the computation speed was never an issue and generally, FOCUS results followed periodicity and stellarator symmetry if the target plasma does. I implemented the periodicity some time ago. A recent presentation by J. Schmitt indicates that finite asymmetry in coils may cause sizable differences in Boozer harmonics. The results motivate me to really consider finishing the stellarator symmetry.
Now the new function is ready for testing in the branch
symmetry
. Anyone who is interested should take a look. I will merge it intodevelop
(andmaster
) after the recent pull request is merged.Some key points:
develop
branch, which means it has the ability to specify a secondary limiter surface by usinglimiter_surf
when using coil-surface separation. More details can be checked Separate plasma surface with limiter surface for coil-surface separation AND stellarator symmetry #42.*.focus
file for coil parameters now has a slightly different I/O. An additional parametersymm
will be read to indicate the symmetry property of this particular coil. Old executables can still read this new format, but the new executable cannot read old formats.There is a detailed developing note available at https://princetonuniversity.github.io/FOCUS/notes/FOCUS_stellarator_symmetry_and_periodicity.html. Please take a look. Some key information: Section 1.4 How to use, Section 3 Summary.
Please feel free to test and report issues.
The text was updated successfully, but these errors were encountered: