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] - chore(data/matrix/basic): clean up of new lemmas on matrix numerals #2996
Conversation
These should have been review comments on @pechersky's recent #2987 PR, but I was too slow. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 🎉
bors merge
…2996) Generalise and improve use of `@[simp]` for some newly added lemmas about matrix numerals. Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Pull request successfully merged into master. Build succeeded: |
@semorrison Why is the section pulled outside of diagonal? The |
I guess that was an oversight on my part when defining the lemmas in #2987. |
@pechersky It sounds like you might need to open another PR, so I've sent you an invitation to collaborate on mathlib so you can create branches in this repo; that'll make it easier for us to see the build status of each commit. |
…eanprover-community#2996) Generalise and improve use of `@[simp]` for some newly added lemmas about matrix numerals. Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Generalise and improve use of
@[simp]
for some newly added lemmas about matrix numerals.