Skip to content

feat: Add first EvalT and ProdT Interaction Lemma - #1344

Merged
jstoobysmith merged 16 commits into
leanprover-community:masterfrom
NicolaBernini:feat/add-first-EvalT-ProdT-Interaction-Lemma-1July2026
Jul 4, 2026
Merged

feat: Add first EvalT and ProdT Interaction Lemma#1344
jstoobysmith merged 16 commits into
leanprover-community:masterfrom
NicolaBernini:feat/add-first-EvalT-ProdT-Interaction-Lemma-1July2026

Conversation

@NicolaBernini

Copy link
Copy Markdown
Collaborator

Overview

Add first EvalT and ProdT Interaction Lemma

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Thank you for this PR, which will now be reviewed. If submitting to ./Physlib or ./QuantumInfo, please see our review guidelines if you are not familiar with the process. You should expect a back and forth with a reviewer before your PR is merged. See also that link for how to add appropriate labels to your PR. The PR will also go through a number of automated checks. You can learn more about these here, including how to run them locally.

If you are submitting to ./PhyslibAlpha there will be a lighter review process, though your PR must still pass the automated checks.

If you want to bring attention to this PR, please write a message on this thread of the Lean Zulip.

Important: If a reviewer adds an awaiting-author label to your PR, once you have addressed the review comments, please remove that label by adding a comment with -awaiting-author. This helps us keep track of reviews.

NicolaBernini and others added 8 commits July 1, 2026 19:47
Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
@jstoobysmith jstoobysmith added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-author A reviewer has asked the author a question or requested changes labels Jul 3, 2026
lemma evalT_prodT_right {n n1 : ℕ} {c : Fin n → C} {c1 : Fin (n1 + 1) → C}
(i : Fin (n1 + 1)) (x : basisIdx (c1 i)) (t : Tensor S c) (t1 : Tensor S c1) :
permT id (IsReindexing.append_succAbove_natAdd (n := n) (n1 := n1) i)
(evalT (Fin.natAdd (m := n1 + 1) n i) (basisIdxCongr (by simp) x) (prodT t t1)) =

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 would swap the equality around so that what is currently on the RHS is now on the LHS.

permT id (IsReindexing.append_succAbove_natAdd (n := n) (n1 := n1) i)
(evalT (Fin.natAdd (m := n1 + 1) n i) (basisIdxCongr (by simp) x) (prodT t t1)) =
prodT t (evalT i x t1) := by
apply Tensor.induction_on_basis (t := t)

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.

If you search for induction_on_basis in Physlib you will see a better way of doing this using induction' t with ....

@jstoobysmith jstoobysmith added the awaiting-author A reviewer has asked the author a question or requested changes label Jul 3, 2026
NicolaBernini and others added 7 commits July 3, 2026 15:30
Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
Co-authored-by: Claude Opus 4.8 <no-reply+claude-opus-4-8@anthropic.com>
@NicolaBernini

Copy link
Copy Markdown
Collaborator Author

Hi @jstoobysmith eventually it's green

@jstoobysmith

Copy link
Copy Markdown
Member

Hey @NicolaBernini would you mind commenting with

-awaiting-author

when done with a PR :) - I use a filter based on this label to determine what PRs to look at.

@github-actions github-actions Bot removed the awaiting-author A reviewer has asked the author a question or requested changes label Jul 4, 2026

@jstoobysmith jstoobysmith left a comment

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.

Looks good though - approved.

@jstoobysmith jstoobysmith added the ready-to-merge This PR is approved and will be merged shortly label Jul 4, 2026
@jstoobysmith
jstoobysmith merged commit 2e21835 into leanprover-community:master Jul 4, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR is approved and will be merged shortly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants