Skip to content

[Merged by Bors] - feat: contMDiffOn_iUnion_iff_of_isOpen#26673

Closed
grunweg wants to merge 2 commits intoleanprover-community:masterfrom
grunweg:contMDiffOn_iUnion
Closed

[Merged by Bors] - feat: contMDiffOn_iUnion_iff_of_isOpen#26673
grunweg wants to merge 2 commits intoleanprover-community:masterfrom
grunweg:contMDiffOn_iUnion

Conversation

@grunweg
Copy link
Copy Markdown
Contributor

@grunweg grunweg commented Jul 3, 2025

A function is C^n on a union of open sets iff it is continuous on each individual set.
The ContinuousOn analogue of this is proven in #26672.

Part of the path towards the geodesics and the Levi-Civita connection.


Open in Gitpod

@grunweg grunweg mentioned this pull request Jul 3, 2025
35 tasks
@github-actions github-actions bot added the t-differential-geometry Manifolds etc label Jul 3, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 3, 2025

PR summary 7672327e9c

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ ContMDiffOn.iUnion_of_isOpen
+ ContMDiffOn.union_of_isOpen
+ contMDiffOn_iUnion_iff_of_isOpen
+ contMDiffOn_union_iff_of_isOpen
+ contMDiff_of_contMDiffOn_iUnion_of_isOpen
+ contMDiff_of_contMDiffOn_union_of_isOpen

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

Copy link
Copy Markdown
Contributor

@Ruben-VandeVelde Ruben-VandeVelde left a comment

Choose a reason for hiding this comment

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

maintainer delegate

variable {s t : Set M}

/-- If a function is `C^k` on two open sets, it is also `C^n` on their union. -/
lemma ContMDiffOn.union_of_isOpen (hf : ContMDiffOn I I' n f s) (hf' : ContMDiffOn I I' n f t)
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.

Question for all of those: is _of_isOpen needed? Are there other true lemmas you'd confuse them with if you removed the suffix?

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.

Good question, I'm honestly not sure.
Given that this lemma has four different cousins with the same issue (ContinuousOn, DifferentiableOn, MDifferentiableOn and ContDiffOn), I'd rather land this PR as-is, being consistent with the others. But feel free to make a PR remaining all these 8 theorems (and I can review that one quickly)!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 3, 2025

🚀 Pull request has been placed on the maintainer queue by Ruben-VandeVelde.

@ghost ghost added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Jul 3, 2025
mathlib-bors bot pushed a commit that referenced this pull request Jul 3, 2025
A function is continuous on a union of open sets iff it is continuous on each individual set.
This extends the results in #22684 to arbitrary unions; the `ContMDiffOn` analogue is proven in #26673.
@sgouezel
Copy link
Copy Markdown
Contributor

sgouezel commented Jul 3, 2025

bors d+

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jul 3, 2025

✌️ grunweg can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@ghost ghost added delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). and removed maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. labels Jul 3, 2025
mathlib-bors bot pushed a commit that referenced this pull request Jul 3, 2025
Analogue of #26673 for `ContDifferentiableOn`.

Transitive mathlib clean-up from the path towards geodesics and the Levi-Civita connection.
mathlib-bors bot pushed a commit that referenced this pull request Jul 3, 2025
Analogue of #26673 for `DifferentiableOn`.

Transitive mathlib clean-up from the path towards geodesics and the Levi-Civita connection.
mathlib-bors bot pushed a commit that referenced this pull request Jul 3, 2025
Analogue of #26673 for `MDifferentiableOn`.

Transitive mathlib clean-up from the path towards geodesics and the Levi-Civita connection.
@grunweg
Copy link
Copy Markdown
Contributor Author

grunweg commented Jul 4, 2025

Thanks for the reviews!
bors merge

mathlib-bors bot pushed a commit that referenced this pull request Jul 4, 2025
A function is `C^n` on a union of open sets iff it is continuous on each individual set.
The `ContinuousOn` analogue of this is proven in #26672.

Part of the path towards the geodesics and the Levi-Civita connection.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jul 4, 2025

Build failed (retrying...):

mathlib-bors bot pushed a commit that referenced this pull request Jul 4, 2025
A function is `C^n` on a union of open sets iff it is continuous on each individual set.
The `ContinuousOn` analogue of this is proven in #26672.

Part of the path towards the geodesics and the Levi-Civita connection.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jul 4, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: contMDiffOn_iUnion_iff_of_isOpen [Merged by Bors] - feat: contMDiffOn_iUnion_iff_of_isOpen Jul 4, 2025
@mathlib-bors mathlib-bors bot closed this Jul 4, 2025
joelriou pushed a commit to joelriou/mathlib4 that referenced this pull request Jul 7, 2025
A function is continuous on a union of open sets iff it is continuous on each individual set.
This extends the results in leanprover-community#22684 to arbitrary unions; the `ContMDiffOn` analogue is proven in leanprover-community#26673.
joelriou pushed a commit to joelriou/mathlib4 that referenced this pull request Jul 7, 2025
Analogue of leanprover-community#26673 for `ContDifferentiableOn`.

Transitive mathlib clean-up from the path towards geodesics and the Levi-Civita connection.
joelriou pushed a commit to joelriou/mathlib4 that referenced this pull request Jul 7, 2025
Analogue of leanprover-community#26673 for `DifferentiableOn`.

Transitive mathlib clean-up from the path towards geodesics and the Levi-Civita connection.
joelriou pushed a commit to joelriou/mathlib4 that referenced this pull request Jul 7, 2025
Analogue of leanprover-community#26673 for `MDifferentiableOn`.

Transitive mathlib clean-up from the path towards geodesics and the Levi-Civita connection.
joelriou pushed a commit to joelriou/mathlib4 that referenced this pull request Jul 7, 2025
A function is `C^n` on a union of open sets iff it is continuous on each individual set.
The `ContinuousOn` analogue of this is proven in leanprover-community#26672.

Part of the path towards the geodesics and the Levi-Civita connection.
callesonne pushed a commit to callesonne/mathlib4 that referenced this pull request Jul 24, 2025
A function is continuous on a union of open sets iff it is continuous on each individual set.
This extends the results in leanprover-community#22684 to arbitrary unions; the `ContMDiffOn` analogue is proven in leanprover-community#26673.
callesonne pushed a commit to callesonne/mathlib4 that referenced this pull request Jul 24, 2025
Analogue of leanprover-community#26673 for `ContDifferentiableOn`.

Transitive mathlib clean-up from the path towards geodesics and the Levi-Civita connection.
callesonne pushed a commit to callesonne/mathlib4 that referenced this pull request Jul 24, 2025
Analogue of leanprover-community#26673 for `DifferentiableOn`.

Transitive mathlib clean-up from the path towards geodesics and the Levi-Civita connection.
callesonne pushed a commit to callesonne/mathlib4 that referenced this pull request Jul 24, 2025
Analogue of leanprover-community#26673 for `MDifferentiableOn`.

Transitive mathlib clean-up from the path towards geodesics and the Levi-Civita connection.
callesonne pushed a commit to callesonne/mathlib4 that referenced this pull request Jul 24, 2025
A function is `C^n` on a union of open sets iff it is continuous on each individual set.
The `ContinuousOn` analogue of this is proven in leanprover-community#26672.

Part of the path towards the geodesics and the Levi-Civita connection.
hrmacbeth pushed a commit to szqzs/mathlib4 that referenced this pull request Jul 28, 2025
A function is continuous on a union of open sets iff it is continuous on each individual set.
This extends the results in leanprover-community#22684 to arbitrary unions; the `ContMDiffOn` analogue is proven in leanprover-community#26673.
hrmacbeth pushed a commit to szqzs/mathlib4 that referenced this pull request Jul 28, 2025
Analogue of leanprover-community#26673 for `ContDifferentiableOn`.

Transitive mathlib clean-up from the path towards geodesics and the Levi-Civita connection.
hrmacbeth pushed a commit to szqzs/mathlib4 that referenced this pull request Jul 28, 2025
Analogue of leanprover-community#26673 for `DifferentiableOn`.

Transitive mathlib clean-up from the path towards geodesics and the Levi-Civita connection.
hrmacbeth pushed a commit to szqzs/mathlib4 that referenced this pull request Jul 28, 2025
Analogue of leanprover-community#26673 for `MDifferentiableOn`.

Transitive mathlib clean-up from the path towards geodesics and the Levi-Civita connection.
hrmacbeth pushed a commit to szqzs/mathlib4 that referenced this pull request Jul 28, 2025
A function is `C^n` on a union of open sets iff it is continuous on each individual set.
The `ContinuousOn` analogue of this is proven in leanprover-community#26672.

Part of the path towards the geodesics and the Levi-Civita connection.
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). t-differential-geometry Manifolds etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants