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/valuation/valuation_subring): define maximal ideal of valuation subring and provide basic API #14656

Closed
wants to merge 17 commits into from

Conversation

mckoen
Copy link
Collaborator

@mckoen mckoen commented Jun 9, 2022

This PR defines the unique maximal ideal of a valuation subring as a subsemigroup of the field. We prove a few equivalent conditions for two valuations to be equivalent, and we use this to show two valuation subrings are equivalent iff they have the same maximal ideal.


Open in Gitpod

@mckoen mckoen 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. labels Jun 9, 2022
@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 Jun 9, 2022
@Vierkantor Vierkantor added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Jul 12, 2022
@mckoen mckoen added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Jul 14, 2022
Copy link
Collaborator

@Vierkantor Vierkantor left a comment

Choose a reason for hiding this comment

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

I'm happy with this but wrote part of it, could someone else do another check?

mckoen and others added 3 commits July 15, 2022 13:43
Co-authored-by: Anne Baanen <Vierkantor@users.noreply.github.com>
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
Co-authored-by: Johan Commelin <johan@commelin.net>
Copy link
Collaborator

@adamtopaz adamtopaz left a comment

Choose a reason for hiding this comment

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

LGTM!

src/ring_theory/valuation/basic.lean Outdated Show resolved Hide resolved
@robertylewis robertylewis added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Jul 19, 2022
@mckoen mckoen 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. and removed awaiting-author A reviewer has asked the author a question or requested changes labels Jul 20, 2022
@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 Jul 21, 2022
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.

Thanks 🎉

bors merge

@leanprover-community-bot-assistant leanprover-community-bot-assistant 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 Jul 26, 2022
bors bot pushed a commit that referenced this pull request Jul 26, 2022
…of valuation subring and provide basic API (#14656)


This PR defines the unique maximal ideal of a valuation subring as a subsemigroup of the field. We prove a few equivalent conditions for two valuations to be equivalent, and we use this to show two valuation subrings are equivalent iff they have the same maximal ideal. 



Co-authored-by: Vierkantor <vierkantor@vierkantor.com>
Co-authored-by: jackmckoen <104791831+jackmckoen@users.noreply.github.com>
@bors
Copy link

bors bot commented Jul 26, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(ring_theory/valuation/valuation_subring): define maximal ideal of valuation subring and provide basic API [Merged by Bors] - feat(ring_theory/valuation/valuation_subring): define maximal ideal of valuation subring and provide basic API Jul 26, 2022
@bors bors bot closed this Jul 26, 2022
@bors bors bot deleted the valuation_maximal branch July 26, 2022 20:33
bottine pushed a commit to bottine/mathlib that referenced this pull request Jul 30, 2022
…of valuation subring and provide basic API (leanprover-community#14656)


This PR defines the unique maximal ideal of a valuation subring as a subsemigroup of the field. We prove a few equivalent conditions for two valuations to be equivalent, and we use this to show two valuation subrings are equivalent iff they have the same maximal ideal. 



Co-authored-by: Vierkantor <vierkantor@vierkantor.com>
Co-authored-by: jackmckoen <104791831+jackmckoen@users.noreply.github.com>
robertylewis pushed a commit that referenced this pull request Aug 2, 2022
…of valuation subring and provide basic API (#14656)


This PR defines the unique maximal ideal of a valuation subring as a subsemigroup of the field. We prove a few equivalent conditions for two valuations to be equivalent, and we use this to show two valuation subrings are equivalent iff they have the same maximal ideal. 



Co-authored-by: Vierkantor <vierkantor@vierkantor.com>
Co-authored-by: jackmckoen <104791831+jackmckoen@users.noreply.github.com>
khwilson pushed a commit that referenced this pull request Aug 2, 2022
…of valuation subring and provide basic API (#14656)


This PR defines the unique maximal ideal of a valuation subring as a subsemigroup of the field. We prove a few equivalent conditions for two valuations to be equivalent, and we use this to show two valuation subrings are equivalent iff they have the same maximal ideal. 



Co-authored-by: Vierkantor <vierkantor@vierkantor.com>
Co-authored-by: jackmckoen <104791831+jackmckoen@users.noreply.github.com>
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+`.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants