Skip to content

feat(LinearAlgebra/Multilinear/DirectSum, LinearAlgebra/DirectSum/{PiTensorProduct,Finsupp}): interaction between MultilinearMap and DirectSum, and between PiTensorProduct and DirectSum#11155

Closed
smorel394 wants to merge 44 commits intomasterfrom
SM.PiTensorProduct.DirectSum
Closed

feat(LinearAlgebra/Multilinear/DirectSum, LinearAlgebra/DirectSum/{PiTensorProduct,Finsupp}): interaction between MultilinearMap and DirectSum, and between PiTensorProduct and DirectSum#11155
smorel394 wants to merge 44 commits intomasterfrom
SM.PiTensorProduct.DirectSum

Conversation

@smorel394
Copy link
Copy Markdown
Collaborator

@smorel394 smorel394 commented Mar 4, 2024

This does three things:

  • MultilinearMap.fromDirectSumEquiv (in LinearAlgebra/MultilinearMap/DirectSum.lean) calculates MultilinearMaps on a family of DirectSums. More precisely, if ι is a Fintype, if κ i is a family of types indexed by ι and we have a R-module M i j for every i : ι and j : κ i, this is the linear equivalence between Π p : (i : ι) → κ i, MultilinearMap R (fun i ↦ M i (p i)) M' and MultilinearMap R (fun i ↦ ⨁ j : κ i, M i j) M'.
  • PiTensorProduct.directSum (in LinearAlgebra/DirectSum/PiTensorProduct.lean): the distributivity property of a PiTensorProduct with respect to DirectSums in all its arguments.
  • finsuppPiTensorProduct (in LinearAlgebra/DirectSum/Finsupp.lean): If ι is a Fintype, κ i is a family of types indexed by ι and M i is a family of R-modules indexed by ι, then the tensor product of the family κ i →₀ M i is linearly equivalent to ∏ i, κ i →₀ ⨂[R] i, M i.

Open in Gitpod

@smorel394 smorel394 added awaiting-review t-algebra Algebra (groups, rings, fields, etc) blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) labels Mar 4, 2024
@smorel394 smorel394 changed the title feat(LinearAlgebra/Multilinear/DirectSum, LinearAlgebra/DirectSum/PiTensorProduct,Finsupp): interaction between MultilinearMap and DirectSum, and between PiTensorProduct and DirectSum feat(LinearAlgebra/Multilinear/DirectSum, LinearAlgebra/DirectSum/{PiTensorProduct,Finsupp}): interaction between MultilinearMap and DirectSum, and between PiTensorProduct and DirectSum Mar 5, 2024
@ghost ghost added merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) and removed blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) labels Mar 8, 2024
@ghost ghost removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 8, 2024
smorel394 and others added 2 commits March 9, 2024 17:53
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
@smorel394 smorel394 added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Mar 9, 2024
@ghost ghost removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Mar 23, 2024
@leanprover-community-bot-assistant leanprover-community-bot-assistant removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Oct 27, 2024
(by
ext f x
dsimp
erw [MultilinearMap.dfinsuppFamily_single, DirectSum.toModule_lof]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

These erws are my fault, and are because we are abusing defeq by using MultilinearMap.dfinsuppFamilyₗ instead of duplicating the construction for directSum. Probably we should spin out a PR to do that, where all the defs and lemmas are one-liners in terms of the dfinsupp versions.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I've done that in this PR for now, but will come back and split later.

@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 Nov 13, 2024
@github-actions github-actions bot removed the large-import Automatically added label for PRs with a significant increase in transitive imports label Nov 15, 2024
@leanprover-community-bot-assistant leanprover-community-bot-assistant removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Nov 15, 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 Jan 15, 2025
@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 May 24, 2025
@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 Nov 19, 2025
@mathlib4-merge-conflict-bot
Copy link
Copy Markdown
Collaborator

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

mathlib-bors bot pushed a commit that referenced this pull request Dec 4, 2025
Adds a basis for MulilinearMap given bases for the domain and codomain.

This PR builds upon work started in #25141 toward #11155.

Co-authored-by: Oliver Nash <7734364+ocfnash@users.noreply.github.com>
Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
Co-authored-by: Eric Wieser <efw@google.com>
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
Co-authored-by: Oliver Nash <github@olivernash.org>
@ocfnash
Copy link
Copy Markdown
Contributor

ocfnash commented Dec 17, 2025

Thanks to @morrison-daniel I believe all of this work is now in master and we can close this PR 🎉

In particular I highlight the PRs #32456 and #32450 and can point to:

etc

@ocfnash ocfnash closed this Dec 17, 2025
@YaelDillies YaelDillies deleted the SM.PiTensorProduct.DirectSum branch December 21, 2025 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) please-adopt Inactive PR (would be valuable to adopt) t-algebra Algebra (groups, rings, fields, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants