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

[Merged by Bors] - refactor(measure_theory/l1_space): remove one of the two definitions of L1 space #6058

Closed
wants to merge 18 commits into from

Conversation

sgouezel
Copy link
Collaborator

@sgouezel sgouezel commented Feb 5, 2021

Currently, we have two independent versions of the L^1 space in mathlib: one coming from the general family of L^p spaces, the other one is an ad hoc construction based on the integrable predicate used in the construction of the Bochner integral.

We remove the second construction, and use instead the L^1 space coming from the family of L^p spaces to construct the Bochner integral. Still, we keep the integrable predicate as it is generally useful, and show that it coincides with the mem_Lp 1 predicate.


@github-actions github-actions bot 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 Feb 5, 2021
@sgouezel sgouezel changed the title refactor(measure_theory/l1_space) refactor(measure_theory/l1_space): remove one of the two definitions of L1 space Feb 5, 2021
@github-actions github-actions bot added merge-conflict Please `git merge origin/master` then a bot will remove this label. 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 Feb 7, 2021
@github-actions
Copy link

🎉 Great news! Looks like all the dependencies have been resolved:

💡 To add or remove a dependency please update this issue/PR description.

Brought to you by Dependent Issues (:robot: ). Happy coding!

@github-actions github-actions bot removed the merge-conflict Please `git merge origin/master` then a bot will remove this label. label Feb 10, 2021
@sgouezel sgouezel added the awaiting-review The author would like community review of the PR label Feb 11, 2021
Such a predicate is already available under the name `mem_ℒp 1`. We give a direct definition which
is easier to use, and show that it is equivalent to `mem_ℒp 1`

In the second part, we establish an API between API and the space `L¹` of equivalence classes of
Copy link
Member

Choose a reason for hiding this comment

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

"API between API and ..." Is it intentional?

@urkud
Copy link
Member

urkud commented Feb 13, 2021

LGTM
bors d+

@bors
Copy link

bors bot commented Feb 13, 2021

✌️ 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.

@sgouezel
Copy link
Collaborator Author

bors r+

@github-actions github-actions bot added ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) and removed awaiting-review The author would like community review of the PR labels Feb 13, 2021
bors bot pushed a commit that referenced this pull request Feb 13, 2021
…of L1 space (#6058)

Currently, we have two independent versions of the `L^1` space in mathlib: one coming from the general family of `L^p` spaces, the other one is an ad hoc construction based on the `integrable` predicate used in the construction of the Bochner integral.

We remove the second construction, and use instead the `L^1` space coming from the family of `L^p` spaces to construct the Bochner integral. Still, we keep the `integrable` predicate as it is generally useful, and show that it coincides with the `mem_Lp 1` predicate.
@bors
Copy link

bors bot commented Feb 13, 2021

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title refactor(measure_theory/l1_space): remove one of the two definitions of L1 space [Merged by Bors] - refactor(measure_theory/l1_space): remove one of the two definitions of L1 space Feb 13, 2021
@bors bors bot closed this Feb 13, 2021
@bors bors bot deleted the refactor_l1 branch February 13, 2021 22:35
b-mehta pushed a commit that referenced this pull request Apr 2, 2021
…of L1 space (#6058)

Currently, we have two independent versions of the `L^1` space in mathlib: one coming from the general family of `L^p` spaces, the other one is an ad hoc construction based on the `integrable` predicate used in the construction of the Bochner integral.

We remove the second construction, and use instead the `L^1` space coming from the family of `L^p` spaces to construct the Bochner integral. Still, we keep the `integrable` predicate as it is generally useful, and show that it coincides with the `mem_Lp 1` predicate.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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.

3 participants