Skip to content

Correct constraint 52 docstring #4072

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

Merged
je-cook merged 2 commits into from
Jan 30, 2026
Merged

Correct constraint 52 docstring #4072

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ac6c17e
Remove references to tbr outside of IFE
timothy-nunn Jan 29, 2026
2eccf27
Clarify that constraint 52 is only available for IFE
timothy-nunn Jan 29, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 8 additions & 2 deletions process/constraints.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1410,9 +1410,15 @@ def constraint_equation_52():
"""Equation for tritium breeding ratio lower limit
author: P B Lloyd, CCFE, Culham Science Centre

tbr: tritium breeding ratio (i_blanket_type=2,3 (KIT HCPB/HCLL))
tbrmin: minimum tritium breeding ratio (If i_blanket_type=1, tbrmin=minimum 5-year time-averaged tritium breeding ratio)
The tritium breeding ratio is only calculated when using the IFE model.

tbr: tritium breeding ratio
tbrmin: minimum tritium breeding ratio
"""
if data_structure.ife_variables.ife != 1:
raise ProcessValueError(
"Constraint 52 is only supported when running the IFE model"
)
cc = (
1.0
- data_structure.fwbs_variables.tbr / data_structure.constraint_variables.tbrmin
Expand Down
6 changes: 3 additions & 3 deletions process/power.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1618,7 +1618,7 @@ def plant_thermal_efficiency(self, eta_turbine):
New Power Module Harrington Cycle correlations Cycle correlations.xls
"""
if fwbs_variables.i_thermal_electric_conversion == 0:
# CCFE HCPB Model (with or without TBR)
# CCFE HCPB Model
if fwbs_variables.i_blanket_type == 1:
# HCPB, efficiency taken from M. Kovari 2016
# "PROCESS": A systems code for fusion power plants - Part 2: Engineering
Expand All @@ -1630,7 +1630,7 @@ def plant_thermal_efficiency(self, eta_turbine):

# Etath from reference. Div power to primary
elif fwbs_variables.i_thermal_electric_conversion == 1:
# CCFE HCPB Model (with or without TBR)
# CCFE HCPB Model
if fwbs_variables.i_blanket_type == 1:
# HCPB, efficiency taken from M. Kovari 2016
# "PROCESS": A systems code for fusion power plants - Part 2: Engineering
Expand All @@ -1647,7 +1647,7 @@ def plant_thermal_efficiency(self, eta_turbine):

# Steam Rankine cycle to be used
elif fwbs_variables.i_thermal_electric_conversion == 3:
# CCFE HCPB Model (with or without TBR)
# CCFE HCPB Model
if fwbs_variables.i_blanket_type == 1:
# If coolant is helium, the steam cycle is assumed to be superheated
# and a different correlation is used. The turbine inlet temperature
Expand Down
3 changes: 0 additions & 3 deletions process/stellarator.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2148,9 +2148,6 @@ def stfwbs(self, output: bool):
"(f_blkt_li6_enrichment)",
fwbs_variables.f_blkt_li6_enrichment,
)
po.ovarre(
self.outfile, "Tritium breeding ratio", "(tbr)", fwbs_variables.tbr
)
po.ovarre(
self.outfile,
"Tritium production rate (g/day)",
Expand Down
11 changes: 0 additions & 11 deletions tests/regression/input_files/st_regression.IN.DAT
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2251,17 +2251,6 @@ i_blkt_inboard = 0

*~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

*icc = 52
* DESCRIPTION: Constraint equation for tritium breeding ratio lower limit
* JUSTIFICATION: Turned off, do not care about minimum TBR.
* VARIABLES: tbrmin minimum tritium breeding ratio (If i_blanket_type=1, tbrmin=minimum 5-year time-averaged tritium breeding ratio), tbr calculated in situ

*tbrmin =
* DESCRIPTION: Minimum tritium breeding ratio (`constraint equation 52`) (default = 1.1)
* JUSTIFICATION: Not using icc = 52

*~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

inuclear = 0
* DESCRIPTION: Switch for nuclear heating in the coils (default = 0)
* =0 Frances Fox model (default)
Expand Down