Section 9.3: remove unnecessary AdherentPt hypotheses and fix statements#484
Open
rkirov wants to merge 1 commit intoteorth:mainfrom
Open
Section 9.3: remove unnecessary AdherentPt hypotheses and fix statements#484rkirov wants to merge 1 commit intoteorth:mainfrom
rkirov wants to merge 1 commit intoteorth:mainfrom
Conversation
rkirov
commented
Apr 11, 2026
| rw [iff_conv f L] at hf; solve_by_elim | ||
|
|
||
| -- Remark 9.3.11 may possibly be inaccurate, in that one may be able to safely delete the hypothesis `AdherentPt x₀ E` in the above theorems. This is something that formalization might be able to clarify! If so, the hypothesis may also be deletable in several of the theorems below. | ||
| -- Remark 9.3.11 is not quite true in Lean: the hypothesis `AdherentPt x₀ E` is safely removed |
Contributor
Author
There was a problem hiding this comment.
all theorems proved without the Adherent condition - https://github.com/rkirov/analysis/blob/main/Analysis/Section_9_3.lean
Contributor
Author
|
Removing AdherentPt hypothesis caused some downstream breakages that I am working through now |
Remark 9.3.11 asked whether the AdherentPt hypothesis could be removed from Proposition 9.3.9 and subsequent theorems. Formalization confirms it can. This removes AdherentPt from 17 theorem statements (keeping it only in Convergesto.uniq where it is genuinely needed). Other statement fixes: - Example 9.3.2 constants: 5 → 5.1, 0.41 → 0.42 (needed for strict <) - sq, linear, quadratic: fix argument order (limit before point) - const, id, sq, linear, quadratic: implicit → explicit args - const body: fun x ↦ c → fun _ ↦ c Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
bfb9358 to
76cf7ab
Compare
Contributor
Author
Should be good to review now. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Remark 9.3.11 asked whether the AdherentPt hypothesis could be removed from Proposition 9.3.9 and subsequent theorems. Formalization confirms it can. This removes AdherentPt from 17 theorem statements (keeping it only in Convergesto.uniq where it is genuinely needed).
Other statement fixes: