-
Notifications
You must be signed in to change notification settings - Fork 1k
[Merged by Bors] - feat(Topology): generalize tendsto_inv_iff from ℝ≥0∞ to ContinuousInv + InvolutiveInv #33920
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Merged by Bors] - feat(Topology): generalize tendsto_inv_iff from ℝ≥0∞ to ContinuousInv + InvolutiveInv #33920
Conversation
PR summary 3de2350c1fImport changes for modified filesNo significant changes to the import graph Import changes for all files
Declarations diff
You can run this locally as follows## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>
## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>The doc-module for No changes to technical debt.You can run this locally as
|
vihdzp
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise LGTM
|
I took the liberty of making the title of your PR more precise: the first one sounded a bit weird given that ℝ≥0∞ is not a topological group... Do not hesitate to change to something else if you prefer. bors d+ |
|
✌️ Citronhat can now approve this pull request. To approve and merge a pull request, simply reply with |
Fair enough. I was close to changing it myself, but I ended up convinced myself that the more readable title justified the "lie". By the way, I have a couple of related questions regarding design philosophy that I hope one of you can clarify:
I would appreciate any insights you have time to share :) |
|
bors r+ |
… + InvolutiveInv (#33920) This PR implements the [TODO](https://github.com/leanprover-community/mathlib4/blob/b2f24e70b375e42e5a60251a2f43baa20ca668f1/Mathlib/Topology/Instances/ENNReal/Lemmas.lean#L468C1-L471C70) in `Topology.Instances.ENNReal.Lemmas` by moving the lemma to a more general setting. - Add a general lemma `tendsto_inv_iff` in `Topology.Algebra.Group.Basic` (with `@[simp]`), whose proof is the same as the previous `ℝ≥0∞` proof, just stated for an arbitrary `G` with `[InvolutiveInv G]` and `[ContinuousInv G]`. - Mark it `@[to_additive (attr := simp)]`, so the additive analogue is generated automatically. - Deprecate `ENNReal.tendsto_inv_iff` in `Topology.Instances.ENNReal.Lemmas`. No mathematical content changes: this is a relocation/deprecation of the existing theorem plus the additive version. Co-authored-by: Simon <Citronhat@gmail.com>
|
Build failed: |
|
bors r+ |
… + InvolutiveInv (#33920) This PR implements the [TODO](https://github.com/leanprover-community/mathlib4/blob/b2f24e70b375e42e5a60251a2f43baa20ca668f1/Mathlib/Topology/Instances/ENNReal/Lemmas.lean#L468C1-L471C70) in `Topology.Instances.ENNReal.Lemmas` by moving the lemma to a more general setting. - Add a general lemma `tendsto_inv_iff` in `Topology.Algebra.Group.Basic` (with `@[simp]`), whose proof is the same as the previous `ℝ≥0∞` proof, just stated for an arbitrary `G` with `[InvolutiveInv G]` and `[ContinuousInv G]`. - Mark it `@[to_additive (attr := simp)]`, so the additive analogue is generated automatically. - Deprecate `ENNReal.tendsto_inv_iff` in `Topology.Instances.ENNReal.Lemmas`. No mathematical content changes: this is a relocation/deprecation of the existing theorem plus the additive version. Co-authored-by: Simon <Citronhat@gmail.com>
|
Pull request successfully merged into master. Build succeeded: |
… + InvolutiveInv (leanprover-community#33920) This PR implements the [TODO](https://github.com/leanprover-community/mathlib4/blob/b2f24e70b375e42e5a60251a2f43baa20ca668f1/Mathlib/Topology/Instances/ENNReal/Lemmas.lean#L468C1-L471C70) in `Topology.Instances.ENNReal.Lemmas` by moving the lemma to a more general setting. - Add a general lemma `tendsto_inv_iff` in `Topology.Algebra.Group.Basic` (with `@[simp]`), whose proof is the same as the previous `ℝ≥0∞` proof, just stated for an arbitrary `G` with `[InvolutiveInv G]` and `[ContinuousInv G]`. - Mark it `@[to_additive (attr := simp)]`, so the additive analogue is generated automatically. - Deprecate `ENNReal.tendsto_inv_iff` in `Topology.Instances.ENNReal.Lemmas`. No mathematical content changes: this is a relocation/deprecation of the existing theorem plus the additive version. Co-authored-by: Simon <Citronhat@gmail.com>
… + InvolutiveInv (leanprover-community#33920) This PR implements the [TODO](https://github.com/leanprover-community/mathlib4/blob/b2f24e70b375e42e5a60251a2f43baa20ca668f1/Mathlib/Topology/Instances/ENNReal/Lemmas.lean#L468C1-L471C70) in `Topology.Instances.ENNReal.Lemmas` by moving the lemma to a more general setting. - Add a general lemma `tendsto_inv_iff` in `Topology.Algebra.Group.Basic` (with `@[simp]`), whose proof is the same as the previous `ℝ≥0∞` proof, just stated for an arbitrary `G` with `[InvolutiveInv G]` and `[ContinuousInv G]`. - Mark it `@[to_additive (attr := simp)]`, so the additive analogue is generated automatically. - Deprecate `ENNReal.tendsto_inv_iff` in `Topology.Instances.ENNReal.Lemmas`. No mathematical content changes: this is a relocation/deprecation of the existing theorem plus the additive version. Co-authored-by: Simon <Citronhat@gmail.com>
… + InvolutiveInv (leanprover-community#33920) This PR implements the [TODO](https://github.com/leanprover-community/mathlib4/blob/b2f24e70b375e42e5a60251a2f43baa20ca668f1/Mathlib/Topology/Instances/ENNReal/Lemmas.lean#L468C1-L471C70) in `Topology.Instances.ENNReal.Lemmas` by moving the lemma to a more general setting. - Add a general lemma `tendsto_inv_iff` in `Topology.Algebra.Group.Basic` (with `@[simp]`), whose proof is the same as the previous `ℝ≥0∞` proof, just stated for an arbitrary `G` with `[InvolutiveInv G]` and `[ContinuousInv G]`. - Mark it `@[to_additive (attr := simp)]`, so the additive analogue is generated automatically. - Deprecate `ENNReal.tendsto_inv_iff` in `Topology.Instances.ENNReal.Lemmas`. No mathematical content changes: this is a relocation/deprecation of the existing theorem plus the additive version. Co-authored-by: Simon <Citronhat@gmail.com>
This PR implements the TODO in
Topology.Instances.ENNReal.Lemmasby moving the lemma to a more general setting.Add a general lemma
tendsto_inv_iffinTopology.Algebra.Group.Basic(with@[simp]), whose proof is the same as the previousℝ≥0∞proof, just stated for an arbitraryGwith[InvolutiveInv G]and[ContinuousInv G].Mark it
@[to_additive (attr := simp)], so the additive analogue is generated automatically.Deprecate
ENNReal.tendsto_inv_iffinTopology.Instances.ENNReal.Lemmas.No mathematical content changes: this is a relocation/deprecation of the existing theorem plus the additive version.