refactor: golf only AEEqOfIntegral, AEMeasurableOrder, ConditionalExpectation/Real#38108
Closed
yuanyi-350 wants to merge 1 commit intoleanprover-community:masterfrom
Closed
refactor: golf only AEEqOfIntegral, AEMeasurableOrder, ConditionalExpectation/Real#38108yuanyi-350 wants to merge 1 commit intoleanprover-community:masterfrom
AEEqOfIntegral, AEMeasurableOrder, ConditionalExpectation/Real#38108yuanyi-350 wants to merge 1 commit intoleanprover-community:masterfrom
Conversation
PR summary 8e3c989104
|
| File | Base Count | Head Count | Change |
|---|---|---|---|
| Mathlib.MeasureTheory.Function.ConditionalExpectation.Real | 2324 | 2340 | +16 (+0.69%) |
Import changes for all files
| Files | Import difference |
|---|---|
15 filesMathlib.Probability.Combinatorics.BinomialRandomGraph.Defs Mathlib.Probability.Distributions.Binomial Mathlib.Probability.Distributions.SetBernoulli Mathlib.Probability.HasLawExists Mathlib.Probability.IdentDistribIndep Mathlib.Probability.Independence.Conditional Mathlib.Probability.Independence.InfinitePi Mathlib.Probability.Independence.ZeroOne Mathlib.Probability.Kernel.CondDistrib Mathlib.Probability.Kernel.Condexp Mathlib.Probability.Kernel.Disintegration.Integral Mathlib.Probability.Kernel.Disintegration.Unique Mathlib.Probability.Kernel.IonescuTulcea.Traj Mathlib.Probability.Moments.SubGaussian Mathlib.Probability.ProductMeasure |
15 |
23 filesMathlib.MeasureTheory.Function.ConditionalExpectation.Real Mathlib.MeasureTheory.Function.FactorsThrough Mathlib.Probability.BorelCantelli Mathlib.Probability.Kernel.CompProdEqIff Mathlib.Probability.Kernel.Composition.AbsolutelyContinuous Mathlib.Probability.Kernel.Disintegration.Density Mathlib.Probability.Kernel.Disintegration.StandardBorel Mathlib.Probability.Kernel.Posterior Mathlib.Probability.Kernel.RadonNikodym Mathlib.Probability.Martingale.Basic Mathlib.Probability.Martingale.BorelCantelli Mathlib.Probability.Martingale.Centering Mathlib.Probability.Martingale.Convergence Mathlib.Probability.Martingale.OptionalSampling Mathlib.Probability.Martingale.OptionalStopping Mathlib.Probability.Martingale.Upcrossing Mathlib.Probability.Process.Adapted Mathlib.Probability.Process.Filtration Mathlib.Probability.Process.HittingTime Mathlib.Probability.Process.LocalProperty Mathlib.Probability.Process.PartitionFiltration Mathlib.Probability.Process.Predictable Mathlib.Probability.Process.Stopping |
16 |
Declarations diff
No declarations were harmed in the making of this PR! 🐙
You can run this locally as follows
## summary with just the declaration names:
./scripts/pr_summary/declarations_diff.sh <optional_commit>
## more verbose report:
./scripts/pr_summary/declarations_diff.sh long <optional_commit>The doc-module for scripts/pr_summary/declarations_diff.sh contains some details about this script.
No changes to technical debt.
You can run this locally as
./scripts/reporting/technical-debt-metrics.sh pr_summary
- The
relativevalue is the weighted sum of the differences with weight given by the inverse of the current value of the statistic. - The
absolutevalue is therelativevalue divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).
Contributor
|
Can you update the PR description to say what this PR does (as opposed to what it doesn't do). |
Collaborator
Author
|
!bench |
|
Benchmark results for c22a484 against 8e3c989 are in. No significant results found. @yuanyi-350
Small changes (1✅)
|
2 tasks
Contributor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR selectively ports part of #38104.
Included files:
Mathlib/MeasureTheory/Function/ConditionalExpectation/Real.leanMathlib/MeasureTheory/Function/AEEqOfIntegral.leanMathlib/MeasureTheory/Function/AEMeasurableOrder.leanMore concretely:
AEEqOfIntegral.lean, the PR rewrites the lemmas saying "if two integrable functions have the same integrals on all finite-measure measurable sets, then they are equal almost everywhere." The old proof went throughLp; the new proof uses the more directAEFinStronglyMeasurablelemmas. Same result, shorter route.AEMeasurableOrder.lean, it cleans up one proof of almost-everywhere measurability by using standard lemmas likemeasure_eq_zero_iff_ae_notMemandDense.exists_betweeninstead of spelling those arguments out manually. This is a pure proof simplification.ConditionalExpectation/Real.lean, the main change is to importConditionalExpectation/CondJensenand use its general results. In particular:L¹contraction statement for conditional expectation is now derived from the general inequality‖E[f | m]‖ ≤ E[‖f‖ | m],∫ |E[f | m]| ≤ ∫ |f|becomes a direct consequence of that norm inequality plusintegral_condExp,|f| ≤ Ra.e., then|E[f | m]| ≤ Ra.e." is re-proved by saying thatflies a.e. in the closed convex interval[-R, R], so its conditional expectation stays in that interval byConvex.condExp_mem.This intentionally excludes the rest of #38104.