Skip to content

feat(RingTheory): uniqueness of the lift of a simple root and the adic completeness of artinian local ring#37356

Open
BryceT233 wants to merge 7 commits intoleanprover-community:masterfrom
BryceT233:henselian_lemmas
Open

feat(RingTheory): uniqueness of the lift of a simple root and the adic completeness of artinian local ring#37356
BryceT233 wants to merge 7 commits intoleanprover-community:masterfrom
BryceT233:henselian_lemmas

Conversation

@BryceT233
Copy link
Copy Markdown
Contributor

This PR adds the following three lemmas:

  • Polynomial.exists_mul_sq_add_linear_part_eq_eval_add: this is an alternative version of Polynomial.eval_add_of_sq_eq_zero and is used in the lemma IsLocalRing.eq_of_eval_eq_zero_of_not_isUnit_sub.
  • An instance that an artinian local ring is adic complete with respect to its maximal ideal.
  • IsLocalRing.eq_of_eval_eq_zero_of_not_isUnit_sub: this is stacks [06RR] which shows the uniqueness of the lift of a simple root given by the henselian property.

Open in Gitpod

@github-actions github-actions bot added the large-import Automatically added label for PRs with a significant increase in transitive imports label Mar 30, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 30, 2026

PR summary b301d257a1

Import changes exceeding 2%

% File
+34.04% Mathlib.RingTheory.AdicCompletion.Noetherian

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.RingTheory.AdicCompletion.Noetherian 1369 1835 +466 (+34.04%)
Import changes for all files
Files Import difference
Mathlib.RingTheory.AdicCompletion.Noetherian 466

Declarations diff

+ IsLocalRing.eq_of_eval_eq_zero_of_not_isUnit_sub
+ exists_mul_sq_add_linear_part_eq_eval_add
+ instance (priority := 100) {A : Type*} [CommRing A] [IsArtinianRing A] [IsLocalRing A] :

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

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

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


No changes to technical debt.

You can run this locally as

./scripts/reporting/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).

@github-actions github-actions bot added the t-ring-theory Ring theory label Mar 30, 2026
@artie2000
Copy link
Copy Markdown
Collaborator

artie2000 commented Apr 1, 2026

could you indicate what you are adding in the PR title?

@BryceT233 BryceT233 changed the title feat(RingTheory): add three lemmas feat(RingTheory): uniqueness of the lift of a simple root and the adic completeness of artinian local ring Apr 1, 2026
@BryceT233
Copy link
Copy Markdown
Contributor Author

The lemmas felt a bit disjointed, and I couldn't think of a good way to summarize them without making the title overly long, so I took the easy way out. I've updated it to better reflect the contents

@artie2000
Copy link
Copy Markdown
Collaborator

if they're independent, then you can make 2 smaller PRs (I know it's annoying)
if they depend on each other, just talk about the hardest thing you proved
sorry idk anything about the maths I was just browsing the queue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

large-import Automatically added label for PRs with a significant increase in transitive imports t-ring-theory Ring theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants