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(ring_theory/graded_algebra): projection map of graded algebra #10116
Conversation
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
…munity/mathlib into graded_ring_dep_2
src/algebra/direct_sum/internal.lean
Outdated
@@ -77,6 +77,24 @@ direct_sum.to_semiring (λ i, (A i).subtype) rfl (λ _ _ _ _, rfl) | |||
direct_sum.submonoid_coe_ring_hom A (direct_sum.of (λ i, A i) i x) = x := | |||
direct_sum.to_semiring_of _ _ _ _ _ | |||
|
|||
section big_operators | |||
|
|||
open_locale big_operators |
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.
I think it's fine to just do this at the top of the file, and then get rid of the section
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, this looks great now! Just a few minor stylistic changes left
bors d+
✌️ jjaassoonn can now approve this pull request. To approve and merge a pull request, simply reply with |
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
Thank you, Eric! bors r+ |
Canceled. |
I pushed one missed style change (edit: and two more I didn't comment on). Let's wait for CI before merging. |
I also combined two |
Also should fix a lint error
bors merge Looks like the linter is finally happy |
…10116) This pr defines and proves some property about `graded_algebra.proj : A →ₗ[R] A` Co-authored-by: jjaassoonn <jujian.zhang1998@out.com> Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
Pull request successfully merged into master. Build succeeded: |
…10116) This pr defines and proves some property about `graded_algebra.proj : A →ₗ[R] A` Co-authored-by: jjaassoonn <jujian.zhang1998@out.com> Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
This pr defines and proves some property about
graded_algebra.proj : A →ₗ[R] A
graded_algebra
#10115