refactor(QED): name odd anomaly-cancellation planes - #1502
Conversation
|
Thank you for this pull-request (PR). If this is your first PR, welcome to the community! Below is what will happen next. Please read carefully if you are not familiar with the process. You may open other PRs while this one is being reviewed, and can stack PRs on top of each other, so don't let these steps slow you down.
Tip: The easiest way to get have a fast review is to submit a PR that is small and self-contained, and has clear documentation explaining why things are the way they are in your chages. If you have any problems or questions, please reach out to the community on the Zulip. |
jstoobysmith
left a comment
There was a problem hiding this comment.
This looks great! Many thanks for this first PR. I have approved, once the workflows finish, I will merge.
Summary
This PR addresses the odd-case naming part of #1046 by replacing the non-descriptive
basis/basis!andP/P!distinction with theUnshiftedandShiftedterminology discussed in the issue.The two planes are now organized as:
Unshifted: the plane usingoddFst/oddSndShifted: the plane usingoddShiftFst/oddShiftSndThe main public names are now:
Unshifted.basisAsCharges/Shifted.basisAsChargesUnshifted.basis/Shifted.basisUnshifted.planeCharges/Shifted.planeChargesUnshifted.planeLinSols/Shifted.planeLinSolsAssociated component, ACC, value, and linear-independence lemmas were moved into the corresponding namespaces and renamed consistently. References in
Odd/LineInCubic.leanandOdd/Parameterization.leanwere updated accordingly.No mathematical statements or proofs are intended to change; this PR is a naming/organization refactor only. File splitting and the even case are intentionally left for follow-up work.
Files changed
Physlib/QFT/QED/AnomalyCancellation/Odd/BasisLinear.leanPhyslib/QFT/QED/AnomalyCancellation/Odd/LineInCubic.leanPhyslib/QFT/QED/AnomalyCancellation/Odd/Parameterization.leanValidation
lake buildgit diff --checklake exe lint_all --fast(no lint errors in the modified Odd files; repository-wide pre-existing style issues remain elsewhere)./scripts/lint-style.shsorryoraxiomPart of #1046.