Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Objective for coil/coil distance #898

Closed
Tracked by #496
f0uriest opened this issue Feb 24, 2024 · 1 comment · Fixed by #977
Closed
Tracked by #496

Objective for coil/coil distance #898

f0uriest opened this issue Feb 24, 2024 · 1 comment · Fixed by #977
Labels
coil stuff relating to coils and coil optimization

Comments

@f0uriest
Copy link
Member

No description provided.

@f0uriest f0uriest mentioned this issue Feb 24, 2024
Closed
9 tasks
@f0uriest f0uriest added the coil stuff relating to coils and coil optimization label Feb 24, 2024
@dpanici
Copy link
Collaborator

dpanici commented Apr 2, 2024

I have a version of this here

@ddudt ddudt mentioned this issue May 15, 2024
Merged
4 tasks
dpanici added a commit that referenced this issue Jun 26, 2024
@dpanici
Coil-Coil & Plasma-Coil Minimum Distance Objectives (#977)
87db5be 
- Adds the objective `CoilsetMinDistance`, which returns the minimum
distance to another coil for each coil in a coilset. Resolves #898
- Adds the objective `PlasmaCoilsetMinDistance`, which returns the
minimum distance to the plasma surface for each coil in a coilset.
Resolves #900, #947

Dependencies: 
- #1016 
- #1017 
-  #1018 

TODO: 
- [x] unit tests for `CoilsetMinDistance`
- [x] unit tests for `PlasmaCoilsetMinDistance`
- [x] regression test using both objectives in a coil optimization
- [x] make it work with surface and/or equilibrium to resolve #947
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
coil stuff relating to coils and coil optimization
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Coil-Coil & Plasma-Coil Minimum Distance Objectives PlasmaControl/DESC
2 participants
@f0uriest @dpanici