Kaya Unalmis

> I don't think cvdrift(binormal drift) zero is any measure of the ripple or a good indicator of the actual ripple (coming from cvdrift0(radial drift)). > > Let me think...

cvdrift0 is not the same as kappa_g, and epsilon effective should definitely use kappa_g. neemov's kappa_g is the same as desc kappa_g.

i haven't changed the 3/2 power on effective ripple because I don't know what to change the name to if I do that

Thank for this. How do these DESC plots change as the DESC equilibrium resolution varies? I'd expect accuracy in effective ripple etc. via a field line integration requires higher DESC...

> also need to add the new objective to docs (or do we want to wait until #1290 for that?) That works. I suggest merging the PRs that are upstream...

> Did we ever run a comparison against WISTELL? Ok, I spent an afternoon on this. Their code errors with singular integral exceptions. Maybe it's something simple. Waiting on their...

Are you sure that DSHAPE should have non positive curvature? tests/test_objective_funs.py::test_mean_curvature is failing

Note to self: - [x] in bounce integrals you used d zeta / B dot grad zeta for field line differential length where the zeta in dzeta denotes the field...

alpha_t should be corrected to data["alpha_t"] = data["theta_PEST_t"] - data["iota"] * data["phi_t"]

Also the formula for alpha uses zeta while its derivatives use phi.