Skip to content

Add NT confinement scaling - #95

Open
nelsonand wants to merge 2 commits into
cfs-energy:mainfrom
nelsonand:NT24_scaling
Open

nelsonand wants to merge 2 commits into
cfs-energy:mainfrom
nelsonand:NT24_scaling

Conversation

@nelsonand

Copy link
Copy Markdown
Contributor

Simple addition to the confinement scalings to incorporate the NT D3D 24 scaling from https://iopscience.iop.org/article/10.1088/1741-4326/ad69a4

@nelsonand

Copy link
Copy Markdown
Contributor Author

@tbody-cfs can we merge this simple addition into main? Thanks! Let me know if there is anything I need to change!

Comment thread cfspopcon/formulas/energy_confinement/energy_confinement_scalings.yaml Outdated
…ngs.yaml

Co-authored-by: Tom Body <116750897+tbody-cfs@users.noreply.github.com>
@tbody-cfs

Copy link
Copy Markdown
Collaborator

@nelsonand Could you point me to a DOI and equation number for this tau_E scaling?

@nelsonand

nelsonand commented Aug 28, 2024

Copy link
Copy Markdown
Contributor Author

@tbody-cfs for some reason the scaling isn't actually provided in the paper... but the data and description, as well as a plot of the scaling, are here: https://iopscience.iop.org/article/10.1088/1741-4326/ad69a4/meta

@tbody-cfs

Copy link
Copy Markdown
Collaborator

@nelsonand what is the expected time for getting a citation for the scaling?

@nelsonand

Copy link
Copy Markdown
Contributor Author

@tbody-cfs Likely within a week or two

@tbody-cfs tbody-cfs changed the title Add NT confinement scaling On hold: Add NT confinement scaling Sep 11, 2024
@tbody-cfs tbody-cfs added the blocked A PR is blocked by another PR label Sep 11, 2024
@tbody-cfs tbody-cfs changed the title On hold: Add NT confinement scaling Add NT confinement scaling Sep 11, 2024
@tbody-cfs

Copy link
Copy Markdown
Collaborator

@nelsonand Great! I'll mark this PR as blocked for now, and we'll put it in as soon as that reference is available

@tbody-cfs

Copy link
Copy Markdown
Collaborator

@nelsonand Any updates here?

@nelsonand

Copy link
Copy Markdown
Contributor Author

@tbody-cfs blast from the past! The citation for this should be https://arxiv.org/abs/2509.04279

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked A PR is blocked by another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants