TokaMaker: Add support for grouping coils into coil sets #38
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request adds support for grouping one or more coil regions into a set that is viewed as a single coil in TokaMaker, including:
gs_Domain
meshing interfaceTokaMaker.coil_sets
dictionary which defines coils as seen by*_coil_currents()
and other coil-related functionalityset_coil_currents()
,get_coil_currents()
,set_coil_reg()
, etc.Note: When coil sets are not specified a single coil set per coil region with the same name is created. When the number of turns in a region is not specified it defaults to 1.
In addition the following changes were made to the core code:
TokaMaker.Tokamaker.set_coil_reg()
)grad_shaf::gs_analyze_saddles()
**Note:**
to@note
This pull request does modify the following TokaMaker APIs.
TokaMaker.gs_Domain.define_region()
now takes acoil_set
argumentTokaMaker.TokaMaker.setup_regions()
now takes acoil_dict
argument for defining coil regionsTokaMaker.TokaMaker.get_coil_currents()
now returns the region-wise current as the second argumentThis pull request does not modify any input files