Conversation
|
Your What do you think @parkjk ? |
|
Any progress here @d-burg? |
|
Status @d-burg? |
|
@d-burg this seems to have gone stale. Let's get it merged or closed as part of your current PR push |
|
@logan-nc should we keep this on the backburner? With my latest changes to this branch (and without TJ's Fitzpatrick-specific flux coordinate "r") I still haven't been able to get the outputs looking correct. However, given how well the STRIDE and TJ Δ' values align, and how close the growth rates are on circular equilibria, I'm not sure if we need this symmetrization? This might be a good question for a future meeting with Dylan or Richard, if we can catch him at MHDW. |
|
Sounds good @d-burg. change the pr status from "open" to "draft" please. if/when you come back to it you can change it back |

The procedure outlined in
sym.ffollows the procedure written by Fitzpatrick through equation (16) (see: Benchmark.pdf). On this first pass and as far as I can tell, I've implemented all GPEC equilibrium quantities, integrals, and matrix multiplies correctly – however, the resulting matrices do not yet follow the desired symmetry properties.One question for now: does this look like I'm calculating the J(psi_N) contour integral correctly? I'm also not totally sure about the need for
jacin this case.GPEC/stride/sym.f
Lines 148 to 165 in 4568a5c
And in the meantime, I'll continue checking through the rest of the algebra (f_S, f_L, etc) for any issues.