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] - refactor(RingTheory.MvPolynomial.Homogeneous): refactor in terms of weightedHomogeneous #7609

Closed
wants to merge 19 commits into from

Conversation

mariainesdff
Copy link
Collaborator

We rewrite the definition of homogeneous polynomial as a special case of weighted homogeneous polynomial.

Question: should all uses of ∑ i in d.support, d i be replaced by degree d?

Co-authored-by: Antoine Chambert-Loir antoine.chambert-loir@u-paris.fr


Open in Gitpod

@mariainesdff mariainesdff added awaiting-review The author would like community review of the PR t-algebra Algebra (groups, rings, fields etc) labels Oct 10, 2023
mariainesdff and others added 3 commits October 10, 2023 17:36
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

@Vierkantor Vierkantor left a comment

Choose a reason for hiding this comment

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

That is a nice refactor! Could you indeed replace ∑ i in d.support, d i with degree d throughout?

Mathlib/RingTheory/MvPolynomial/Homogeneous.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/MvPolynomial/Homogeneous.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/MvPolynomial/WeightedHomogeneous.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/MvPolynomial/WeightedHomogeneous.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/MvPolynomial/WeightedHomogeneous.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/MvPolynomial/Homogeneous.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/MvPolynomial/Homogeneous.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/MvPolynomial/Homogeneous.lean Outdated Show resolved Hide resolved
@Vierkantor Vierkantor added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Oct 23, 2023
@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Oct 31, 2023
@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Feb 22, 2024
@mariainesdff mariainesdff 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 Feb 22, 2024
@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Feb 23, 2024
@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Feb 23, 2024
Copy link
Member

@riccardobrasca riccardobrasca left a comment

Choose a reason for hiding this comment

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

Thanks!

bors d+

Mathlib/RingTheory/MvPolynomial/Homogeneous.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/MvPolynomial/Homogeneous.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/MvPolynomial/WeightedHomogeneous.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/MvPolynomial/WeightedHomogeneous.lean Outdated Show resolved Hide resolved
@mathlib-bors
Copy link

mathlib-bors bot commented Mar 8, 2024

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

@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added delegated and removed awaiting-review The author would like community review of the PR labels Mar 8, 2024
@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Mar 22, 2024
@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Apr 12, 2024
@mariainesdff
Copy link
Collaborator Author

bors r+

mathlib-bors bot pushed a commit that referenced this pull request Apr 15, 2024
…eightedHomogeneous (#7609)

We rewrite the definition of homogeneous polynomial as a special case of weighted homogeneous polynomial.

Question: should all uses of `∑ i in d.support, d i` be replaced by `degree d`?

Co-authored-by: Antoine Chambert-Loir <antoine.chambert-loir@u-paris.fr>



Co-authored-by: María Inés de Frutos-Fernández <88536493+mariainesdff@users.noreply.github.com>
@mathlib-bors
Copy link

mathlib-bors bot commented Apr 15, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title refactor(RingTheory.MvPolynomial.Homogeneous): refactor in terms of weightedHomogeneous [Merged by Bors] - refactor(RingTheory.MvPolynomial.Homogeneous): refactor in terms of weightedHomogeneous Apr 15, 2024
@mathlib-bors mathlib-bors bot closed this Apr 15, 2024
@mathlib-bors mathlib-bors bot deleted the mariainesdff/homogeneous branch April 15, 2024 15:52
atarnoam pushed a commit that referenced this pull request Apr 16, 2024
…eightedHomogeneous (#7609)

We rewrite the definition of homogeneous polynomial as a special case of weighted homogeneous polynomial.

Question: should all uses of `∑ i in d.support, d i` be replaced by `degree d`?

Co-authored-by: Antoine Chambert-Loir <antoine.chambert-loir@u-paris.fr>



Co-authored-by: María Inés de Frutos-Fernández <88536493+mariainesdff@users.noreply.github.com>
semorrison pushed a commit that referenced this pull request Apr 17, 2024
…eightedHomogeneous (#7609)

We rewrite the definition of homogeneous polynomial as a special case of weighted homogeneous polynomial.

Question: should all uses of `∑ i in d.support, d i` be replaced by `degree d`?

Co-authored-by: Antoine Chambert-Loir <antoine.chambert-loir@u-paris.fr>

Co-authored-by: María Inés de Frutos-Fernández <88536493+mariainesdff@users.noreply.github.com>
semorrison pushed a commit that referenced this pull request Apr 17, 2024
…eightedHomogeneous (#7609)

We rewrite the definition of homogeneous polynomial as a special case of weighted homogeneous polynomial.

Question: should all uses of `∑ i in d.support, d i` be replaced by `degree d`?

Co-authored-by: Antoine Chambert-Loir <antoine.chambert-loir@u-paris.fr>

Co-authored-by: María Inés de Frutos-Fernández <88536493+mariainesdff@users.noreply.github.com>
uniwuni pushed a commit that referenced this pull request Apr 19, 2024
…eightedHomogeneous (#7609)

We rewrite the definition of homogeneous polynomial as a special case of weighted homogeneous polynomial.

Question: should all uses of `∑ i in d.support, d i` be replaced by `degree d`?

Co-authored-by: Antoine Chambert-Loir <antoine.chambert-loir@u-paris.fr>



Co-authored-by: María Inés de Frutos-Fernández <88536493+mariainesdff@users.noreply.github.com>
callesonne pushed a commit that referenced this pull request Apr 22, 2024
…eightedHomogeneous (#7609)

We rewrite the definition of homogeneous polynomial as a special case of weighted homogeneous polynomial.

Question: should all uses of `∑ i in d.support, d i` be replaced by `degree d`?

Co-authored-by: Antoine Chambert-Loir <antoine.chambert-loir@u-paris.fr>



Co-authored-by: María Inés de Frutos-Fernández <88536493+mariainesdff@users.noreply.github.com>
Jun2M pushed a commit that referenced this pull request Apr 24, 2024
…eightedHomogeneous (#7609)

We rewrite the definition of homogeneous polynomial as a special case of weighted homogeneous polynomial.

Question: should all uses of `∑ i in d.support, d i` be replaced by `degree d`?

Co-authored-by: Antoine Chambert-Loir <antoine.chambert-loir@u-paris.fr>



Co-authored-by: María Inés de Frutos-Fernández <88536493+mariainesdff@users.noreply.github.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

4 participants