Skip to content
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

[Merged by Bors] - feat(ring_theory/dedekind_domain): localizing a Dedekind domain at a prime gives DVR #18264

Closed
wants to merge 4 commits into from

Conversation

Vierkantor
Copy link
Collaborator

This PR shows one direction of the implication is_dedekind_domain → is_dedekind_domain_dvr. The first step is to show that any localization of a Dedekind domain is again Dedekind, then use the very useful discrete_valuation_ring.tfae to show that this implies the localization at a prime is a DVR.

Beyond general usefulness, I need this for ideal.span_norm.


Open in Gitpod

…prime gives DVR

This PR shows one direction of the implication `is_dedekind_domain → is_dedekind_domain_dvr`. The first step is to show that any localization of a Dedekind domain is again Dedekind, then use the very useful `discrete_valuation_ring.tfae` to show that this implies the localization at a prime is a DVR.

Beyond general usefulness, I need this for `ideal.span_norm`.
@Vierkantor Vierkantor added awaiting-review The author would like community review of the PR awaiting-CI The author would like to see what CI has to say before doing more work. t-algebra Algebra (groups, rings, fields etc) labels Jan 23, 2023
@github-actions github-actions bot removed the awaiting-CI The author would like to see what CI has to say before doing more work. label Jan 23, 2023
@riccardobrasca riccardobrasca self-assigned this Jan 24, 2023
src/ring_theory/dedekind_domain/dvr.lean Outdated Show resolved Hide resolved
src/ring_theory/dedekind_domain/dvr.lean Show resolved Hide resolved
src/ring_theory/dedekind_domain/dvr.lean Show resolved Hide resolved
src/ring_theory/dedekind_domain/dvr.lean Outdated Show resolved Hide resolved
src/ring_theory/dedekind_domain/dvr.lean Outdated Show resolved Hide resolved
@riccardobrasca
Copy link
Member

Thanks!

bors merge

@github-actions github-actions bot added ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) and removed awaiting-review The author would like community review of the PR labels Jan 25, 2023
bors bot pushed a commit that referenced this pull request Jan 25, 2023
…prime gives DVR (#18264)

This PR shows one direction of the implication `is_dedekind_domain → is_dedekind_domain_dvr`. The first step is to show that any localization of a Dedekind domain is again Dedekind, then use the very useful `discrete_valuation_ring.tfae` to show that this implies the localization at a prime is a DVR.

Beyond general usefulness, I need this for `ideal.span_norm`.



Co-authored-by: Vierkantor <vierkantor@vierkantor.com>
Co-authored-by: Anne Baanen <Vierkantor@users.noreply.github.com>
@bors
Copy link

bors bot commented Jan 25, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(ring_theory/dedekind_domain): localizing a Dedekind domain at a prime gives DVR [Merged by Bors] - feat(ring_theory/dedekind_domain): localizing a Dedekind domain at a prime gives DVR Jan 25, 2023
@bors bors bot closed this Jan 25, 2023
@bors bors bot deleted the dedekind_domain-implies-dvr branch January 25, 2023 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) t-algebra Algebra (groups, rings, fields etc)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants