refactor(QED): remove erw from sum lemmas - #1504
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.
Looks good - many thanks. Approved
|
Thanks for the review! |
Summary
This PR removes the two remaining uses of
erwfromPhyslib/QFT/QED/AnomalyCancellation/Basic.lean.The affected lemmas are:
PureU1.sum_of_chargesPureU1.sum_of_anomaly_free_linearIn both proofs, the previous
erwrelied on definitional equality while applying the induction hypothesis. The new proofs make that step explicit withchange, instantiate the induction hypothesis directly, and usecongrArgto add the final summand to both sides.No lemma statements or mathematical content are changed.
Validation
lake build +Physlib.QFT.QED.AnomalyCancellation.BasicLEAN_NUM_THREADS=1 lake build(9269 jobs)linter.style.multiGoalandlinter.style.longLinegit diff --check./scripts/lint-style.sherwremains in the modified filelake exe lint_all: build/import/sorry/Lean linters pass; repository-wide pre-existing style and transitive-import issues remain in unrelated filesContributes to #385.