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/pi_Lp): L^p distance on finite products #3059

Closed
wants to merge 10 commits into from

Conversation

sgouezel
Copy link
Collaborator

@sgouezel sgouezel commented Jun 13, 2020

L^p edistance (or distance, or norm) on finite products of emetric spaces (or metric spaces, or normed groups), put on a type synonym pi_Lp p hp α to avoid instance clashes, and being careful to register as uniformity the product uniformity.


Blocked by #2988

@sgouezel sgouezel added the blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. label Jun 13, 2020
@gebner
Copy link
Member

gebner commented Jun 13, 2020

A very general question, and I hope I didn't miss an explanation on Zulip: is there a reason the Lᵖ-norm is only defined here for the special case of functions on finite measure spaces with the counting measure? Are there issues because these are dependent products?

@semorrison semorrison added awaiting-review The author would like community review of the PR and removed blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. labels Jun 13, 2020
@semorrison semorrison changed the title feat(topology/metric_space/pi_Lp): L^p distance on finite products (blocked by #2988) feat(topology/metric_space/pi_Lp): L^p distance on finite products Jun 13, 2020
@sgouezel
Copy link
Collaborator Author

The goal here is to show that the topology defined by the L^p norm is the same as the product topology. This is only true for finite products.

We will definitely need to define the L^p space for functions over a base with a measure (probably in a non-dependent setting), but the measure-theory part of mathlib is not completely ready yet.

@PatrickMassot
Copy link
Member

It would be nice to have a global documentation for this uniformity juggling thing. Maybe you could also add a note discussing the two confusing interpretations: measure theoretic L^p that confused Gabriel but also the potential confusion about norm spaces since I guess people could complain the statement should be a special case equivalence of norms on R^n if they read the context too quickly. At the proof level one could also argue that one could use equivalence of norms after sending everything in some R^n, but we wouldn't get the explicit Lipschitz constants that you derive here, so maybe this is also worth one line of implementation notes.

bors d+

@bors
Copy link

bors bot commented Jun 13, 2020

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

@PatrickMassot PatrickMassot 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 Jun 13, 2020
src/topology/metric_space/pi_Lp.lean Outdated Show resolved Hide resolved
src/topology/metric_space/pi_Lp.lean Outdated Show resolved Hide resolved
Co-authored-by: Bryan Gin-ge Chen <bryangingechen@gmail.com>
@PatrickMassot
Copy link
Member

Sébastien doesn't seem to like delegation, so let's:
bors r+

@bors
Copy link

bors bot commented Jun 13, 2020

👎 Rejected by label

@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 Jun 13, 2020
@PatrickMassot PatrickMassot removed the awaiting-author A reviewer has asked the author a question or requested changes label Jun 13, 2020
@gebner
Copy link
Member

gebner commented Jun 13, 2020

bors merge

bors bot pushed a commit that referenced this pull request Jun 13, 2020
…3059)

`L^p` edistance (or distance, or norm) on finite products of emetric spaces (or metric spaces, or normed groups), put on a type synonym `pi_Lp p hp α` to avoid instance clashes, and being careful to register as uniformity the product uniformity.
@bors
Copy link

bors bot commented Jun 13, 2020

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(topology/metric_space/pi_Lp): L^p distance on finite products [Merged by Bors] - feat(topology/metric_space/pi_Lp): L^p distance on finite products Jun 13, 2020
@sgouezel
Copy link
Collaborator Author

Hey, I was not done with the comments ! :-) Followup in #3070

cipher1024 pushed a commit to cipher1024/mathlib that referenced this pull request Mar 15, 2022
…eanprover-community#3059)

`L^p` edistance (or distance, or norm) on finite products of emetric spaces (or metric spaces, or normed groups), put on a type synonym `pi_Lp p hp α` to avoid instance clashes, and being careful to register as uniformity the product uniformity.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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

5 participants