Skip to content
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/metric_space/isometry): use namespace, add lemmas #15591

Closed
wants to merge 9 commits into from

Conversation

urkud
Copy link
Member

@urkud urkud commented Jul 21, 2022

  • Use namespace isometry.
  • Add lemmas like isometry.preimage_ball.

Open in Gitpod

* Use `namespace isometry`.
* Add lemmas like `isometry.preimage_ball`.
@urkud urkud added the awaiting-review The author would like community review of the PR label Jul 21, 2022
Copy link
Collaborator

@sgouezel sgouezel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors d+
Thanks!

@@ -30,130 +30,137 @@ between pseudoemetric spaces, or equivalently the distance between pseudometric
def isometry [pseudo_emetric_space α] [pseudo_emetric_space β] (f : α → β) : Prop :=
∀x1 x2 : α, edist (f x1) (f x2) = edist x1 x2

/-- On pseudometric spaces, a map is an isometry if and only if it preserves nonnegative
distances. -/
lemma isometry_emetric_iff_nndist [pseudo_metric_space α] [pseudo_metric_space β] {f : α → β} :
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The name looks uselessly verbose to me, especially since there is absolutely no emetric in sight. What about isometry_iff_nndist_eq (and the next one could be renamed isometry_iff_dist_eq).

src/topology/metric_space/isometry.lean Outdated Show resolved Hide resolved
@bors
Copy link

bors bot commented Jul 22, 2022

✌️ urkud can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@leanprover-community-bot-assistant leanprover-community-bot-assistant added delegated The PR author may merge after reviewing final suggestions. and removed awaiting-review The author would like community review of the PR labels Jul 22, 2022
urkud and others added 5 commits July 22, 2022 07:05
@urkud
Copy link
Member Author

urkud commented Jul 22, 2022

bors merge

@github-actions github-actions bot added the ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) label Jul 22, 2022
bors bot pushed a commit that referenced this pull request Jul 22, 2022
* Use `namespace isometry`.
* Add lemmas like `isometry.preimage_ball`.
@bors
Copy link

bors bot commented Jul 22, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(topology/metric_space/isometry): use namespace, add lemmas [Merged by Bors] - feat(topology/metric_space/isometry): use namespace, add lemmas Jul 22, 2022
@bors bors bot closed this Jul 22, 2022
@bors bors bot deleted the YK-isometry-lemmas branch July 22, 2022 21:09
joelriou pushed a commit that referenced this pull request Jul 23, 2022
* Use `namespace isometry`.
* Add lemmas like `isometry.preimage_ball`.
bottine pushed a commit to bottine/mathlib that referenced this pull request Jul 30, 2022
…prover-community#15591)

* Use `namespace isometry`.
* Add lemmas like `isometry.preimage_ball`.
robertylewis pushed a commit that referenced this pull request Aug 2, 2022
* Use `namespace isometry`.
* Add lemmas like `isometry.preimage_ball`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated The PR author may merge after reviewing final suggestions. ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants