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

feat(ring_theory/graded_algebra): mv_polynomial is ℕ-graded #10119

Open
wants to merge 134 commits into
base: master
Choose a base branch
from

Conversation

@github-actions github-actions bot added the blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. label Nov 2, 2021
@jjaassoonn jjaassoonn changed the title feat(ring_theory/graded_ring): mv_polynomial is graded by (λ i : ℕ, (homogeneous_submodule σ R i).to_add_subgroup) feat(ring_theory/graded_ring): mv_polynomial is ℕ-graded Nov 2, 2021
@jjaassoonn jjaassoonn changed the title feat(ring_theory/graded_ring): mv_polynomial is ℕ-graded feat(ring_theory/graded_algebra): mv_polynomial is ℕ-graded Dec 13, 2021
@jjaassoonn
Copy link
Collaborator Author

This likely overlaps a lot with #8913; once the dependencies clear up, we should compare the approaches.

The two induction lemmas in #8913 seems very useful. The map mv_polynomial sigma R \to \Oplus i, homogensou_submodule sigma R in this pr isn't very informative.

@eric-wieser
Copy link
Member

I merged master to make the diff smaller

@leanprover-community-bot-assistant leanprover-community-bot-assistant removed the blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. label Mar 4, 2022
@leanprover-community leanprover-community deleted a comment from github-actions bot Mar 4, 2022
@leanprover-community-bot-assistant leanprover-community-bot-assistant added the merge-conflict Please `git merge origin/master` then a bot will remove this label. label Mar 4, 2022
@leanprover-community-bot-assistant leanprover-community-bot-assistant removed the merge-conflict Please `git merge origin/master` then a bot will remove this label. label Mar 5, 2022
@semorrison semorrison added the awaiting-author A reviewer has asked the author a question or requested changes label Mar 8, 2022
@leanprover-community-bot-assistant leanprover-community-bot-assistant added the merge-conflict Please `git merge origin/master` then a bot will remove this label. label Mar 13, 2022
@leanprover-community-bot-assistant leanprover-community-bot-assistant removed the merge-conflict Please `git merge origin/master` then a bot will remove this label. label Mar 28, 2022
@jjaassoonn jjaassoonn 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 Mar 28, 2022
@jjaassoonn jjaassoonn removed the awaiting-review The author would like community review of the PR label Jul 19, 2022
@semorrison semorrison added awaiting-author A reviewer has asked the author a question or requested changes merge-conflict Please `git merge origin/master` then a bot will remove this label. awaiting-CI The author would like to see what CI has to say before doing more work. labels Mar 29, 2023
@semorrison semorrison added the too-late This PR was ready too late for inclusion in mathlib3 label Jul 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-author A reviewer has asked the author a question or requested changes awaiting-CI The author would like to see what CI has to say before doing more work. merge-conflict Please `git merge origin/master` then a bot will remove this label. too-late This PR was ready too late for inclusion in mathlib3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants