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

[Merged by Bors] - feat(measure_theory/interval_integral): introduce ∫ x in a..b, f x, prove FTC-1#3640

Closed
urkud wants to merge 166 commits intomasterfrom
interval-integral
Closed

[Merged by Bors] - feat(measure_theory/interval_integral): introduce ∫ x in a..b, f x, prove FTC-1#3640
urkud wants to merge 166 commits intomasterfrom
interval-integral

Conversation

@urkud
Copy link
Copy Markdown
Member

@urkud urkud commented Jul 31, 2020

Define ∫ x in a..b, f x ∂μ as ∫ x in Ioc a b, f x ∂μ - ∫ x in Ioc b a, f x ∂μ. With this definition for a probability measure μ we have F_μ(b)-F_μ(a)=∫ x in a..b, f x ∂μ, where F_μ is the cumulative distribution function.


Depends on #3618, #3642, #3643, #3645, #3647, #3652, #3653, #3663 #3685 #3686 .
I hope that the rest can be reviewed in one chunk. Tell me please if you prefer me to further split this PR.

urkud added 30 commits July 23, 2020 09:59
We have a general `norm_zero` lemma and these lemmas are not used
before we introduce `normd_group` instances.
A compact set `s` has finite (resp., zero) measure if every point of
`s` has a neighborhood within `s` of finite (resp., zero) measure.
@urkud urkud changed the title feat(measure_theory/interval_integral): introduce ∫ x in a..b, f x, prove FTC-1 (deps: 3663, 3685, 3686) feat(measure_theory/interval_integral): introduce ∫ x in a..b, f x, prove FTC-1 Aug 4, 2020
@urkud urkud added the awaiting-review The author would like community review of the PR label Aug 4, 2020
Copy link
Copy Markdown
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.

Yury, could you rebase this on master please?

@urkud
Copy link
Copy Markdown
Member Author

urkud commented Aug 4, 2020

Yury, could you rebase this on master please?

I've recently merged master. Is there any reason to rebase? Is it OK if I just squash the whole history into 1 commit instead?

@PatrickMassot
Copy link
Copy Markdown
Member

The goal is simply to have a reviewable diff. Right now it seems the diff tab is indeed reasonable, but I'm almost sure it wasn't when I asked for a rebase. Maybe my browser didn't update the page.

@urkud
Copy link
Copy Markdown
Member Author

urkud commented Aug 5, 2020

Now I have a version with has_strict_deriv_at as well as left/right derivatives. Should I push it here or wait till this one is merged and open another PR?

Copy link
Copy Markdown
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, I only have minor comments. I'd rather merge this one and postpone your improvements to another PR, if you don't mind.

urkud and others added 2 commits August 5, 2020 11:53
@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 Aug 5, 2020
@urkud urkud 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 Aug 6, 2020
@sgouezel
Copy link
Copy Markdown
Collaborator

sgouezel commented Aug 6, 2020

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 Aug 6, 2020
bors bot pushed a commit that referenced this pull request Aug 6, 2020
… prove FTC-1 (#3640)

Define `∫ x in a..b, f x ∂μ` as `∫ x in Ioc a b, f x ∂μ - ∫ x in Ioc b a, f x ∂μ`. With this definition for a probability measure `μ` we have `F_μ(b)-F_μ(a)=∫ x in a..b, f x ∂μ`, where `F_μ` is the cumulative distribution function.
@bors
Copy link
Copy Markdown

bors bot commented Aug 6, 2020

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(measure_theory/interval_integral): introduce ∫ x in a..b, f x, prove FTC-1 [Merged by Bors] - feat(measure_theory/interval_integral): introduce ∫ x in a..b, f x, prove FTC-1 Aug 6, 2020
@bors bors bot closed this Aug 6, 2020
@bors bors bot deleted the interval-integral branch August 6, 2020 07:26
@urkud urkud restored the interval-integral branch August 6, 2020 12:40
@urkud urkud deleted the interval-integral branch August 6, 2020 12:50
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.

4 participants