-
Notifications
You must be signed in to change notification settings - Fork 346
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(NumberTheory/NumberField/FinitePlaces): the finite places of a number field #19667
Conversation
…umber field --- <!-- The text above the `---` will become the commit message when your PR is merged. Please leave a blank newline before the `---`, otherwise GitHub will format the text above it as a title. For details on the "pull request lifecycle" in mathlib, please see: https://leanprover-community.github.io/contribute/index.html In particular, note that most reviewers will only notice your PR if it passes the continuous integration checks. Please ask for help on https://leanprover.zulipchat.com if needed. To indicate co-authors, include lines at the bottom of the commit message (that is, before the `---`) using the following format: Co-authored-by: Author Name <[email protected]> If you are moving or deleting declarations, please include these lines at the bottom of the commit message (that is, before the `---`) using the following format: Moves: - Vector.* -> Mathlib.Vector.* - ... Deletions: - Nat.bit1_add_bit1 - ... Any other comments you want to keep out of the PR commit should go below the `---`, and placed outside this HTML comment, or else they will be invisible to reviewers. If this PR depends on other PRs, please list them below this comment, using the following format: - [ ] depends on: #abc [optional extra text] - [ ] depends on: #xyz [optional extra text] --> [![Open in Gitpod](https://gitpod.io/button/open-in-gitpod.svg)](https://gitpod.io/from-referrer/)
PR summary 4e978f3c00Import changes for modified filesNo significant changes to the import graph Import changes for all files
|
Thank you @MichaelStollBayreuth for the review and the suggestions! |
Co-authored-by: Michael Stoll <[email protected]>
Co-authored-by: Michael Stoll <[email protected]>
Co-authored-by: Michael Stoll <[email protected]>
Co-authored-by: Michael Stoll <[email protected]>
Co-authored-by: Michael Stoll <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me now!
maintainer merge
🚀 Pull request has been placed on the maintainer queue by MichaelStollBayreuth. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot! Just to flag that in the FLT project we are also thinking about this sort of thing; in particular this file https://github.com/ImperialCollegeLondon/FLT/blob/main/FLT/HaarMeasure/DistribHaarChar.lean defines the norm on the units of any locally compact topological field as being the factor which Haar measure is scaled by under multiplication. What we need in FLT is the assertion that if B is a f.d. central simple algebra over a number field K, then multiplication by a unit in B doesn't change Haar measure on B tensor_K A_K. This is a generalization of what your goal is here so we should take care not to duplicate work. See for example ImperialCollegeLondon/FLT#223 .
bors merge
‖embedding (maximalIdeal w) x‖ = w x := by | ||
conv_rhs => rw [← mk_maximalIdeal w, apply] | ||
|
||
theorem pos_iff {w : FinitePlace K} {x : K} : 0 < w x ↔ x ≠ 0 := AbsoluteValue.pos_iff w.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remark: this could have just been w.1.pos_iff
…umber field (#19667) This file defines finite places of a number field `K` as absolute values coming from an embedding into a completion of `K` associated to a non-zero prime ideal of `𝓞 K` with some basic APIs. This is part of a project which aims to define heights. The next step is the product formula.
Pull request successfully merged into master. Build succeeded: |
Thanks! |
This file defines finite places of a number field
K
as absolute values coming from an embeddinginto a completion of
K
associated to a non-zero prime ideal of𝓞 K
with some basic APIs.This is part of a project which aims to define heights. The next step is the product formula.