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.

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

Remove TBR calculation #3572

Closed
timothy-nunn opened this issue Mar 6, 2025 · 3 comments · Fixed by #3605
Closed

Remove TBR calculation #3572

timothy-nunn opened this issue Mar 6, 2025 · 3 comments · Fixed by #3605
Assignees

Comments

@timothy-nunn
Copy link
Contributor

Remove the Tritium Breeding Ratio model from PROCESS entirely

@stuartmuldrew @chris-ashe

@mkovari
Copy link
Collaborator

mkovari commented Mar 7, 2025

What's this all about?

@stuartmuldrew
Copy link
Collaborator

i_blanket_type=3 uses the i_blanket_type=1 CCFE HCPB blanket model, but extends it to include a TBR approximation based on fitting to models developed by Jonathan Shimwell for EUROfuion-DEMO.

The fitting allows for only three fixed blanket thicknesses set using blanket_thickness, with options THIN, MEDIUM or THICK. This overwrites any blanket thicknesses given in the input.

Discussions with Jonathan Shimwell indicated that this model is outdated and unlikely to be accurate based on latest considerations. It is proposed we remove the i_blanket_type=3 TBR extension and blanket_thickness.

@mkovari
Copy link
Collaborator

mkovari commented Mar 13, 2025

Fair enough.

@chris-ashe chris-ashe linked a pull request Mar 25, 2025 that will close this issue
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants