Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

[Merged by Bors] - feat(topology/metric_space): multiplicative opposites inherit the same (pseudo_?)(e?)metric and uniform_space#12120

Closed
eric-wieser wants to merge 9 commits intomasterfrom
eric-wieser/metric_space-op
Closed

[Merged by Bors] - feat(topology/metric_space): multiplicative opposites inherit the same (pseudo_?)(e?)metric and uniform_space#12120
eric-wieser wants to merge 9 commits intomasterfrom
eric-wieser/metric_space-op

Conversation

@eric-wieser
Copy link
Copy Markdown
Member

@eric-wieser eric-wieser commented Feb 18, 2022

This puts the "obvious" metric on the opposite type such that dist (op x) (op y) = dist x y.

This also merges subtype.pseudo_dist_eq and subtype.dist_eq as the latter was a special case of the former.


Open in Gitpod

@eric-wieser eric-wieser added the awaiting-review The author would like community review of the PR label Feb 18, 2022
@eric-wieser eric-wieser force-pushed the eric-wieser/metric_space-op branch from 17205ad to 856369d Compare February 18, 2022 12:54
@urkud urkud added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Feb 22, 2022
@eric-wieser eric-wieser added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Feb 22, 2022
@eric-wieser eric-wieser changed the title feat(topology/metric_space): multiplicative opposites inherit the same metric feat(topology/metric_space): multiplicative opposites inherit the same (pseudo_?)(e?)metric and uniform_space Feb 22, 2022
@urkud
Copy link
Copy Markdown
Member

urkud commented Feb 23, 2022

Otherwise LGTM.
bors d+

@bors
Copy link
Copy Markdown

bors bot commented Feb 23, 2022

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

@github-actions github-actions bot 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 Feb 23, 2022
@eric-wieser
Copy link
Copy Markdown
Member Author

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 Feb 24, 2022
bors bot pushed a commit that referenced this pull request Feb 24, 2022
…e `(pseudo_?)(e?)metric` and `uniform_space` (#12120)

This puts the "obvious" metric on the opposite type such that `dist (op x) (op y) = dist x y`.

This also merges `subtype.pseudo_dist_eq` and `subtype.dist_eq` as the latter was a special case of the former.
@bors
Copy link
Copy Markdown

bors bot commented Feb 24, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(topology/metric_space): multiplicative opposites inherit the same (pseudo_?)(e?)metric and uniform_space [Merged by Bors] - feat(topology/metric_space): multiplicative opposites inherit the same (pseudo_?)(e?)metric and uniform_space Feb 24, 2022
@bors bors bot closed this Feb 24, 2022
@bors bors bot deleted the eric-wieser/metric_space-op branch February 24, 2022 02:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

2 participants