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(measure_theory/integral/riesz_markov_kakutani): begin construction of content for Riesz-Markov-Kakutani #16367

Closed
wants to merge 6 commits into from

Conversation

ReimannJ
Copy link
Collaborator

@ReimannJ ReimannJ commented Sep 4, 2022

Begin construction of content for Riesz-Markov-Kakutani

Construct a map on the compact sets and prove monotonicity and subadditivity.


Co-authored-by: Kalle Kytölä kalle.kytola@aalto.fi

Open in Gitpod

@ReimannJ ReimannJ added the awaiting-review The author would like community review of the PR label Sep 4, 2022
@JasonKYi JasonKYi changed the title <feat>(measure_theory/integral): begin construction of content for Riesz-Markov-Kakutani feat(measure_theory/integral/riesz_markov_kakutani): begin construction of content for Riesz-Markov-Kakutani Sep 4, 2022
@ADedecker ADedecker added the t-analysis Analysis (normed *, calculus) label Sep 4, 2022
Copy link
Member

@ADedecker ADedecker left a comment

Choose a reason for hiding this comment

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

This looks really nice, thanks 🎉
The only thing that bothers me a little is the module docstring: could you rewrite it a little so that it is clear that the full theorem isn't there yet? I know this is temporary, but it feels wrong to claim we have the theorem when it is not in the file.

@ReimannJ
Copy link
Collaborator Author

ReimannJ commented Sep 7, 2022

This looks really nice, thanks 🎉 The only thing that bothers me a little is the module docstring: could you rewrite it a little so that it is clear that the full theorem isn't there yet? I know this is temporary, but it feels wrong to claim we have the theorem when it is not in the file.

Good point, done!

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.

This looks very good, thanks!

src/measure_theory/integral/riesz_markov_kakutani.lean Outdated Show resolved Hide resolved
src/measure_theory/integral/riesz_markov_kakutani.lean Outdated Show resolved Hide resolved
src/measure_theory/integral/riesz_markov_kakutani.lean Outdated Show resolved Hide resolved
src/measure_theory/integral/riesz_markov_kakutani.lean Outdated Show resolved Hide resolved
src/measure_theory/integral/riesz_markov_kakutani.lean Outdated Show resolved Hide resolved
src/measure_theory/integral/riesz_markov_kakutani.lean Outdated Show resolved Hide resolved
@sgouezel sgouezel 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 Sep 13, 2022
@ReimannJ ReimannJ 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 Oct 9, 2022
@ADedecker ADedecker 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 Oct 10, 2022
Co-authored-by: Anatole Dedecker <anatolededecker@gmail.com>
@ReimannJ ReimannJ 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 Oct 11, 2022
@ADedecker
Copy link
Member

Thanks again 🐙

maintainer merge

@github-actions
Copy link

🚀 Pull request has been placed on the maintainer queue by ADedecker.

Copy link
Member

@PatrickMassot PatrickMassot left a comment

Choose a reason for hiding this comment

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

bors merge

@leanprover-community-bot-assistant leanprover-community-bot-assistant 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 Oct 11, 2022
bors bot pushed a commit that referenced this pull request Oct 11, 2022
…on of content for Riesz-Markov-Kakutani (#16367)

Begin construction of content for Riesz-Markov-Kakutani 

Construct a map on the compact sets and prove monotonicity and subadditivity.



Co-authored-by: ReimannJ <105789986+ReimannJ@users.noreply.github.com>
@bors
Copy link

bors bot commented Oct 11, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(measure_theory/integral/riesz_markov_kakutani): begin construction of content for Riesz-Markov-Kakutani [Merged by Bors] - feat(measure_theory/integral/riesz_markov_kakutani): begin construction of content for Riesz-Markov-Kakutani Oct 11, 2022
@bors bors bot closed this Oct 11, 2022
@bors bors bot deleted the riesz_markov_kakutani branch October 11, 2022 13:09
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+`.) t-analysis Analysis (normed *, calculus)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants