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

Minkowski #1000

Merged
merged 8 commits into from
Nov 13, 2023
Merged

Minkowski #1000

merged 8 commits into from
Nov 13, 2023

Conversation

hoheinzollern
Copy link
Collaborator

@hoheinzollern hoheinzollern commented Aug 1, 2023

Motivation for this change

This PR introduces Minkowski's inequality, a consequence of Hoelder's.

Things done/to do
  • added corresponding entries in CHANGELOG_UNRELEASED.md
  • added corresponding documentation in the headers
Compatibility with MathComp 2.0
  • I added the label TODO: HB port to make sure someone ports this PR to
    the hierarchy-builder branch or I already opened an issue or PR (please cross reference).
Automatic note to reviewers

Read this Checklist and put a milestone if possible.

@affeldt-aist affeldt-aist added this to the 0.6.5 milestone Aug 5, 2023
@hoheinzollern hoheinzollern force-pushed the minkowski branch 4 times, most recently from a9f7045 to cb3368e Compare August 10, 2023 10:07
@affeldt-aist affeldt-aist modified the milestones: 0.6.5, 0.6.6 Sep 28, 2023
@affeldt-aist
Copy link
Member

Minkowski should also hold with p = 1

@hoheinzollern
Copy link
Collaborator Author

Minkowski should also hold with p = 1

It does, I just updated the proof.

@hoheinzollern hoheinzollern force-pushed the minkowski branch 2 times, most recently from a70eaa8 to 5211206 Compare September 30, 2023 19:29
@affeldt-aist affeldt-aist mentioned this pull request Oct 1, 2023
3 tasks
@hoheinzollern
Copy link
Collaborator Author

I tried to simplify the proof after our meeting, but it's impossible to merge the "ge0_le_integral" steps as proposed in the meeting as they are interleaved with other operations that are incompatible

@@ -924,6 +943,21 @@ Qed.
Lemma poweR_eq0_eq0 x r : 0 <= x -> x `^ r = 0 -> x = 0.
Proof. by move=> + /eqP => /poweR_eq0-> /andP[/eqP]. Qed.

Lemma gt0_ler_poweR (r : R) : (0 <= r)%R ->
{in `[0, +oo] &, {homo poweR ^~ r : x y / x <= y >-> x <= y}}.
Copy link
Member

@affeldt-aist affeldt-aist Oct 27, 2023

Choose a reason for hiding this comment

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

Maybe we should use Rpos.nneg here? I misread.

@@ -87,7 +85,15 @@ under eq_integral => x _ do rewrite ger0_norm ?powR_ge0//.
by rewrite fp//; apply: integral_ge0 => t _; rewrite lee_fin powR_ge0.
Qed.

Lemma Lnorm_powR_K f (p : R) : p != 0%R ->
Copy link
Member

Choose a reason for hiding this comment

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

This is maybe not a good name, I looked at the proof steps using this lemma but couldn't come with an improvement. :-(

Copy link
Member

Choose a reason for hiding this comment

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

At least, powR_Lnorm would be better (head symbol goes first).

@affeldt-aist affeldt-aist self-requested a review October 28, 2023 13:37
Copy link
Member

@affeldt-aist affeldt-aist left a comment

Choose a reason for hiding this comment

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

We spent enough time on this one I think.
I'll merge soon unless there are comments.

@affeldt-aist affeldt-aist merged commit d7cdecc into math-comp:master Nov 13, 2023
30 checks passed
affeldt-aist pushed a commit to affeldt-aist/analysis that referenced this pull request Nov 14, 2023
* Minkowski's inequality and accessory lemmas

---------
Co-authored-by: Alessandro Bruni <[email protected]>
Co-authored-by: Reynald Affeldt <[email protected]>
proux01 pushed a commit that referenced this pull request Nov 15, 2023
* Minkowski's inequality and accessory lemmas

---------
Co-authored-by: Alessandro Bruni <[email protected]>
Co-authored-by: Reynald Affeldt <[email protected]>
@proux01 proux01 added TODO: MC2 port This PR must be ported to mathcomp 2 now that the. Remove this label when the port is done. and removed TODO: MC2 port This PR must be ported to mathcomp 2 now that the. Remove this label when the port is done. labels Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants