Skip to content

feat: a SliceModel typeclass for models with corners for embedded submanifolds#25505

Closed
grunweg wants to merge 15 commits intomasterfrom
MR-slicemodels2
Closed

feat: a SliceModel typeclass for models with corners for embedded submanifolds#25505
grunweg wants to merge 15 commits intomasterfrom
MR-slicemodels2

Conversation

@grunweg
Copy link
Copy Markdown
Contributor

@grunweg grunweg commented Jun 5, 2025

TODO: write a proper PR description (once things settle down a bit, and the dependent PRs have landed)
Then ask e.g. Sebastien for review.

One sorry left, about embeddings between subtypes.

Rebased and extended version of #24550:

  • on the current master branch
  • does not import the definition of smooth immersions/embeddings: this only defines slice models (no more)

Open in Gitpod

@grunweg grunweg added WIP Work in progress t-differential-geometry Manifolds etc labels Jun 5, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 5, 2025

PR summary 8b2141b23d

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference
Mathlib.Geometry.Manifold.SliceModel (new file) 1

Declarations diff

+ EuclideanSpace.finAddEquivProd
+ EuclideanSpace.sumEquivProd
+ SliceModel
+ SliceModel.ofEmbedding
+ instTrans
+ instance (h : (E × F) ≃L[𝕜] E') : SliceModel F (𝓘(𝕜, E)) (𝓘(𝕜, E'))
+ instance : SliceModel (⊥ : Subspace 𝕜 E) I I
+ instance [h : SliceModel F I I'] : SliceModel F (I.prod J) (I'.prod J)
+ instance [h : SliceModel F I I'] : SliceModel F (J.prod I) (J.prod I')
+ instance {n m : ℕ} [NeZero n] :
+ inverse
+ inverse_left_inv
+ inverse_right_inv
+ prodAssoc
+ prodAssoc_apply
+ prodAssoc_symm_apply
+ prodAssoc_toEquiv
+ prodAssoc_toLinearEquiv
+ prodUnique
+ prodUnique_apply
+ prodUnique_symm_apply
+ prodUnique_toEquiv
++ instance {n : ℕ} [NeZero n] :

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
Copy link
Copy Markdown
Collaborator

mathlib4-dependent-issues-bot commented Jun 6, 2025

@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 Jun 6, 2025
@grunweg grunweg changed the title Mr slicemodels2 feat: a SliceModel typeclass for models with corners for embedded submanifolds Jun 6, 2025
@grunweg
Copy link
Copy Markdown
Contributor Author

grunweg commented Jun 18, 2025

This PR has been migrated to a fork-based workflow: #26087

@grunweg grunweg closed this Jun 18, 2025
@grunweg grunweg deleted the MR-slicemodels2 branch June 18, 2025 17:59
mathlib-bors bot pushed a commit that referenced this pull request Jun 20, 2025
I need this for #25505 (to show that R^n is a submanifold of R^m for n \leq m) and for my bordism theory branch.

Previous zulip discussions:
- [#Is there code for X? > Product of Euclidean spaces @ 💬](https://leanprover.zulipchat.com/#narrow/channel/217875-Is-there-code-for-X.3F/topic/Product.20of.20Euclidean.20spaces/near/500124944)
- [#mathlib4 > Create &#96;LinearIsometryEquiv&#96; with &#96;WithLp 2 (α × β)&#96; and.. @ 💬](https://leanprover.zulipchat.com/#narrow/channel/287929-mathlib4/topic/Create.20.60LinearIsometryEquiv.60.20with.20.60WithLp.202.20.28.CE.B1.20.C3.97.20.CE.B2.29.60.20and.2E.2E/near/505938119)
- [#mathlib4 > Manifold structure on Moebius strip @ 💬](https://leanprover.zulipchat.com/#narrow/channel/287929-mathlib4/topic/Manifold.20structure.20on.20Moebius.20strip/near/522156122)

Co-authored-by: @peabrainiac
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) migrated-to-fork t-differential-geometry Manifolds etc WIP Work in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants