Skip to content

[Merged by Bors] - feat(LinearAlgebra/Transvection): the determinant of a transvection is equal to 1.#32757

Closed
AntoineChambert-Loir wants to merge 160 commits intoleanprover-community:masterfrom
AntoineChambert-Loir:ACL/SL-transvection-calcul
Closed

[Merged by Bors] - feat(LinearAlgebra/Transvection): the determinant of a transvection is equal to 1.#32757
AntoineChambert-Loir wants to merge 160 commits intoleanprover-community:masterfrom
AntoineChambert-Loir:ACL/SL-transvection-calcul

Conversation

@AntoineChambert-Loir
Copy link
Copy Markdown
Collaborator

@AntoineChambert-Loir AntoineChambert-Loir commented Dec 11, 2025

Prove that the determinant of a transvection is equal to 1

The proof goes by showing that the determinant of LinearMap.transvection f v is 1 + f v (even if f v is nonzero).

I first treat the case of a field, distinguishing whether f v = 0 (so that we get
a transvection) or not (so that we have a dilation).

Then, by base change to the field of fractions, I can handle the case of a domain. Finally, the general case is treated by base change from the “universal case” where the base ring is the ring of polynomials in $$2n$$ indeterminates corresponding to the coefficients of $$f$$ and $$v$$.


Open in Gitpod

Copy link
Copy Markdown
Contributor

@alreadydone alreadydone left a comment

Choose a reason for hiding this comment

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

Is it easier to use Matrix.det_of_upperTriangular?

@AntoineChambert-Loir
Copy link
Copy Markdown
Collaborator Author

Over a field, that's essentially what is done. But that wouldn't suffice over a ring.

@alreadydone
Copy link
Copy Markdown
Contributor

Oh, I see, it's because {v} may not extend to a basis.

@AntoineChambert-Loir
Copy link
Copy Markdown
Collaborator Author

Exactly!

@jsm28 jsm28 removed their assignment Jan 17, 2026
@jsm28 jsm28 added the t-algebra Algebra (groups, rings, fields, etc) label Jan 17, 2026
@dagurtomas dagurtomas assigned ocfnash and unassigned dagurtomas Jan 22, 2026
Copy link
Copy Markdown
Contributor

@ocfnash ocfnash left a comment

Choose a reason for hiding this comment

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

Thanks for this lovely work and also for your extraordinary patience!

bors d+

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jan 23, 2026

✌️ AntoineChambert-Loir can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@ghost ghost added the delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). label Jan 23, 2026
@github-actions github-actions bot added the large-import Automatically added label for PRs with a significant increase in transitive imports label Jan 23, 2026
@AntoineChambert-Loir
Copy link
Copy Markdown
Collaborator Author

bors r+

mathlib-bors bot pushed a commit that referenced this pull request Jan 23, 2026
…s equal to 1. (#32757)

Prove that the determinant of a transvection is equal to 1

The proof goes by showing that the determinant of `LinearMap.transvection f v` is `1 + f v` (even if `f v` is nonzero).

I first treat the case of a field, distinguishing whether `f v = 0` (so that we get
a transvection) or not (so that we have a dilation).

Then, by base change to the field of fractions, I can handle the case of a domain. Finally, the general case is treated by base change from the “universal case” where the base ring is the ring of polynomials in $$2n$$ indeterminates corresponding to the coefficients of $$f$$ and $$v$$.

Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jan 23, 2026

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(LinearAlgebra/Transvection): the determinant of a transvection is equal to 1. [Merged by Bors] - feat(LinearAlgebra/Transvection): the determinant of a transvection is equal to 1. Jan 23, 2026
@mathlib-bors mathlib-bors bot closed this Jan 23, 2026
goliath-klein pushed a commit to PrParadoxy/mathlib4 that referenced this pull request Jan 24, 2026
…s equal to 1. (leanprover-community#32757)

Prove that the determinant of a transvection is equal to 1

The proof goes by showing that the determinant of `LinearMap.transvection f v` is `1 + f v` (even if `f v` is nonzero).

I first treat the case of a field, distinguishing whether `f v = 0` (so that we get
a transvection) or not (so that we have a dilation).

Then, by base change to the field of fractions, I can handle the case of a domain. Finally, the general case is treated by base change from the “universal case” where the base ring is the ring of polynomials in $$2n$$ indeterminates corresponding to the coefficients of $$f$$ and $$v$$.

Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>
YellPika pushed a commit to YellPika/mathlib4 that referenced this pull request Feb 3, 2026
…s equal to 1. (leanprover-community#32757)

Prove that the determinant of a transvection is equal to 1

The proof goes by showing that the determinant of `LinearMap.transvection f v` is `1 + f v` (even if `f v` is nonzero).

I first treat the case of a field, distinguishing whether `f v = 0` (so that we get
a transvection) or not (so that we have a dilation).

Then, by base change to the field of fractions, I can handle the case of a domain. Finally, the general case is treated by base change from the “universal case” where the base ring is the ring of polynomials in $$2n$$ indeterminates corresponding to the coefficients of $$f$$ and $$v$$.

Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). large-import Automatically added label for PRs with a significant increase in transitive imports t-algebra Algebra (groups, rings, fields, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants