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(Data/Matrix): Equalities relating mulVec and smul #12538

Closed
wants to merge 9 commits into from

Conversation

Rida-Hamadani
Copy link
Collaborator

@Rida-Hamadani Rida-Hamadani commented Apr 30, 2024

These lemmas should make working with vectors easier in some cases. Check this related Zulip topic.


Open in Gitpod

Copy link
Member

@eric-wieser eric-wieser left a comment

Choose a reason for hiding this comment

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

Can you add the vecMul_diagonal variants too?

@eric-wieser eric-wieser added awaiting-author A reviewer has asked the author a question or requested changes awaiting-CI t-algebra Algebra (groups, rings, fields etc) labels Apr 30, 2024
@trivial1711
Copy link
Collaborator

Hi, can you add a version for Matrix.scalar?

Co-authored-by: Mitchell Lee <130885647+trivial1711@users.noreply.github.com>
@Rida-Hamadani
Copy link
Collaborator Author

Hi, can you add a version for Matrix.scalar?

Sure, working on it

@eric-wieser
Copy link
Member

Hi, can you add a version for Matrix.scalar?

I don't think we want this, since scalar is not the simp-normal form.

@trivial1711
Copy link
Collaborator

Hi, can you add a version for Matrix.scalar?

I don't think we want this, since scalar is not the simp-normal form.

Yes, good point.

@Rida-Hamadani Rida-Hamadani added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Apr 30, 2024
@eric-wieser
Copy link
Member

bors d+

This looks great, thanks! I pushed some golfs and tweaked a name.

Please merge once this passes CI :)

@mathlib-bors
Copy link

mathlib-bors bot commented May 1, 2024

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

@github-actions github-actions bot added delegated and removed awaiting-review The author would like community review of the PR labels May 1, 2024
@Rida-Hamadani
Copy link
Collaborator Author

bors r+

mathlib-bors bot pushed a commit that referenced this pull request May 2, 2024
These lemmas should make working with vectors easier in some cases. Check [this](https://leanprover.zulipchat.com/#narrow/stream/113489-new-members/topic/Factor.20out.20a.20vector) related Zulip topic.



Co-authored-by: Rida Hamadani <106540880+Rida-Hamadani@users.noreply.github.com>
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
@mathlib-bors
Copy link

mathlib-bors bot commented May 2, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Data/Matrix): Equalities relating mulVec and smul [Merged by Bors] - feat(Data/Matrix): Equalities relating mulVec and smul May 2, 2024
@mathlib-bors mathlib-bors bot closed this May 2, 2024
@mathlib-bors mathlib-bors bot deleted the rida/matrix branch May 2, 2024 01:12
apnelson1 pushed a commit that referenced this pull request May 12, 2024
These lemmas should make working with vectors easier in some cases. Check [this](https://leanprover.zulipchat.com/#narrow/stream/113489-new-members/topic/Factor.20out.20a.20vector) related Zulip topic.



Co-authored-by: Rida Hamadani <106540880+Rida-Hamadani@users.noreply.github.com>
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
callesonne pushed a commit that referenced this pull request May 16, 2024
These lemmas should make working with vectors easier in some cases. Check [this](https://leanprover.zulipchat.com/#narrow/stream/113489-new-members/topic/Factor.20out.20a.20vector) related Zulip topic.



Co-authored-by: Rida Hamadani <106540880+Rida-Hamadani@users.noreply.github.com>
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated t-algebra Algebra (groups, rings, fields etc)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants