Skip to content

[Merged by Bors] - feat: Hölder framework for lp spaces#35197

Closed
j-loreaux wants to merge 14 commits into
leanprover-community:masterfrom
j-loreaux:lpHolder
Closed

[Merged by Bors] - feat: Hölder framework for lp spaces#35197
j-loreaux wants to merge 14 commits into
leanprover-community:masterfrom
j-loreaux:lpHolder

Conversation

@j-loreaux

@j-loreaux j-loreaux commented Feb 12, 2026

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Feb 12, 2026

Copy link
Copy Markdown

PR summary 7b185b2fb9

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference
Mathlib.Analysis.Normed.Lp.lpHolder (new file) 1874

Declarations diff

+ add_apply
+ bilin_of_top_left
+ bilin_of_top_right
+ bilin_of_zero_left
+ bilin_of_zero_right
+ dualPairing
+ dualPairing_apply
+ holderL
+ holder_gen_bound
+ holder_top_left_bound
+ holder_top_right_bound
+ holderₗ
+ mapCLM
+ neg_apply
+ norm_dualPairing
+ norm_holderL_le
+ norm_mapCLM_le
+ norm_postcomp_le
+ norm_tsumCLM_le
+ nsmul_apply
+ sub_apply
+ unique
+ zero_apply
+ zsmul_apply
++ holder
- PreLp.unique

You can run this locally as follows
## from your `mathlib4` directory:
git clone https://github.com/leanprover-community/mathlib-ci.git ../mathlib-ci

## summary with just the declaration names:
../mathlib-ci/scripts/pr_summary/declarations_diff.sh <optional_commit>

## more verbose report:
../mathlib-ci/scripts/pr_summary/declarations_diff.sh long <optional_commit>

The doc-module for scripts/pr_summary/declarations_diff.sh in the mathlib-ci repository contains some details about this script.


No changes to strong technical debt.
No changes to weak technical debt.

Current commit 7b185b2fb9
Reference commit 23bafd5167

This script lives in the mathlib-ci repository. To run it locally, from your mathlib4 directory:

git clone https://github.com/leanprover-community/mathlib-ci.git ../mathlib-ci
../mathlib-ci/scripts/reporting/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).

@mathlib-dependent-issues mathlib-dependent-issues Bot added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Feb 12, 2026
@loefflerd loefflerd added the t-analysis Analysis (normed *, calculus) label Feb 14, 2026
@mathlib-merge-conflicts

Copy link
Copy Markdown

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

@mathlib-merge-conflicts mathlib-merge-conflicts 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 Feb 18, 2026
@mathlib-merge-conflicts mathlib-merge-conflicts 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 Mar 6, 2026
mathlib-bors Bot pushed a commit that referenced this pull request Mar 18, 2026
…35198)

Currently the API exists only for `Real.HolderConjugate`. This generalizes it in anticipation of #35197, which implements the Hölder framework for `lp` spaces.
@mathlib-merge-conflicts mathlib-merge-conflicts 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 Mar 18, 2026
@mathlib-merge-conflicts

Copy link
Copy Markdown

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

justus-springer pushed a commit to justus-springer/mathlib4 that referenced this pull request Mar 28, 2026
…eanprover-community#35198)

Currently the API exists only for `Real.HolderConjugate`. This generalizes it in anticipation of leanprover-community#35197, which implements the Hölder framework for `lp` spaces.
@mathlib-dependent-issues mathlib-dependent-issues Bot removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Mar 30, 2026
@mathlib-dependent-issues

Copy link
Copy Markdown

@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 1, 2026
@loefflerd loefflerd removed their assignment Apr 3, 2026

@mike1729 mike1729 left a comment

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.

I found a couple of small nits.

Comment thread Mathlib/Analysis/Normed/Lp/lpHolder.lean Outdated
Comment thread Mathlib/Analysis/Normed/Lp/lpSpace.lean Outdated
Comment thread Mathlib/Analysis/Normed/Lp/lpHolder.lean Outdated
Comment thread Mathlib/Analysis/Normed/Lp/lpHolder.lean Outdated
Comment thread Mathlib/Analysis/Normed/Lp/lpHolder.lean Outdated
Comment thread Mathlib/Analysis/Normed/Lp/lpHolder.lean Outdated
j-loreaux and others added 2 commits April 21, 2026 16:30
Co-authored-by: Michal Swietek <4404982+mike1729@users.noreply.github.com>
Co-authored-by: Jireh Loreaux <loreaujy@gmail.com>
@mathlib-triage mathlib-triage Bot assigned themathqueen and unassigned urkud May 15, 2026
Comment thread Mathlib/Analysis/Normed/Lp/lpHolder.lean
Comment thread Mathlib/Analysis/Normed/Lp/lpHolder.lean Outdated
@[simps!]
noncomputable def holderₗ (B : (i : ι) → E i →L[𝕜] F i →L[𝕜] G i) {K : ℝ} (hBK : ∀ i, ‖B i‖ ≤ K) :
lp E p →ₗ[𝕜] lp F q →ₗ[𝕜] lp G r :=
.mk₂ 𝕜 (holder r B hBK) ?_ ?_ ?_ ?_ where finally

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There is a PR about the convention for where and finally: leanprover-community/leanprover-community.github.io#844
This was my take from the zulip discussion, but if you prefer your current version, then feel free to revive the discussion

@mathlib-triage mathlib-triage Bot assigned sgouezel and unassigned themathqueen May 31, 2026

@dupuisf dupuisf left a comment

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.

Looks good!

bors d+

Comment on lines +67 to +69
lemma norm_tsumCLM_le {α 𝕜 E : Type*} [NontriviallyNormedField 𝕜]
[NormedAddCommGroup E] [NormedSpace 𝕜 E] [CompleteSpace E] :
‖tsumCLM 𝕜 α E‖ ≤ 1 :=

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.

This looks a bit out of place in this file, could we put it in lpSpace instead?

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.

This was discussed earlier in the review comments. Doing so would increase the imports of that file more than I would prefer to do.

@mathlib-bors

mathlib-bors Bot commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

✌️ j-loreaux can now approve this pull request until 2026-06-16 14:25 UTC (in 2 weeks). To approve and merge, reply with bors r+. More detailed instructions are available here.

@mathlib-triage mathlib-triage Bot added the delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). label Jun 2, 2026
@j-loreaux

Copy link
Copy Markdown
Contributor Author

bors merge

@mathlib-triage mathlib-triage Bot added the ready-to-merge This PR has been sent to bors. label Jun 4, 2026
mathlib-bors Bot pushed a commit that referenced this pull request Jun 4, 2026
Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>
@mathlib-bors

mathlib-bors Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors Bot changed the title feat: Hölder framework for lp spaces [Merged by Bors] - feat: Hölder framework for lp spaces Jun 4, 2026
@mathlib-bors mathlib-bors Bot closed this Jun 4, 2026
ReemMelamed pushed a commit to ReemMelamed/mathlib4 that referenced this pull request Jun 20, 2026
Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). ready-to-merge This PR has been sent to bors. t-analysis Analysis (normed *, calculus)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants