Repository navigation
Addition of an evaluation regression file to test running of nbi model and secondary current drive - #4725
Addition of an evaluation regression file to test running of nbi model and secondary current drive#4725kj5248 wants to merge 1 commit into
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4725 +/- ##
=======================================
Coverage 50.26% 50.26%
=======================================
Files 181 181
Lines 30633 30633
=======================================
Hits 15397 15397
Misses 15236 15236 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
| alphan = 0.9 * Plasma density profile index (⍺ₙ) | ||
| alphat = 1.4 * Plasma temperature profile index (⍺ₜ) | ||
| aspect = 1.8 * Plasma aspect ratio (A) (`iteration variable 1`) | ||
| beta_total_vol_avg = 0.12207750374856029 * Volume averaged total plasma beta (⟨β⟩) (`iteration variable 5`) (calculated if stellarator) |
There was a problem hiding this comment.
| beta_total_vol_avg = 0.12207750374856029 * Volume averaged total plasma beta (⟨β⟩) (`iteration variable 5`) (calculated if stellarator) |
| *---------------------Numerics---------------------* | ||
|
|
||
| i_process_run_mode = -2 * Evaluation mode | ||
| i_figure_merit = -5 * | ||
| n_equality_constraints = 3 * number of equality constraints to be satisfied | ||
| epsvmc = 1e-9 * Error tolerance for optimiser |
There was a problem hiding this comment.
Can you move this section to the top?
| icc = 1 * | ||
| icc = 2 * | ||
| icc = 11 * |
There was a problem hiding this comment.
Probably worth commenting that the constraints below here are not enforced but are kept "on" so that they get reported in the OUT.DAT
| dx_tf_hts_tape_hastelloy = 1e-5 * thickness of Hastelloy layer in tape (m) | ||
| dx_tf_croco_strand_copper = 2.0e-3 * Thickness of CroCo strand copper tube (m) (`iteration variable 158`) | ||
|
|
||
| *-----------------Tfcoil Variables-----------------* |
There was a problem hiding this comment.
How easy would it be to have this file use a different TF coil model? E.g. a resistive TF. This is another huge area of the code that is untested
| p_hcd_injected_max = 225.0 * maximum allowable value for injected power (MW) (`constraint equation 30`) | ||
| p_hcd_secondary_injected_mw = 10 * secondary total fixed auxiliary injected power (MW) | ||
|
|
||
| *------------------Dcll Variables------------------* |
There was a problem hiding this comment.
Could this file also maybe use the DCLL instead of another blanket model? Again, this untested and would be useful!
chris-ashe
left a comment
chris-ashe
left a comment
There was a problem hiding this comment.
I have noticed aswell that the beam outline for the top down toroidal cross-section does not show when the NBI is secondary here
|
We can absolutely add DCLL blankets and resistive magnets to this file. But I know users do sometimes uses the files in the regression tests as starting points or examples input files, and this will create an odd frankentechnology machine. I think that is fine, but is worth highlighting. |
@ajpearcey thanks for pointing this out. Maybe we keep this file as is and make some more eval files which have more realistic configurations that use these models? |
Closes #4062
Description
Added in a eval regression file that will use secondary current drive and an nbi model as those are both currently untested.
Checklist
I confirm that I have completed the following checks: