Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Merged by Bors] - feat(LinearAlgebra/CliffordAlgebra): port SpinGroup #9111

Closed
wants to merge 58 commits into from

Conversation

utensil
Copy link
Collaborator

@utensil utensil commented Dec 16, 2023

In this PR, we define lipschitzGroup, pinGroup and spinGroup, and prove some basic lemmas.

Ported from leanprover-community/mathlib3/pull/16040.

Co-authored-by: Jiale Miao [email protected]
Co-authored-by: Eric Wieser [email protected]


Open in Gitpod

This PR is a port of leanprover-community/mathlib3#16040 .

This PR can be reviewed here or discussed on Zulip.

utensil and others added 3 commits December 17, 2023 00:33
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@utensil utensil changed the title feat: port LinearAlgebra.CliffordAlgebra.SpinGroup feat(LinearAlgebra/CliffordAlgebra): port SpinGroup Dec 16, 2023
@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added the blocked-by-other-PR This PR depends on another PR to Mathlib (this label is automatically managed by a bot) label Dec 16, 2023
simp only [SMul.smul, SetLike.mem_coe, LinearMap.mem_range, DistribMulAction.toLinearMap_apply,
ConjAct.ofConjAct_toConjAct] at ha
rcases ha with ⟨⟨b, hb⟩, ha1⟩
(hx : x ∈ lipschitz Q) : ConjAct.toConjAct x • LinearMap.range (ι Q) ≤ LinearMap.range (ι Q) := by
Copy link
Collaborator Author

@utensil utensil Dec 17, 2023

Choose a reason for hiding this comment

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

@_Eric Wieser|310045 said:

Having dot notation work on bundled morphisms like this was often convenient in Lean 3; it would be a shame to lose it

@_Yaël Dillies|387244 said:

This is the general issue with bundled morphisms not taking dot notation anymore. This is really annoying.

@github-actions github-actions bot removed the awaiting-author A reviewer has asked the author a question or requested changes label Aug 19, 2024
mathlib-bors bot pushed a commit that referenced this pull request Aug 19, 2024
In this PR, we define `lipschitzGroup`, `pinGroup` and `spinGroup`, and prove some basic lemmas.

Ported from leanprover-community/mathlib3/pull/16040.

Co-authored-by: Jiale Miao <[email protected]>
Co-authored-by: Eric Wieser <[email protected]>



Co-authored-by: utensil <[email protected]>
Co-authored-by: Utensil <[email protected]>
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Aug 19, 2024

Build failed (retrying...):

mathlib-bors bot pushed a commit that referenced this pull request Aug 19, 2024
In this PR, we define `lipschitzGroup`, `pinGroup` and `spinGroup`, and prove some basic lemmas.

Ported from leanprover-community/mathlib3/pull/16040.

Co-authored-by: Jiale Miao <[email protected]>
Co-authored-by: Eric Wieser <[email protected]>



Co-authored-by: utensil <[email protected]>
Co-authored-by: Utensil <[email protected]>
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Aug 19, 2024

Build failed (retrying...):

mathlib-bors bot pushed a commit that referenced this pull request Aug 19, 2024
In this PR, we define `lipschitzGroup`, `pinGroup` and `spinGroup`, and prove some basic lemmas.

Ported from leanprover-community/mathlib3/pull/16040.

Co-authored-by: Jiale Miao <[email protected]>
Co-authored-by: Eric Wieser <[email protected]>



Co-authored-by: utensil <[email protected]>
Co-authored-by: Utensil <[email protected]>
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Aug 19, 2024

Build failed (retrying...):

mathlib-bors bot pushed a commit that referenced this pull request Aug 19, 2024
In this PR, we define `lipschitzGroup`, `pinGroup` and `spinGroup`, and prove some basic lemmas.

Ported from leanprover-community/mathlib3/pull/16040.

Co-authored-by: Jiale Miao <[email protected]>
Co-authored-by: Eric Wieser <[email protected]>



Co-authored-by: utensil <[email protected]>
Co-authored-by: Utensil <[email protected]>
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Aug 19, 2024

Build failed:

@eric-wieser
Copy link
Member

My personal opinion is that this PR has been in an acceptable state for a(n unacceptably) long time, and that the Subgroup vs Submonoid design choice is not something that should prevent future work from happening. I agree that this can be figured out in a later PR. So...

I agree with this, but didn't find the time to come back and say it. Sorry for letting this drag on so long @utensil!

Copy link

github-actions bot commented Aug 19, 2024

PR summary 1a12eaa0eb

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference
Mathlib.LinearAlgebra.CliffordAlgebra.SpinGroup 1258

Declarations diff

+ coe_mem_iff_mem
+ instance : Group (pinGroup Q)
+ instance : Group (spinGroup Q)
+ instance : Inhabited (pinGroup Q)
+ instance : Inhabited (spinGroup Q)
+ instance : Star (pinGroup Q)
+ instance : Star (spinGroup Q)
+ instance : StarMul (pinGroup Q)
+ instance : StarMul (spinGroup Q)
+ involute_eq
+ lipschitzGroup
+ mem_even
+ mem_lipschitzGroup
+ mem_pin
+ mem_unitary
+ pinGroup
+ spinGroup
+ units_involute_act_eq_conjAct
+ units_mem_iff
++ coe_mul_star_self
++ coe_star
++ coe_star_mul_self
++ mem_iff
++ mul_star_self
++ mul_star_self_of_mem
++ star_eq_inv
++ star_eq_inv'
++ star_mem
++ star_mem_iff
++ star_mul_self
++ star_mul_self_of_mem
++ toUnits
++ toUnits_injective
++ units_mem_lipschitzGroup
+++ conjAct_smul_range_ι
+++ conjAct_smul_ι_mem_range_ι
+++ involute_act_ι_mem_range_ι

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.

@eric-wieser
Copy link
Member

bors merge

mathlib-bors bot pushed a commit that referenced this pull request Aug 19, 2024
In this PR, we define `lipschitzGroup`, `pinGroup` and `spinGroup`, and prove some basic lemmas.

Ported from leanprover-community/mathlib3/pull/16040.

Co-authored-by: Jiale Miao <[email protected]>
Co-authored-by: Eric Wieser <[email protected]>



Co-authored-by: utensil <[email protected]>
Co-authored-by: Utensil <[email protected]>
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Aug 19, 2024

Build failed:

@bryangingechen
Copy link
Contributor

Weird, looks like one of the runners went offline before the build finished: https://github.com/leanprover-community/mathlib4/actions/runs/10460865400

Let's try again.
bors r+

mathlib-bors bot pushed a commit that referenced this pull request Aug 19, 2024
In this PR, we define `lipschitzGroup`, `pinGroup` and `spinGroup`, and prove some basic lemmas.

Ported from leanprover-community/mathlib3/pull/16040.

Co-authored-by: Jiale Miao <[email protected]>
Co-authored-by: Eric Wieser <[email protected]>



Co-authored-by: utensil <[email protected]>
Co-authored-by: Utensil <[email protected]>
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Aug 19, 2024

Build failed (retrying...):

mathlib-bors bot pushed a commit that referenced this pull request Aug 19, 2024
In this PR, we define `lipschitzGroup`, `pinGroup` and `spinGroup`, and prove some basic lemmas.

Ported from leanprover-community/mathlib3/pull/16040.

Co-authored-by: Jiale Miao <[email protected]>
Co-authored-by: Eric Wieser <[email protected]>



Co-authored-by: utensil <[email protected]>
Co-authored-by: Utensil <[email protected]>
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Aug 19, 2024

Build failed:

@bryangingechen
Copy link
Contributor

One of the runners ran out of space. Re-running
bors r+

mathlib-bors bot pushed a commit that referenced this pull request Aug 20, 2024
In this PR, we define `lipschitzGroup`, `pinGroup` and `spinGroup`, and prove some basic lemmas.

Ported from leanprover-community/mathlib3/pull/16040.

Co-authored-by: Jiale Miao <[email protected]>
Co-authored-by: Eric Wieser <[email protected]>



Co-authored-by: utensil <[email protected]>
Co-authored-by: Utensil <[email protected]>
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Aug 20, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(LinearAlgebra/CliffordAlgebra): port SpinGroup [Merged by Bors] - feat(LinearAlgebra/CliffordAlgebra): port SpinGroup Aug 20, 2024
@mathlib-bors mathlib-bors bot closed this Aug 20, 2024
@mathlib-bors mathlib-bors bot deleted the utensil/SpinGroup branch August 20, 2024 00:28
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 9, 2024
In this PR, we define `lipschitzGroup`, `pinGroup` and `spinGroup`, and prove some basic lemmas.

Ported from leanprover-community/mathlib3/pull/16040.

Co-authored-by: Jiale Miao <[email protected]>
Co-authored-by: Eric Wieser <[email protected]>



Co-authored-by: utensil <[email protected]>
Co-authored-by: Utensil <[email protected]>
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 9, 2024
In this PR, we define `lipschitzGroup`, `pinGroup` and `spinGroup`, and prove some basic lemmas.

Ported from leanprover-community/mathlib3/pull/16040.

Co-authored-by: Jiale Miao <[email protected]>
Co-authored-by: Eric Wieser <[email protected]>



Co-authored-by: utensil <[email protected]>
Co-authored-by: Utensil <[email protected]>
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 12, 2024
In this PR, we define `lipschitzGroup`, `pinGroup` and `spinGroup`, and prove some basic lemmas.

Ported from leanprover-community/mathlib3/pull/16040.

Co-authored-by: Jiale Miao <[email protected]>
Co-authored-by: Eric Wieser <[email protected]>



Co-authored-by: utensil <[email protected]>
Co-authored-by: Utensil <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintainer-merge ready-to-merge This PR has been sent to bors. t-algebra Algebra (groups, rings, fields, etc)
Projects
None yet
Development

Successfully merging this pull request may close these issues.