Repository navigation
STEP TF turn structure - #4643
chris-ashe wants to merge 20 commits into
Conversation
…or model handling and improve comments
… related calculations
…mproved clarity and accuracy
…ingTFCoil for accurate critical current calculations
…nboard_peak_ripple_symmetric in SuperconductingTFCoil
…tions in STEPSuperconductingTFCoil
…add tf_cicc_areas_and_masses method for improved accuracy
…add tf_step_inboard_areas_and_fractions method for improved accuracy
…culations in CROCOSuperconductingTFCoil
…erconductingTFCoil to compute superconductor lengths
…essValueError in STEPSuperconductingTFCoil
|
| """Cross-sectional area of the stabiliser in the TF turn [m²]""" | ||
|
|
||
| n_tf_turn_superconducting_strands: int = 0.0 | ||
| """Number of superconducting strands in the TF turn""" |
| # Copper mass [kg] | ||
| self.data.tfcoil.m_tf_coil_copper = ( | ||
| m_tf_coil_copper = ( | ||
| self.data.tfcoil.len_tf_coil |
There was a problem hiding this comment.
When itart == 1 include the centre column length cplen is not included in len_tf_coil. This is accounted for else where or is copper the missing?
There was a problem hiding this comment.
This is true of all the masses in section. I can see some if statements for itart == 1. Also is it useful to list, list the copper masses, without the centre rod included? Are the centre rod masses/area calculated as well, we these will be needed for costing or other engineering models.
…e unit tests for tf_cicc_areas_and_masses
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #4643 +/- ##
==========================================
- Coverage 49.92% 49.86% -0.06%
==========================================
Files 151 151
Lines 30236 30821 +585
==========================================
+ Hits 15094 15369 +275
- Misses 15142 15452 +310 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Description
This pull request adds support for a new "step-stacked tape" type to the superconducting toroidal field (TF) coil modeling, along with the necessary data fields and model registration. The changes ensure the new coil type is selectable, its parameters are tracked, and its model is properly invoked in the workflow.
Support for Step-Stacked Tape TF Coil:
STEPSuperconductingTFCoilto the imports and model registration inmain.py, and included it in the list of models for execution.caller.pyto call the newstep_sctfcoilmodel when theSTEP_STACKED_TAPEturn type is selected.3to thei_tf_turn_typechoices ininput.py.Checklist
I confirm that I have completed the following checks: