Skip to content

feat: extended charts are local diffeomorphisms on their source#9273

Closed
grunweg wants to merge 42 commits intomasterfrom
MR-charts-local-diffeos-v3
Closed

feat: extended charts are local diffeomorphisms on their source#9273
grunweg wants to merge 42 commits intomasterfrom
MR-charts-local-diffeos-v3

Conversation

@grunweg
Copy link
Copy Markdown
Contributor

@grunweg grunweg commented Dec 25, 2023


XXX: is this PR worth keeping, given isInvertible_mfderiv_extChartAt? shows that extended charts have invertible differential (regardless of whether they are an interior point)?
Perhaps, all one should add is "the differential as an isomorphism..."

TODO: I think this should hold at any interior point - as the inverse of the extended chart, restricted to the interior of its source, will be a local inverse.

The analogous result for un-extended charts is shown in #8160.

Unresolved questions

Open in Gitpod

…rentials:

- make variables implicit;
- make more variables implicit and re-use some other variables.
Move Diffeomorph.toPartialDiffeomorph further down.
Touch up the docs; introduce PartialDiffeomorph.toLocalHomeomorph.
and PartialHomeomorph.continuous_{to,inv}Fun to continuousOn_{to,inv}Fun.
might need to read up on the variable command.
This is mathematically cleaner (I show injectivity and surjectivity of the
differential separately), cleaner in Lean (it's good practice to avoid heavy
tactic blocks in definitions) and makes all proofs by rfl work.

This requires completeness of the tangent spaces; this is slightly unfortunate,
as it was automatic in the previous approaches. Opinions welcome.

I'm a bit unsure about the local instances; we'll see if this can be avoided.
@ghost ghost added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Dec 25, 2023
@ghost ghost added the blocked-by-other-PR label May 28, 2024
@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 Dec 19, 2024
@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label May 8, 2025
@mathlib4-dependent-issues-bot
Copy link
Copy Markdown
Collaborator

@leanprover-community leanprover-community deleted a comment May 8, 2025
@grunweg grunweg added the awaiting-author A reviewer has asked the author a question or requested changes. label Oct 13, 2025
@grunweg grunweg mentioned this pull request Dec 10, 2025
5 tasks
@grunweg
Copy link
Copy Markdown
Contributor Author

grunweg commented Feb 15, 2026

I don't think this PR is still useful; the correct proof is to use the invertibility of their mfderiv.

@grunweg grunweg closed this Feb 15, 2026
@grunweg grunweg deleted the MR-charts-local-diffeos-v3 branch February 15, 2026 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-author A reviewer has asked the author a question or requested changes. merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) t-differential-geometry Manifolds etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants