Skip to content

[Merged by Bors] - feat(FieldTheory/SeparableDegree): Field.Emb is infinite for transcendental extension#18770

Closed
acmepjz wants to merge 14 commits intomasterfrom
acmepjz_field_emb_infinite
Closed

[Merged by Bors] - feat(FieldTheory/SeparableDegree): Field.Emb is infinite for transcendental extension#18770
acmepjz wants to merge 14 commits intomasterfrom
acmepjz_field_emb_infinite

Conversation

@acmepjz
Copy link
Copy Markdown
Collaborator

@acmepjz acmepjz commented Nov 8, 2024

  • Field.infinite_emb_of_transcendental, Field.finSepDegree_eq_zero_of_transcendental: Field.Emb is infinite and Field.finSepDegree is zero for transcendental extensions
  • remove algebraic assumptions in isPurelyInseparable_(of|iff)_finSepDegree_eq_one
  • prove IsPurelyInseparable.of_injective_comp_algebraMap
  • change Field.Emb from def to abbrev

Open in Gitpod

@acmepjz acmepjz added WIP Work in progress t-algebra Algebra (groups, rings, fields, etc) labels Nov 8, 2024
@acmepjz acmepjz changed the title test feat(FieldTheory/SeparableDegree): Field.Emb is infinite for transcendental extension Nov 8, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 8, 2024

PR summary f17a842ef8

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.FieldTheory.SeparableDegree 1421 1425 +4 (+0.28%)
Mathlib.FieldTheory.PurelyInseparable 1435 1439 +4 (+0.28%)
Import changes for all files
Files Import difference
8 files Mathlib.FieldTheory.IsPerfectClosure Mathlib.FieldTheory.SeparableClosure Mathlib.RingTheory.Unramified.Field Mathlib.FieldTheory.SeparableDegree Mathlib.FieldTheory.Galois.GaloisClosure Mathlib.RingTheory.Etale.Field Mathlib.FieldTheory.Galois.Profinite Mathlib.FieldTheory.PurelyInseparable
4

Declarations diff

+ IsPurelyInseparable.of_injective_comp_algebraMap
+ finSepDegree_eq_zero_of_transcendental
+ infinite_emb_of_transcendental

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.


No changes to technical debt.

You can run this locally as

./scripts/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Nov 8, 2024
@acmepjz acmepjz removed the WIP Work in progress label Nov 8, 2024
@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 Nov 11, 2024
# Conflicts:
#	Mathlib/FieldTheory/SeparableDegree.lean
@leanprover-community-bot-assistant leanprover-community-bot-assistant removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Nov 11, 2024
@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Nov 15, 2024
@acmepjz acmepjz requested a review from alreadydone November 15, 2024 11:00
Copy link
Copy Markdown
Contributor

@alreadydone alreadydone left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM!
It turns out #9392 is not needed, and Field.embProdEmbOfIsAlgebraic is enough.
Here are some golfs:

Co-authored-by: Junyan Xu <junyanxu.math@gmail.com>
@alreadydone
Copy link
Copy Markdown
Contributor

maintainer merge

@github-actions
Copy link
Copy Markdown

🚀 Pull request has been placed on the maintainer queue by alreadydone.

@github-actions github-actions bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Nov 16, 2024
@riccardobrasca
Copy link
Copy Markdown
Member

Thanks!

bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Nov 16, 2024
mathlib-bors bot pushed a commit that referenced this pull request Nov 16, 2024
…endental extension (#18770)

- `Field.infinite_emb_of_transcendental`, `Field.finSepDegree_eq_zero_of_transcendental`: `Field.Emb` is infinite and `Field.finSepDegree` is zero for transcendental extensions
- remove algebraic assumptions in `isPurelyInseparable_(of|iff)_finSepDegree_eq_one`
- prove `IsPurelyInseparable.of_injective_comp_algebraMap`
- change `Field.Emb` from `def` to `abbrev`
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Nov 16, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(FieldTheory/SeparableDegree): Field.Emb is infinite for transcendental extension [Merged by Bors] - feat(FieldTheory/SeparableDegree): Field.Emb is infinite for transcendental extension Nov 16, 2024
@mathlib-bors mathlib-bors bot closed this Nov 16, 2024
@mathlib-bors mathlib-bors bot deleted the acmepjz_field_emb_infinite branch November 16, 2024 01:01
TobiasLeichtfried pushed a commit that referenced this pull request Nov 21, 2024
…endental extension (#18770)

- `Field.infinite_emb_of_transcendental`, `Field.finSepDegree_eq_zero_of_transcendental`: `Field.Emb` is infinite and `Field.finSepDegree` is zero for transcendental extensions
- remove algebraic assumptions in `isPurelyInseparable_(of|iff)_finSepDegree_eq_one`
- prove `IsPurelyInseparable.of_injective_comp_algebraMap`
- change `Field.Emb` from `def` to `abbrev`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. ready-to-merge This PR has been sent to bors. t-algebra Algebra (groups, rings, fields, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants