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

Banachsteinhaus #334

Merged
merged 18 commits into from
Sep 5, 2024
Merged

Banachsteinhaus #334

merged 18 commits into from
Sep 5, 2024

Conversation

mkerjean
Copy link
Collaborator

A proof of Baire and Banach-Steinhaus theorem, initiated by Theo Vignon.

Waiting for PR#183 to be merged to rebase on master.

@mkerjean mkerjean force-pushed the banachsteinhaus branch 3 times, most recently from 8f51f87 to 4c508d1 Compare February 18, 2021 22:26
@affeldt-aist
Copy link
Member

I tried to simplify a bit the scripts. Tell me if I introduced to much mess, Maybe it is worth doing one more pass and discuss whether to put lemmas in other files. Should le_closed_ball moved in normedtype.v (maybe using the homo notation). Should floor_nat_comp go to reals.v? Should bounded_landau go to landau.v? @CohenCyril

@mkerjean
Copy link
Collaborator Author

I tried to simplify a bit the scripts. Tell me if I introduced to much mess.
Seems good to me, thanks :)

@mkerjean
Copy link
Collaborator Author

I tried to simplify a bit the scripts. Tell me if I introduced to much mess, Maybe it is worth doing one more pass and discuss whether to put lemmas in other files. Should le_closed_ball moved in normedtype.v (maybe using the homo notation). Should floor_nat_comp go to reals.v? Should bounded_landau go to landau.v? @CohenCyril

As bounded_landau depends of the newly introduced defininition bounded_fun_norm, shouldn't it stay in near Banach-Steinhaus theorem for the time being ? And then Banach-Steinhaus needs to be inserted as a section/module in landau.v ?

@affeldt-aist
Copy link
Member

As bounded_landau depends of the newly introduced defininition bounded_fun_norm,
shouldn't it stay in near Banach-Steinhaus theorem for the time being ?

I was thinking that bounded_fun_norm would go to landau.v too.

And then Banach-Steinhaus needs to be inserted as a section/module in landau.v ?

Indeed, we wrongly stated that it should go to normedtype.v during the meeting. :-(

@affeldt-aist affeldt-aist force-pushed the banachsteinhaus branch 2 times, most recently from 0ae9d57 to a1f8ed0 Compare March 29, 2021 08:18
@affeldt-aist
Copy link
Member

Follow-up to Friday's meeting: I tried to replace floor_nat_comp with more primitive lemmas and the usage of ceil.

@mkerjean mkerjean force-pushed the banachsteinhaus branch 3 times, most recently from 6575bf8 to c09b12e Compare April 1, 2021 16:08
@mkerjean
Copy link
Collaborator Author

mkerjean commented Apr 1, 2021

At the end Baire's theorem depends on sequences, thus Baire's and Banach-Steinhauss theorem have been added in sequences.v, as well as the auxiliary definitions on bounded function (to be removed one day).

@affeldt-aist
Copy link
Member

NB: This PR uses the lemma gtz0_norm expecting it to be integrated into MathComp.
This is rather this lemma:

Lemma natr_absz i : `|i|%:R = `|i|%:~R :> R.
Proof. by rewrite -abszE. Qed.

that is likely to be merged into MathComp (see PR math-comp/math-comp#732).
Maybe update this PR accordingly?

@mkerjean
Copy link
Collaborator Author

mkerjean commented Feb 7, 2022

NB: This PR uses the lemma gtz0_norm expecting it to be integrated into MathComp. This is rather this lemma:

Lemma natr_absz i : `|i|%:R = `|i|%:~R :> R.
Proof. by rewrite -abszE. Qed.

that is likely to be merged into MathComp (see PR math-comp/math-comp#732). Maybe update this PR accordingly?

These lemmas have been integrated to MathComp 1.13 and Analysis seems to depend on math-comp 1.13, but I can't find them within sequence.v in this branch, any idea why ?

@CohenCyril
Copy link
Member

CohenCyril commented Feb 7, 2022

These lemmas have been integrated to MathComp 1.13 and Analysis seems to depend on math-comp 1.13, but I can't find them within sequence.v in this branch, any idea why ?

Hi! mathcomp-analysis is compatible with both (and with 1.14 too), and the default nix-shell uses the most conservative of the two (so 1.12.0).
I am strongly in support of dropping 1.12.0 ASAP, that would solve your issue and several of mine :)

@CohenCyril
Copy link
Member

Meanwhile is you use nix-shell --argstr bundle 8.14 it will use Coq 8.14 and force mathcomp to 1.13.

@mkerjean mkerjean mentioned this pull request Feb 8, 2022
@affeldt-aist affeldt-aist added the enhancement ✨ This issue/PR is about adding new features enhancing the library label Mar 15, 2023
@affeldt-aist affeldt-aist added this to the 1.2.0 milestone Apr 19, 2024
@affeldt-aist affeldt-aist modified the milestones: 1.2.0, 1.3.0 Jun 3, 2024
@affeldt-aist affeldt-aist modified the milestones: 1.3.0, 1.4.0 Jul 25, 2024
@affeldt-aist affeldt-aist merged commit 3828a15 into master Sep 5, 2024
22 checks passed
@affeldt-aist affeldt-aist deleted the banachsteinhaus branch December 2, 2024 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ✨ This issue/PR is about adding new features enhancing the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants