Skip to content

feat: Prove equivalence of isDedekindDomain and isDedekindDomainDvr#14242

Open
js2357 wants to merge 18 commits intomasterfrom
js2357/DedekindDomainDvr
Open

feat: Prove equivalence of isDedekindDomain and isDedekindDomainDvr#14242
js2357 wants to merge 18 commits intomasterfrom
js2357/DedekindDomainDvr

Conversation

@js2357
Copy link
Copy Markdown
Collaborator

@js2357 js2357 commented Jun 28, 2024

Prove that isDedekindDomainDvr is equivalent to both isDedekindDomain and isDedekindDomainInv.

Specifically, prove isDedekindDomainDvr A → isDedekindDomainInv A, because isDedekindDomain A → isDedekindDomainDvr A and IsDedekindDomain A ↔ IsDedekindDomainInv A are already in Mathlib.


Open in Gitpod

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 28, 2024

PR summary 83a78b5dcb

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.RingTheory.DedekindDomain.Dvr 1254 1270 +16 (+1.28%)
Import changes for all files
Files Import difference
16 files Mathlib.RingTheory.FractionalIdeal.Norm Mathlib.NumberTheory.Cyclotomic.Rat Mathlib.NumberTheory.Cyclotomic.Discriminant Mathlib.NumberTheory.NumberField.FractionalIdeal Mathlib.NumberTheory.NumberField.Discriminant Mathlib.NumberTheory.NumberField.EquivReindex Mathlib.NumberTheory.Cyclotomic.Three Mathlib.NumberTheory.FLT.Three Mathlib.NumberTheory.NumberField.CanonicalEmbedding.ConvexBody Mathlib.NumberTheory.Cyclotomic.PID Mathlib.NumberTheory.NumberField.Units.DirichletTheorem Mathlib.NumberTheory.NumberField.Units.Regulator Mathlib.RingTheory.Ideal.Norm Mathlib.NumberTheory.NumberField.House Mathlib.NumberTheory.NumberField.ClassNumber Mathlib.NumberTheory.NumberField.CanonicalEmbedding.Basic
2
Mathlib.RingTheory.DedekindDomain.PID 4
Mathlib.RingTheory.DedekindDomain.Dvr 16
Mathlib.RingTheory.FractionalIdeal.LocalizedAtPrime 1251

Declarations diff

+ Field.isLocalization_self
+ IsDedekindDomainDvr.isDedekindDomain
+ IsDedekindDomainDvr.isDedekindDomainInv
+ coe_localizedAtPrime
+ exists_smul_mem_of_mem_localizedAtPrime
+ instance : IsFractionRing Aₚ K
+ instance : IsLocalizedModule P.primeCompl (I.localizedAtPrime_inclusion P)
+ instance : IsScalarTower A Aₚ K
+ isDedekindDomainInv_iff_isDedekindDomainDvr
+ isDedekindDomain_iff_isDedekindDomainDvr
+ localizedAtPrime_add
+ localizedAtPrime_div
+ localizedAtPrime_div_le
+ localizedAtPrime_inclusion
+ localizedAtPrime_inv
+ localizedAtPrime_mul
+ localizedAtPrime_ne_zero
+ localizedAtPrime_one
+ localizedAtPrime_zero
+ mem_localizedAtPrime_iff
+ mem_localizedAtPrime_iff_eq
+ not_le_of_localizedAtPrime_eq_one
+ self_subset_localizedAtPrime

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.

@ghost ghost added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Jun 28, 2024
@ghost
Copy link
Copy Markdown

ghost commented Jun 28, 2024

@js2357 js2357 added awaiting-review t-algebra Algebra (groups, rings, fields, etc) labels Jun 28, 2024
mathlib-bors bot pushed a commit that referenced this pull request Jun 30, 2024
Prove five lemmas about fractional ideals (zero_mem, fg_of_isNoetherianRing, den_mem_inv, num_le_mul_inv, bot_lt_mul_inv). 

This is part 1/4 of a proof of `isDedekindDomain_iff_isDedekindDomainDvr`. 
Part 2: #14216
Part 3: #14237
Part 4: #14242
@ghost ghost added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jun 30, 2024
dagurtomas pushed a commit that referenced this pull request Jul 2, 2024
Prove five lemmas about fractional ideals (zero_mem, fg_of_isNoetherianRing, den_mem_inv, num_le_mul_inv, bot_lt_mul_inv). 

This is part 1/4 of a proof of `isDedekindDomain_iff_isDedekindDomainDvr`. 
Part 2: #14216
Part 3: #14237
Part 4: #14242
@ghost ghost removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jul 28, 2024
mathlib-bors bot pushed a commit that referenced this pull request Aug 21, 2024
…of fractional ideals (#14216)

Define the extension of a fractional ideal along a ring homomorphism, and prove some basic facts about extensions.

This PR is part 2/4 of a proof of `isDedekindDomain_iff_isDedekindDomainDvr`.
Part 1: #14099
Part 3: #14237
Part 4: #14242

- [x] depends on: #14099
@ghost ghost added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Aug 21, 2024
@ghost ghost removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Aug 21, 2024
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 9, 2024
…of fractional ideals (#14216)

Define the extension of a fractional ideal along a ring homomorphism, and prove some basic facts about extensions.

This PR is part 2/4 of a proof of `isDedekindDomain_iff_isDedekindDomainDvr`.
Part 1: #14099
Part 3: #14237
Part 4: #14242

- [x] depends on: #14099
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 9, 2024
…of fractional ideals (#14216)

Define the extension of a fractional ideal along a ring homomorphism, and prove some basic facts about extensions.

This PR is part 2/4 of a proof of `isDedekindDomain_iff_isDedekindDomainDvr`.
Part 1: #14099
Part 3: #14237
Part 4: #14242

- [x] depends on: #14099
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 12, 2024
…of fractional ideals (#14216)

Define the extension of a fractional ideal along a ring homomorphism, and prove some basic facts about extensions.

This PR is part 2/4 of a proof of `isDedekindDomain_iff_isDedekindDomainDvr`.
Part 1: #14099
Part 3: #14237
Part 4: #14242

- [x] depends on: #14099
@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 Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) new-contributor This PR was made by a contributor with at most 5 merged PRs. Welcome to the community! t-algebra Algebra (groups, rings, fields, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants