Skip to content

3164 updated itpa confinement scalings itpa20#3166

Merged
timothy-nunn merged 3 commits intomainfrom
3164-updated-itpa-confinement-scalings-itpa20
May 1, 2024
Merged

3164 updated itpa confinement scalings itpa20#3166
timothy-nunn merged 3 commits intomainfrom
3164-updated-itpa-confinement-scalings-itpa20

Conversation

@mkovari
Copy link
Copy Markdown
Collaborator

@mkovari mkovari commented Apr 30, 2024

ITPA scaling

Checklist

I confirm that I have completed the following checks:

  • No change in any tests as they don't use the new model.
  • I have NOT added new tests as I am not sure how.where appropriate for the changes I have made.
  • No change to any tests.
  • Doc change made and previewed.

Michael Kovari and others added 3 commits April 30, 2024 15:28
…b.com:ukaea/PROCESS into 3164-updated-itpa-confinement-scalings-itpa20
@mkovari mkovari linked an issue Apr 30, 2024 that may be closed by this pull request
Comment thread process/physics.py
@chris-ashe chris-ashe added the Physics Relating to the physics models label Apr 30, 2024
Copy link
Copy Markdown
Collaborator

@chris-ashe chris-ashe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am happy enough with the changes. Results are output correctly and the docs are updated

@chris-ashe
Copy link
Copy Markdown
Collaborator

@j-a-foster Can you please check also

Copy link
Copy Markdown
Collaborator

@j-a-foster j-a-foster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm happy with the changes and the output is correct. Ready for merge.

@timothy-nunn timothy-nunn merged commit 7aa8375 into main May 1, 2024
@timothy-nunn timothy-nunn deleted the 3164-updated-itpa-confinement-scalings-itpa20 branch May 1, 2024 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Physics Relating to the physics models

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Updated ITPA confinement scalings: ITPA20

4 participants