Skip to content

feat(RingTheory/Ideal): restriction maps on decomposition and inertia groups in a Galois tower#36544

Open
xroblot wants to merge 2 commits intoleanprover-community:masterfrom
xroblot:galideals
Open

feat(RingTheory/Ideal): restriction maps on decomposition and inertia groups in a Galois tower#36544
xroblot wants to merge 2 commits intoleanprover-community:masterfrom
xroblot:galideals

Conversation

@xroblot
Copy link
Copy Markdown
Collaborator

@xroblot xroblot commented Mar 12, 2026

Adds a new file Mathlib/RingTheory/Ideal/Galois.lean with results on the action of Gal(L/K) on ideals of a ring B ⊆ L for a Galois subextension F/K of L/K. The main results are the surjectivity and kernel descriptions of the natural maps from the decomposition (resp. inertia) group of P in Gal(L/K) to the decomposition (resp. inertia) group of p in Gal(F/K), induced by restriction.


Open in Gitpod

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 12, 2026

PR summary 55217bd0cd

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference
Mathlib.RingTheory.Ideal.Galois (new file) 2195

Declarations diff

+ AlgEquiv.algebraMap_restrictNormalHom_smul
+ AlgEquiv.algebraMap_restrictNormalHom_smul'
+ comap_smul_eq_restrictNormalHom_smul_comap
+ inertiaMapOfLiesOver
+ inertiaMapOfLiesOver_ker
+ inertiaMapOfLiesOver_surjective
+ restrictScalars_smul
+ smul_liesOver_of_restrictNormalHom_mem_stabilizer
+ stabilizerMapOfLiesOver
+ stabilizerMapOfLiesOver_ker
+ stabilizerMapOfLiesOver_surjective

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

@riccardobrasca
Copy link
Copy Markdown
Member

Can you please fix the conflict?

`AlgEquiv.restrictNormalHom F : Gal(L/K) →* Gal(F/K)`.
-/
@[simps! apply_coe]
noncomputable def stabilizerMapOfLiesOver :
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I may be wrong, but we now have Gal(A/B): do we really need to introduce all these fields? The notation is becoming a nightmare.

I understand we want more flexibility, but if we completely remove fields from the statement then in proofs we can just use the fraction field (where hopefully instances are easy to provide).

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

As you know, the main application I have in mind is for number fields, where K is not defeq to FractionRing (𝓞 K), so removing the fields from the statements would make things very painful in practice. I agree though that the number of hypotheses is becoming unwieldy and I don't have a good solution for this.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I mean, remove any mention to the field from statements, using the Galois group Gal(A/B).

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Oh, I see. But in this case, it would be very helpful to have a ring version of AlgEquiv.restrictNormalHom. I thought we had something like that but I cannot find it anymore.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah, would be good to add an analogue of this for rings (or, better yet, generalize to rings?)

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

Labels

t-ring-theory Ring theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants