|
| 1 | +# Questions for the morning |
| 2 | + |
| 3 | +1. **PR #408 review-package sharing worked?** (You flipped both; reviewers should confirm access.) |
| 4 | +2. **Certified-grid default**: kinetic_grid_tol is committed default-OFF (0). The B2 sweep will |
| 5 | + inform a recommended value (likely 1e-3); do you want it ON by default in the kinetic example |
| 6 | + decks, or opt-in only until it has survived a real DIII-D kinetic workflow? |
| 7 | +3. **Kinetic reference truth**: Solovev-calculated et[1] moves ~1e-3 between m64 and m256 and the |
| 8 | + ladder is non-monotonic — there is no converged reference in the examples. Is there a case you |
| 9 | + trust as kinetic ground truth (PENTRC benchmark?) worth wiring in before we claim accuracy wins? |
| 10 | +4. **G/H aliasing**: Kinetic/G vs Ideal/G/H statistics are identical in the h5 at every size — |
| 11 | + expected (small corrections on a shared large scale), or worth a look? |
| 12 | +5. **Scope check**: certification currently applies only to kinetic_source="calculated". The |
| 13 | + "fixed" test-matrix path stays on the equilibrium grid. OK? |
| 14 | +6. **q=5 Δ′ non-convergence** (out of scope per your earlier call) — still parked; unchanged. |
| 15 | + |
| 16 | +7. **Kinetic harness deltas vs develop are LARGE and (pending attribution) mostly inherited from |
| 17 | + #398's Solovev geometry fix**: et[1] shifts of 1.7-2.5%, Im parts up to 23%, NTV torque |
| 18 | + magnitude change, and — striking — **NTV ψ-quadrature evaluations 840 → 60**: the torque |
| 19 | + quadrature appears to have been spending 14× the evaluations resolving geometry noise. An |
| 20 | + attribution harness (vs the #398 branch instead of develop) is running/complete — see |
| 21 | + attr_regress.log. Physics judgment needed: are the kinetic-case changes acceptable as |
| 22 | + "the geometry-noise fix propagating into resonance-sensitive quantities" (my reading), and |
| 23 | + should the kinetic harness baselines simply be re-baselined once #398 lands? |
| 24 | +8. **Certified-grid PR**: B1 is committed on the #408 branch (default OFF, knob-off bit-identical), |
| 25 | + but arguably belongs in its own stacked PR per the plan ("kinetic work lands as its own PR |
| 26 | + stacked after Phase A's"). I did NOT split it out overnight — say the word and I'll |
| 27 | + cherry-pick 2225813fc onto a new branch stacked on #408 and open the PR with its review |
| 28 | + package, or keep it in #408 if you prefer fewer moving pieces. |
0 commit comments