Rename q to better represent its meaning
#3106
Closed
chris-ashe
started this conversation in
Ideas
Replies: 3 comments
-
|
So there shouldn't be one "q" variable, and have unique ones for each definition? |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Can we close this discussion now @chris-ashe ? |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Closed with #3108 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
This is a follow on from #321 which did a quick variable rename to make the definition of
qclearer in more places.The definition of
qdoes not initially give the impression that it is the safety factor near the edge of the plasma but is instead a function of plasma radius.Ideas about renaming
qto a different variableqedgeetc are possible though will make old input files temporarily broken until they are all updated. This definition also changes meaning depending on what model are being ran; stellarator/plasma current scalings etc.Beta Was this translation helpful? Give feedback.
All reactions