Skip to content

[Merged by Bors] - feat(Data/Finsupp/MonomialOrder/DegLex): homogeneous lexicographic order#19455

Closed
AntoineChambert-Loir wants to merge 16 commits intomasterfrom
ACL/Groebner-deglex
Closed

[Merged by Bors] - feat(Data/Finsupp/MonomialOrder/DegLex): homogeneous lexicographic order#19455
AntoineChambert-Loir wants to merge 16 commits intomasterfrom
ACL/Groebner-deglex

Conversation

@AntoineChambert-Loir
Copy link
Copy Markdown
Collaborator

@AntoineChambert-Loir AntoineChambert-Loir commented Nov 25, 2024

Definition of the homogeneous lexicographic order

This is part of an ongoing work on basic of Gröbner theory.
A subsequent PR will add the homogeneous reverse lexicographic order.


Open in Gitpod

@github-actions github-actions bot added the t-data Data (lists, quotients, numbers, etc) label Nov 25, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 25, 2024

PR summary 233c68d4a6

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference
Mathlib.Data.Finsupp.MonomialOrder.DegLex (new file) 882

Declarations diff

+ DegLex.exists_iff
+ DegLex.forall_iff
+ DegLex.rec
+ Lex.single_antitone
+ Lex.single_le_iff
+ Lex.single_lt_iff
+ Lex.single_strictAnti
+ degLex
+ degLex_def
+ degLex_le_iff
+ degLex_lt_iff
+ degLex_single_le_iff
+ degLex_single_lt_iff
+ degree_add
+ degree_single
+ instance :
+ instance : LinearOrder (DegLex (α →₀ ℕ))
+ instance : OrderedCancelAddCommMonoid (DegLex (α →₀ ℕ))
+ instance [AddCommMonoid α] :
+ instance [LT α] : LT (DegLex (α →₀ ℕ))
+ isStrictOrder
+ le_iff
+ lt_def
+ lt_iff
+ monotone_degree
+ ofDegLex
+ ofDegLex_add
+ ofDegLex_inj
+ ofDegLex_injective
+ ofDegLex_symm_eq
+ ofDegLex_toDegLex
+ orderBot
+ single_antitone
+ single_le_iff
+ single_lt_iff
+ single_strictAnti
+ toDegLex
+ toDegLex_add
+ toDegLex_inj
+ toDegLex_injective
+ toDegLex_ofDegLex
+ toDegLex_symm_eq
+ wellFounded
+ wellFoundedLT
++ DegLex

You can run this locally as follows
## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>

The doc-module for script/declarations_diff.sh contains some details about this script.


No changes to technical debt.

You can run this locally as

./scripts/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Nov 25, 2024
@AntoineChambert-Loir AntoineChambert-Loir added t-algebraic-geometry Algebraic geometry t-order Order theory labels Nov 25, 2024
@AntoineChambert-Loir AntoineChambert-Loir changed the title feat(Mathlib/Data/Finsupp/MonomialOrder/DegLex): homogeneous lexicographic order feat(Data/Finsupp/MonomialOrder/DegLex): homogeneous lexicographic order Nov 25, 2024
@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Nov 25, 2024
@mathlib4-dependent-issues-bot
Copy link
Copy Markdown
Collaborator

This PR/issue depends on:

@leanprover-community-bot-assistant leanprover-community-bot-assistant added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Nov 25, 2024
@leanprover-community-bot-assistant leanprover-community-bot-assistant removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Nov 25, 2024
@joelriou
Copy link
Copy Markdown
Contributor

I have only very minor suggestions, otherwise it looks good to me.

@joelriou joelriou added the awaiting-author A reviewer has asked the author a question or requested changes. label Nov 26, 2024
@AntoineChambert-Loir AntoineChambert-Loir removed the awaiting-author A reviewer has asked the author a question or requested changes. label Nov 27, 2024
@joelriou joelriou added the awaiting-author A reviewer has asked the author a question or requested changes. label Nov 27, 2024
@AntoineChambert-Loir AntoineChambert-Loir removed the awaiting-author A reviewer has asked the author a question or requested changes. label Nov 29, 2024
@AntoineChambert-Loir
Copy link
Copy Markdown
Collaborator Author

Yet another question: this file requires RingTheory/MvPolynomial/MonomialOrder. So probably it better fits that hierarchy than the Data/Finsupp one. What do you think?

@joelriou
Copy link
Copy Markdown
Contributor

joelriou commented Dec 3, 2024

Yet another question: this file requires RingTheory/MvPolynomial/MonomialOrder. So probably it better fits that hierarchy than the Data/Finsupp one. What do you think?

It seems we do not have RingTheory/MvPolynomial/MonomialOrder?! Anyway, I think it is fine in Data.Finsupp.

@joelriou joelriou added the awaiting-author A reviewer has asked the author a question or requested changes. label Dec 3, 2024
AntoineChambert-Loir and others added 3 commits December 4, 2024 11:23
Co-authored-by: Joël Riou <37772949+joelriou@users.noreply.github.com>
Co-authored-by: Joël Riou <37772949+joelriou@users.noreply.github.com>
@AntoineChambert-Loir AntoineChambert-Loir removed the awaiting-author A reviewer has asked the author a question or requested changes. label Dec 4, 2024
@joelriou
Copy link
Copy Markdown
Contributor

joelriou commented Dec 4, 2024

Thanks!

bors merge

@ghost ghost added the ready-to-merge This PR has been sent to bors. label Dec 4, 2024
mathlib-bors bot pushed a commit that referenced this pull request Dec 4, 2024
…der (#19455)

Definition of the homogeneous lexicographic order

This is part of an ongoing work on basic of Gröbner theory.
A subsequent PR will add the homogeneous reverse lexicographic order.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Dec 4, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Data/Finsupp/MonomialOrder/DegLex): homogeneous lexicographic order [Merged by Bors] - feat(Data/Finsupp/MonomialOrder/DegLex): homogeneous lexicographic order Dec 4, 2024
@mathlib-bors mathlib-bors bot closed this Dec 4, 2024
@mathlib-bors mathlib-bors bot deleted the ACL/Groebner-deglex branch December 4, 2024 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors. t-algebraic-geometry Algebraic geometry t-data Data (lists, quotients, numbers, etc) t-order Order theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants