Skip to content

feat: immersions are locally embeddings#28905

Open
grunweg wants to merge 40 commits intoleanprover-community:masterfrom
grunweg:immersions-locallyembeddings
Open

feat: immersions are locally embeddings#28905
grunweg wants to merge 40 commits intoleanprover-community:masterfrom
grunweg:immersions-locallyembeddings

Conversation

@grunweg
Copy link
Copy Markdown
Contributor

@grunweg grunweg commented Aug 25, 2025

Hopefully, this can be used for the topological portion of #28865. (That part is not certain yet, the rest works.)


Needs some clean-up in both the immersions file, and need to move the helper results about embeddings to the correct location.

Open in Gitpod

@github-actions
Copy link
Copy Markdown

github-actions bot commented Aug 25, 2025

PR summary 54bd5cb866

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.Geometry.Manifold.Immersion 1964 1971 +7 (+0.36%)
Mathlib.Geometry.Manifold.ContMDiff.Atlas 1954 1957 +3 (+0.15%)
Import changes for all files
Files Import difference
Mathlib.Geometry.Manifold.ContMDiff.Atlas 3
Mathlib.Geometry.Manifold.Immersion 7

Declarations diff

+ ContDiffWithinAt.fst
+ ContDiffWithinAt.snd
+ IsEmbedding.subtype_map_of_subset
+ _root_.Topology.IsEmbedding.comp_restrict
+ contDiffAt_prod_iff
+ contDiffOn_prod_iff
+ contDiffWithinAt_prod_iff
+ contDiff_prod_iff
+ contMDiffOn_extend
+ contMDiffOn_writtenInExtend_iff
+ contMDiffWithinAt_writtenInExtend_iff
+ eqOn_domChart_source
+ extend_target_eq_image_source
+ isClosedEmbedding_symm_restrict
+ isEmbedding_extend_restrict
+ isEmbedding_extend_symm_restrict
+ isEmbedding_restr_domChart_source
+ isEmbedding_symm_restrict
+ rightInverse_restrict
++ contMDiffAt
++ continuousAt
++ exists_nbhd_restr_isEmbedding
++ mapsto_domChart_source_codChart_source
+++ contMDiff

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

@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Aug 25, 2025
@mathlib4-dependent-issues-bot
Copy link
Copy Markdown
Collaborator

mathlib4-dependent-issues-bot commented Aug 25, 2025

@grunweg grunweg force-pushed the immersions-locallyembeddings branch from fcc5d4d to 8e35b1c Compare August 28, 2025 09:45
@mathlib4-merge-conflict-bot mathlib4-merge-conflict-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 Sep 8, 2025
@mathlib4-merge-conflict-bot
Copy link
Copy Markdown
Collaborator

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

@grunweg grunweg force-pushed the immersions-locallyembeddings branch from abf0809 to 28ba7ba Compare December 18, 2025 16:51
@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 Dec 18, 2025
@grunweg grunweg added the t-differential-geometry Manifolds etc label Jan 12, 2026
@mathlib4-merge-conflict-bot
Copy link
Copy Markdown
Collaborator

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

@mathlib4-merge-conflict-bot mathlib4-merge-conflict-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 Jan 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) 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.

3 participants