[Merged by Bors] - feat(LinearAlgebra/BilinearForm/TensorProduct): base change of bilinear forms #6306
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…asic-TensorProduct.congr
…asic-TensorProduct.congr
…asic-TensorProduct.congr
…ra instance This also moves some lemmas about `IsUnit (_ : Module.End R M)` to an earlier file as they are nothing to do with `Algebra`.
eric-wieser
added
awaiting-review
awaiting-CI
github-actions
bot
removed
the
awaiting-CI
leanprover-community-mathlib4-bot
added
the
merge-conflict
eric-wieser
removed
the
merge-conflict
leanprover-community-mathlib4-bot
added
the
merge-conflict
leanprover-community-mathlib4-bot
removed
the
merge-conflict
eric-wieser
force-pushed
the
eric-wieser/BilinForm.baseChange
branch
from
4151d50 to
6a10f7e
Compare
ocfnash
approved these changes
Aug 17, 2023
| variable [SMulCommClass R A M₁] [SMulCommClass A R M₁] [IsScalarTower R A M₁] | ||
| variable [Module R M₂] | ||
|
|
||
| /-- The tensor product of two bilinear forms injects into bilinear forms on tensor products. |
Contributor
There was a problem hiding this comment.
Suggested change
| /-- The tensor product of two bilinear forms injects into bilinear forms on tensor products. | |
| variable (R A) in | |
| /-- The tensor product of two bilinear forms injects into bilinear forms on tensor products. |
Comment on lines
42
to
43
| Note this is heterobasic; the bilinear form on the left can take values in a larger ring than | ||
| the one on the right. -/ |
Contributor
There was a problem hiding this comment.
Suggested change
| Note this is heterobasic; the bilinear form on the left can take values in a larger ring than | |
| the one on the right. -/ | |
| Note this is heterobasic; the bilinear form on the left can take values in an algebra over | |
| the ring in which the right bilinear form is valued. -/ |
Mathlib/LinearAlgebra/Dual.lean
Outdated
| end | ||
|
|
||
| namespace AlgebraTensorModule | ||
| set_option autoImplicit false |
Contributor
There was a problem hiding this comment.
Suggested change
| set_option autoImplicit false |
|
✌️ eric-wieser can now approve this pull request. To approve and merge a pull request, simply reply with |
leanprover-community-mathlib4-bot
added
delegated
Member
Author
|
bors merge |
bors bot
pushed a commit
that referenced
this pull request
Aug 17, 2023
…ar forms (#6306) This generalizes the existing `BilinForm.tensorDistrib` to be heterogenous in the rings it uses, such that a base change, ```lean protected def baseChange (B : BilinForm R M₂) : BilinForm A (A ⊗[R] M₂) := ``` falls out as a special case. I do not attempt to generalize `BilinForm.tensorDistribEquiv`. Unfortunately, this changes the defeq as ```diff -(B₁.tmul B₂) (m₁ ⊗ₜ m₂) (m₁' ⊗ₜ m₂') = B₁ m₁ m₁' * B₂ m₂ m₂' +(B₁.tmul B₂) (m₁ ⊗ₜ m₂) (m₁' ⊗ₜ m₂') = B₂ m₂ m₂' • B₁ m₁ m₁' ``` We could fix this by using the right action instead, but that's a lot of work for a very minorly annoying defeq. This also breaks the defeq of `tensorDistribEquiv B = tensorDistrib B`; though the reason is more complicated than the scalar action issue above. It would be fixed if we defined all the homogenous operations on tensor products as special cases of the heterogenous ones, but that's also a lot of work for a very small win. This is a port of work from pygae/lean-ga#31, and almost at the end of the path to a base change of quadratic forms and clifford algebras. This was independently developed at the Leiden workshop as [`BilinForm.baseChange`](https://github.com/alexjbest/ant-lorentz/blob/1f97add294b2d50f99537c15583666d78b0d7e24/AntLorentz/BaseChange.lean#L75-L85), though the results there are not sorry-free.
|
This PR was included in a batch that was canceled, it will be automatically retried |
bors bot
pushed a commit
that referenced
this pull request
Aug 17, 2023
…ar forms (#6306) This generalizes the existing `BilinForm.tensorDistrib` to be heterogenous in the rings it uses, such that a base change, ```lean protected def baseChange (B : BilinForm R M₂) : BilinForm A (A ⊗[R] M₂) := ``` falls out as a special case. I do not attempt to generalize `BilinForm.tensorDistribEquiv`. Unfortunately, this changes the defeq as ```diff -(B₁.tmul B₂) (m₁ ⊗ₜ m₂) (m₁' ⊗ₜ m₂') = B₁ m₁ m₁' * B₂ m₂ m₂' +(B₁.tmul B₂) (m₁ ⊗ₜ m₂) (m₁' ⊗ₜ m₂') = B₂ m₂ m₂' • B₁ m₁ m₁' ``` We could fix this by using the right action instead, but that's a lot of work for a very minorly annoying defeq. This also breaks the defeq of `tensorDistribEquiv B = tensorDistrib B`; though the reason is more complicated than the scalar action issue above. It would be fixed if we defined all the homogenous operations on tensor products as special cases of the heterogenous ones, but that's also a lot of work for a very small win. This is a port of work from pygae/lean-ga#31, and almost at the end of the path to a base change of quadratic forms and clifford algebras. This was independently developed at the Leiden workshop as [`BilinForm.baseChange`](https://github.com/alexjbest/ant-lorentz/blob/1f97add294b2d50f99537c15583666d78b0d7e24/AntLorentz/BaseChange.lean#L75-L85), though the results there are not sorry-free.
|
Pull request successfully merged into master. Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
bors
bot
changed the title
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This generalizes the existing
BilinForm.tensorDistribto be heterogenous in the rings it uses, such that a base change,falls out as a special case. I do not attempt to generalize
BilinForm.tensorDistribEquiv.Unfortunately, this changes the defeq as
We could fix this by using the right action instead, but that's a lot of work for a very minorly annoying defeq.
This also breaks the defeq of
tensorDistribEquiv B = tensorDistrib B; though the reason is more complicated than the scalar action issue above. It would be fixed if we defined all the homogenous operations on tensor products as special cases of the heterogenous ones, but that's also a lot of work for a very small win.This is a port of work from pygae/lean-ga#31, and almost at the end of the path to a base change of quadratic forms and clifford algebras.
This was independently developed at the Leiden workshop as
BilinForm.baseChange, though the results there are not sorry-free.