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: Generalize absNorm to fractional ideals #9613

Closed
wants to merge 8 commits into from

Conversation

xroblot
Copy link
Collaborator

@xroblot xroblot commented Jan 10, 2024

This PR defines the absolute ideal norm of a fractional ideal I : FractionalIdeal R⁰ K where
K is a fraction field of R as a zero-preserving group homomorphism with values in and proves that it generalises the norm on (integral) ideals (and some other classical result).

Also in this PR:

  • Add the directory Mathlib/RingTheory/FractionalIdeal and move the file Mathlib/RingTheory/FractionalIdeal.lean to Mathlib/RingTheory/FractionalIdeal/Basic.lean. The new results are in Mathlib/RingTheory/FractionalIdeal/Norm.lean
  • Define the numerator and denominator of a fractional ideal. These are used to define the norm. Also define a linear equiv between a fractional ideal and its numerator.
  • Several technical lemmas.

Open in Gitpod

@xroblot xroblot added the t-number-theory Number theory (also use t-algebra or t-analysis to specialize) label Jan 10, 2024
@xroblot xroblot added the WIP Work in progress label Jan 10, 2024
@xroblot xroblot added awaiting-review The author would like community review of the PR and removed WIP Work in progress labels Jan 10, 2024
@riccardobrasca riccardobrasca self-assigned this Jan 10, 2024
Mathlib/Algebra/Module/Submodule/Map.lean Show resolved Hide resolved
Mathlib/RingTheory/FractionalIdeal/Basic.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/FractionalIdeal/Basic.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/FractionalIdeal/Norm.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/FractionalIdeal/Basic.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/FractionalIdeal/Norm.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/FractionalIdeal/Norm.lean Outdated Show resolved Hide resolved
Copy link
Member

@riccardobrasca riccardobrasca left a comment

Choose a reason for hiding this comment

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

The file FractionalIdeal.Basic is getting huge, we should split it... but in another PR, thanks!

bors merge

@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review The author would like community review of the PR labels Jan 17, 2024
mathlib-bors bot pushed a commit that referenced this pull request Jan 17, 2024
This PR defines the absolute ideal norm of a fractional ideal `I : FractionalIdeal R⁰ K` where
`K` is a fraction field of `R` as a zero-preserving group homomorphism with values in `ℚ` and proves that it generalises the norm on (integral) ideals  (and some other classical result). 

Also in this PR:
- Add the directory `Mathlib/RingTheory/FractionalIdeal` and move the file `Mathlib/RingTheory/FractionalIdeal.lean` to `Mathlib/RingTheory/FractionalIdeal/Basic.lean`. The new results are in `Mathlib/RingTheory/FractionalIdeal/Norm.lean`
- Define the `numerator` and `denominator` of a fractional ideal. These are used to define the norm. Also define a linear equiv between a fractional ideal and its `numerator`.
- Several technical lemmas.
@mathlib-bors
Copy link

mathlib-bors bot commented Jan 17, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: Generalize absNorm to fractional ideals [Merged by Bors] - feat: Generalize absNorm to fractional ideals Jan 17, 2024
@mathlib-bors mathlib-bors bot closed this Jan 17, 2024
@mathlib-bors mathlib-bors bot deleted the xfr-fract_ideal_norm branch January 17, 2024 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has been sent to bors. t-number-theory Number theory (also use t-algebra or t-analysis to specialize)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants