Skip to content

[Merged by Bors] - feat: real-valued Lᵖ norm#23881

Closed
YaelDillies wants to merge 7 commits intomasterfrom
nnlpnorm
Closed

[Merged by Bors] - feat: real-valued Lᵖ norm#23881
YaelDillies wants to merge 7 commits intomasterfrom
nnlpnorm

Conversation

@YaelDillies
Copy link
Copy Markdown
Contributor

@YaelDillies YaelDillies commented Apr 9, 2025


Open in Gitpod

@YaelDillies YaelDillies requested a review from b-mehta April 9, 2025 16:43
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2025

PR summary 4e498b8337

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference
Mathlib.MeasureTheory.Function.LpSeminorm.LpNorm (new file) 2301

Declarations diff

+ ae_le_lpNorm_exponent_top
+ coe_lt_ofReal
+ eLpNorm_eq_lintegral_rpow_enorm_toReal
+ lpNorm
+ lpNorm_abs
+ lpNorm_add_le
+ lpNorm_add_le'
+ lpNorm_conj
+ lpNorm_const
+ lpNorm_const'
+ lpNorm_const_smul
+ lpNorm_div_natCast
+ lpNorm_eq_integral_norm_rpow_toReal
+ lpNorm_eq_zero
+ lpNorm_expect_le
+ lpNorm_exponent_top_eq_essSup
+ lpNorm_exponent_zero
+ lpNorm_fun_abs
+ lpNorm_fun_div_natCast
+ lpNorm_fun_mul_natCast
+ lpNorm_fun_natCast_mul
+ lpNorm_fun_neg
+ lpNorm_fun_zero
+ lpNorm_le_add_lpNorm_add
+ lpNorm_le_lpNorm_add_lpNorm_sub
+ lpNorm_le_lpNorm_add_lpNorm_sub'
+ lpNorm_measure_zero
+ lpNorm_mono_real
+ lpNorm_mul_natCast
+ lpNorm_natCast_mul
+ lpNorm_neg
+ lpNorm_nnreal_eq_integral_norm_rpow
+ lpNorm_nonneg
+ lpNorm_norm
+ lpNorm_nsmul
+ lpNorm_of_isEmpty
+ lpNorm_of_not_aestronglyMeasurable
+ lpNorm_of_not_memLp
+ lpNorm_one
+ lpNorm_one'
+ lpNorm_one_eq_integral_norm
+ lpNorm_smul_measure_of_ne_top
+ lpNorm_smul_measure_of_ne_zero
+ lpNorm_sub_comm
+ lpNorm_sub_le
+ lpNorm_sub_le_lpNorm_sub_add_lpNorm_sub
+ lpNorm_sum_le
+ lpNorm_zero
+ ofReal_essSup
+ ofReal_limsup
+ ofReal_lpNorm
+ toReal_eLpNorm
+ toReal_essSup
+ toReal_limsup
- eLpNorm_eq_lintegral_rpow_enorm

You can run this locally as follows
## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>

The doc-module for script/declarations_diff.sh contains some details about this script.


No changes to technical debt.

You can run this locally as

./scripts/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@github-actions github-actions bot added the t-measure-probability Measure theory / Probability theory label Apr 9, 2025
@leanprover-community-bot-assistant leanprover-community-bot-assistant added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Apr 15, 2025
@leanprover-community-bot-assistant
Copy link
Copy Markdown
Collaborator

This pull request has conflicts, please merge master and resolve them.

@github-actions github-actions bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Apr 15, 2025
Copy link
Copy Markdown
Contributor

@b-mehta b-mehta 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 reasonable enough to me, but I'd like an analyst to check too.

maintainer merge

Comment thread Mathlib/MeasureTheory/Function/LpSeminorm/NNLpNorm.lean Outdated
Comment thread Mathlib/MeasureTheory/Function/LpSeminorm/Defs.lean Outdated
@github-actions
Copy link
Copy Markdown

🚀 Pull request has been placed on the maintainer queue by b-mehta.

@github-actions github-actions bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Apr 15, 2025
Comment thread Mathlib/MeasureTheory/Function/LpSeminorm/NNLpNorm.lean Outdated
Comment thread Mathlib/MeasureTheory/Function/LpSeminorm/TriangleInequality.lean Outdated
@sgouezel sgouezel added awaiting-author A reviewer has asked the author a question or requested changes. and removed maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. labels Apr 17, 2025
@YaelDillies YaelDillies removed the awaiting-author A reviewer has asked the author a question or requested changes. label Apr 18, 2025
Copy link
Copy Markdown
Contributor

@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.

I'm not completely sure I am convinced by the PR: one lesson from Carleson is that when you formalize analysis, you need the ennreal-valued norms, and the real-valued norms. Can you explain why you think the nnreal valued version is more important than the real-valued?

variable {α E : Type*} {m : MeasurableSpace α} {p : ℝ≥0∞} {q : ℝ} {μ ν : Measure α}
[NormedAddCommGroup E] {f g h : α → E}

lemma coe_nnLpNorm_eq_eLpNorm (hf : MemLp f p μ) : nnLpNorm f p μ = eLpNorm f p μ := by
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@[simp] ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Do you think the MemLp assumption will be inferrable in practice?

Comment thread Mathlib/MeasureTheory/Function/LpSeminorm/NNLpNorm.lean Outdated
Comment thread Mathlib/MeasureTheory/Function/LpSeminorm/NNLpNorm.lean Outdated
Comment thread Mathlib/MeasureTheory/Function/LpSeminorm/NNLpNorm.lean Outdated
@sgouezel sgouezel added the awaiting-author A reviewer has asked the author a question or requested changes. label Apr 23, 2025
@leanprover-community-bot-assistant leanprover-community-bot-assistant added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label May 13, 2025
@leanprover-community-bot-assistant
Copy link
Copy Markdown
Collaborator

This pull request has conflicts, please merge master and resolve them.

@github-actions github-actions bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label May 14, 2025
@leanprover-community-bot-assistant leanprover-community-bot-assistant added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jul 16, 2025
@leanprover-community-bot-assistant
Copy link
Copy Markdown
Collaborator

This pull request has conflicts, please merge master and resolve them.

@leanprover-community-bot-assistant leanprover-community-bot-assistant removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jul 17, 2025
@mathlib4-merge-conflict-bot mathlib4-merge-conflict-bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Aug 18, 2025
@mathlib4-merge-conflict-bot
Copy link
Copy Markdown
Collaborator

This pull request has conflicts, please merge master and resolve them.

@github-actions github-actions bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Aug 18, 2025
@mathlib4-merge-conflict-bot mathlib4-merge-conflict-bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Nov 19, 2025
@mathlib4-merge-conflict-bot
Copy link
Copy Markdown
Collaborator

This pull request has conflicts, please merge master and resolve them.

@YaelDillies YaelDillies removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Feb 5, 2026
Comment thread Mathlib/MeasureTheory/Function/LpSeminorm/LpNorm.lean Outdated
Comment thread Mathlib/MeasureTheory/Function/LpSeminorm/LpNorm.lean Outdated
Comment thread Mathlib/MeasureTheory/Function/LpSeminorm/LpNorm.lean Outdated
Comment thread Mathlib/MeasureTheory/Function/LpSeminorm/LpNorm.lean Outdated
Comment thread Mathlib/MeasureTheory/Function/LpSeminorm/LpNorm.lean Outdated
@sgouezel sgouezel added the awaiting-author A reviewer has asked the author a question or requested changes. label Feb 7, 2026
@YaelDillies YaelDillies removed the awaiting-author A reviewer has asked the author a question or requested changes. label Feb 8, 2026
Comment thread Mathlib/Order/Filter/ENNReal.lean Outdated
@sgouezel
Copy link
Copy Markdown
Contributor

sgouezel commented Feb 9, 2026

bors r+
Thanks!

@mathlib-triage mathlib-triage bot added the ready-to-merge This PR has been sent to bors. label Feb 9, 2026
mathlib-bors bot pushed a commit that referenced this pull request Feb 9, 2026
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Feb 9, 2026

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: real-valued Lᵖ norm [Merged by Bors] - feat: real-valued Lᵖ norm Feb 9, 2026
@mathlib-bors mathlib-bors bot closed this Feb 9, 2026
@mathlib-bors mathlib-bors bot deleted the nnlpnorm branch February 9, 2026 07:57
justus-springer pushed a commit to justus-springer/mathlib4 that referenced this pull request Feb 10, 2026
Ruben-VandeVelde added a commit to Ruben-VandeVelde/mathlib4 that referenced this pull request Feb 12, 2026
michaellee94 pushed a commit to michaellee94/mathlib4 that referenced this pull request Feb 15, 2026
rao107 pushed a commit to rao107/mathlib4 that referenced this pull request Feb 18, 2026
Maldooor pushed a commit to Maldooor/mathlib4 that referenced this pull request Feb 25, 2026
pfaffelh pushed a commit to pfaffelh/mathlib4 that referenced this pull request Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors. t-measure-probability Measure theory / Probability theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants