Skip to content

[Merged by Bors] - feat: Chevalley's theorem #21021

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

Closed
wants to merge 12 commits into from
Closed

Conversation

YaelDillies
Copy link
Collaborator

@YaelDillies YaelDillies commented Jan 24, 2025

This is https://stacks.math.columbia.edu/tag/00FE.

From GrowthInGroups (LeanCamCombi)

Co-authored-by: Andrew Yang


Open in Gitpod

Copy link

github-actions bot commented Jan 24, 2025

PR summary 2df7b1e4c8

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference
Mathlib.RingTheory.Spectrum.Prime.ChevalleyComplexity (new file) 1648

Declarations diff

+ BasicConstructibleSetData
+ chevalley_mvPolynomialC
+ chevalley_mvPolynomial_mvPolynomial
+ chevalley_polynomialC
+ degBound_casesOn_succ
+ degBound_le_degBound
+ degBound_pos
+ degBound_succ
+ degBound_zero
+ instance : DecidableEq (BasicConstructibleSetData R) := Classical.decEq _
+ map_comp'
+ map_id'
+ numBound_casesOn_succ
+ numBound_mono
+ numBound_succ
+ numBound_zero
++ numBound
++- map
++- map_comp
++- map_id
++- toSet
++- toSet_map
+-++ degBound

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.


Increase in tech debt: (relative, absolute) = (1.00, 0.00)
Current number Change Type
1166 1 erw

Current commit 2df7b1e4c8
Reference commit 8deb89281c

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).

Constructible sets in the prime spectrum are made of open sets in the prime spectrum, which are
themselves made from a family of polynomials.

We say an open set *has complexity at most `M`* if it can be written as the zero locus of a family
Copy link
Collaborator

@AntoineChambert-Loir AntoineChambert-Loir Feb 1, 2025

Choose a reason for hiding this comment

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

Would it be worthy to try to track degree and number of open sets separately?
(I'd say yes: in some parts of AG, it's useful to know that some varieties are defined by equations of degree 2.)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Do you mean "number of polynomials" rather than "number of open sets"? We're tracking the former, not the latter.

Given the shape of the bound (a tower of size depending on both the degree and number of polynomials used to describe the original variety), I doubt it is possible to track both variables independently.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe what Antoine is asking, is to have separate bounds on the degree and the number of polynomials, not necessarily independent of each other? In this case, it seems that this is what actually happens in chevalley_mvPolynomial_mvPolynomial, but this doc string does suggest something else, namely that there is only one M bounding both the number of polynomials and degrees. Could you clarify this in this docstring?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, I wrote the docstring informally because the fact that there is a bound is all that matters, really. chevalley_mvPolynomial_mvPolynomial features two bounds solely because we can't (usefully) make those bounds into one due to mutual recursion.

@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot added blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) and removed blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) labels Feb 3, 2025
@mathlib4-dependent-issues-bot
Copy link
Collaborator

This PR/issue depends on:

This is https://stacks.math.columbia.edu/tag/00FE.

From GrowthInGroups (LeanCamCombi)
Co-authored-by: Andrew Yang
Copy link
Collaborator

@chrisflav chrisflav left a comment

Choose a reason for hiding this comment

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

I went through most of the overall strategy and docstrings, but I yet have to have a closer look at the proofs, which I'll hopefully find time to do before the end of the week.

@chrisflav chrisflav added the awaiting-author A reviewer has asked the author a question or requested changes. label Mar 12, 2025
@YaelDillies YaelDillies removed the awaiting-author A reviewer has asked the author a question or requested changes. label Mar 12, 2025
@jcommelin jcommelin added the awaiting-author A reviewer has asked the author a question or requested changes. label Mar 17, 2025
@YaelDillies YaelDillies removed the awaiting-author A reviewer has asked the author a question or requested changes. label Mar 17, 2025
Copy link
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

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

bors d+

@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Mar 17, 2025

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

@YaelDillies
Copy link
Collaborator Author

YaelDillies commented Mar 17, 2025

Thank you everyone for the reviews! This is a great milestone.

bors merge

mathlib-bors bot pushed a commit that referenced this pull request Mar 17, 2025
This is https://stacks.math.columbia.edu/tag/00FE.

From GrowthInGroups (LeanCamCombi)

Co-authored-by: Andrew Yang



Co-authored-by: Andrew Yang <[email protected]>
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Mar 17, 2025

Build failed (retrying...):

@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Mar 17, 2025

Canceled.

@YaelDillies
Copy link
Collaborator Author

bors merge

mathlib-bors bot pushed a commit that referenced this pull request Mar 17, 2025
This is https://stacks.math.columbia.edu/tag/00FE.

From GrowthInGroups (LeanCamCombi)

Co-authored-by: Andrew Yang



Co-authored-by: Andrew Yang <[email protected]>
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Mar 17, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: Chevalley's theorem [Merged by Bors] - feat: Chevalley's theorem Mar 17, 2025
@mathlib-bors mathlib-bors bot closed this Mar 17, 2025
@mathlib-bors mathlib-bors bot deleted the chevalley branch March 17, 2025 20:22
tukamilano pushed a commit that referenced this pull request Mar 20, 2025
This is https://stacks.math.columbia.edu/tag/00FE.

From GrowthInGroups (LeanCamCombi)

Co-authored-by: Andrew Yang



Co-authored-by: Andrew Yang <[email protected]>
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.

7 participants