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/function/uniform_integrable): add API for uniform integrability in the probability sense #12678

Closed
wants to merge 8 commits into from

Conversation

JasonKYi
Copy link
Member

@JasonKYi JasonKYi commented Mar 14, 2022

Uniform integrability in probability theory is commonly defined as the uniform existence of a number for which the expectation of the random variables restricted on the set for which they are greater than said number is arbitrarily small. We have defined it
in mathlib, on the other hand, as uniform integrability in the measure theory sense + existence of a uniform bound of the Lp norms.

This PR proves the first definition implies the second while a later PR will deal with the reverse direction.


Open in Gitpod

@JasonKYi JasonKYi added awaiting-review The author would like community review of the PR awaiting-CI The author would like to see what CI has to say before doing more work. labels Mar 14, 2022
@JasonKYi JasonKYi added this to Under review in Martingale theory Mar 14, 2022
@github-actions github-actions bot removed the awaiting-CI The author would like to see what CI has to say before doing more work. label Mar 14, 2022
Comment on lines +710 to +711
(h : ∀ ε : ℝ, 0 < ε → ∃ C : ℝ≥0, 0 < C ∧
∀ i, snorm ({x | C ≤ ∥f i x∥₊}.indicator (f i)) p μ ≤ ennreal.of_real ε) :
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why do you use ennreal.of_real ε for ε real rather than define ε as an ennreal?

Also that property can be written as tendsto some_function_of_C at_top (nhds 0). Would we gain something by using the tendsto API? I am not sure it's worth changing.

Copy link
Member Author

Choose a reason for hiding this comment

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

I used ennreal.of_real ε since I had used the same formulation for unif_integrable. I don't remember exactly why I used ennreal.of_real ε to define unif_integrable but I recall we needed \epsilon \ne \infty sometimes which taking epsilon to be real avoids.

Copy link
Member Author

@JasonKYi JasonKYi Mar 18, 2022

Choose a reason for hiding this comment

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

About the second comment. I think we will use the current formulation more than the tendsto version. If needed, we can always show that the two formulations are equivalent

Copy link
Collaborator

Choose a reason for hiding this comment

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

After some tests, I am convinced that this file and some statements of the egorov file would be simpler and better with ε : ennreal than with the current ennreal.of_real ε, but I will PR all those changes later. Let's keep the current formulation for now.

src/measure_theory/function/uniform_integrable.lean Outdated Show resolved Hide resolved
src/measure_theory/function/uniform_integrable.lean Outdated Show resolved Hide resolved
src/measure_theory/function/uniform_integrable.lean Outdated Show resolved Hide resolved
src/measure_theory/function/uniform_integrable.lean Outdated Show resolved Hide resolved
@RemyDegenne RemyDegenne 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 Mar 18, 2022
@JasonKYi JasonKYi 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 Mar 18, 2022
@RemyDegenne
Copy link
Collaborator

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 Mar 21, 2022
bors bot pushed a commit that referenced this pull request Mar 21, 2022
… integrability in the probability sense (#12678)

Uniform integrability in probability theory is commonly defined as the uniform existence of a number for which the expectation of the random variables restricted on the set for which they are greater than said number is arbitrarily small. We have defined it 
in mathlib, on the other hand, as uniform integrability in the measure theory sense + existence of a uniform bound of the Lp norms. 

This PR proves the first definition implies the second while a later PR will deal with the reverse direction.
@bors
Copy link

bors bot commented Mar 21, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(measure_theory/function/uniform_integrable): add API for uniform integrability in the probability sense [Merged by Bors] - feat(measure_theory/function/uniform_integrable): add API for uniform integrability in the probability sense Mar 21, 2022
@bors bors bot closed this Mar 21, 2022
Martingale theory automation moved this from Under review to Done! 🎉 Mar 21, 2022
@bors bors bot deleted the JasonKYi/more_unif_int branch March 21, 2022 18:05
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
Development

Successfully merging this pull request may close these issues.

None yet

2 participants