Skip to content

[Merged by Bors] - feat(NumberField/InfinitePlace/Ramification): add placesOver#36132

Closed
smmercuri wants to merge 10 commits intoleanprover-community:masterfrom
smmercuri:infinitePlacesOver
Closed

[Merged by Bors] - feat(NumberField/InfinitePlace/Ramification): add placesOver#36132
smmercuri wants to merge 10 commits intoleanprover-community:masterfrom
smmercuri:infinitePlacesOver

Conversation

@smmercuri
Copy link
Copy Markdown
Collaborator

@smmercuri smmercuri commented Mar 4, 2026

  • This PR defines sets of infinite places of L that lie above a fixed infinite place v of K, as well as subsets of ramified/unramified such places.
  • Also define analogous concepts for complex embeddings.
  • Fix order of parameters in ComplexEmbedding.LiesOver to match the order of params in AbsoluteValue.LiesOver make it protected.

Open in Gitpod

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 4, 2026

PR summary 1dc478f792

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ coe_mk_comp
+ conjugate_embedding_mem_mixedEmbeddingsOver
+ disjoint_ramifiedPlacesOver_unramifiedPlacesOver
+ disjoint_unmixedEmbeddingsOver_mixedEmbeddingsOver
+ embedding_mem_mixedEmbeddingsOver
+ instance {φ : K →+* ℂ} {ψ : L →+* ℂ} [ComplexEmbedding.LiesOver ψ φ] :
+ liesOver_conjugate_embedding_of_mem_ramifiedPlacesOver
+ liesOver_embedding_of_mem_ramifiedPlacesOver
+ mixedEmbeddingsOver
+ mk_mem_ramifiedPlacesOver
+ mk_mem_unramifiedPlacesOver
+ placesOver
+ ramifiedPlacesOver
+ union_ramifiedPlacesOver_unramifiedPlacesOver
+ union_unmixedEmbeddingsOver_mixedEmbeddingsOver
+ unmixedEmbeddingsOver
+ unramifiedPlacesOver
++-- LiesOver
- LiesOver.comp_eq

You can run this locally as follows
## summary with just the declaration names:
./scripts/pr_summary/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/pr_summary/declarations_diff.sh long <optional_commit>

The doc-module for scripts/pr_summary/declarations_diff.sh contains some details about this script.


No changes to technical debt.

You can run this locally as

./scripts/reporting/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).

@riccardobrasca
Copy link
Copy Markdown
Member

@xroblot can you have a look?

@riccardobrasca
Copy link
Copy Markdown
Member

(there is a building error btw)

@riccardobrasca riccardobrasca self-assigned this Mar 4, 2026
@smmercuri smmercuri added the FLT part of the ongoing formalization of Fermat's Last Theorem label Mar 4, 2026
@riccardobrasca
Copy link
Copy Markdown
Member

Thanks!

bors d+

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Mar 5, 2026

✌️ smmercuri can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@mathlib-triage mathlib-triage bot added the delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). label Mar 5, 2026
@smmercuri
Copy link
Copy Markdown
Collaborator Author

bors r+

mathlib-bors bot pushed a commit that referenced this pull request Mar 5, 2026
- This PR defines sets of infinite places of `L` that lie above a fixed infinite place `v` of `K`, as well as subsets of ramified/unramified such places.
- Also define analogous concepts  for complex embeddings.
- Fix order of parameters in `ComplexEmbedding.LiesOver` to match the order of params in `AbsoluteValue.LiesOver` make it protected.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Mar 5, 2026

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(NumberField/InfinitePlace/Ramification): add placesOver [Merged by Bors] - feat(NumberField/InfinitePlace/Ramification): add placesOver Mar 5, 2026
@mathlib-bors mathlib-bors bot closed this Mar 5, 2026
xroblot pushed a commit to xroblot/mathlib4 that referenced this pull request Mar 9, 2026
…rover-community#36132)

- This PR defines sets of infinite places of `L` that lie above a fixed infinite place `v` of `K`, as well as subsets of ramified/unramified such places.
- Also define analogous concepts  for complex embeddings.
- Fix order of parameters in `ComplexEmbedding.LiesOver` to match the order of params in `AbsoluteValue.LiesOver` make it protected.
seewoo5 pushed a commit to seewoo5/mathlib4 that referenced this pull request Mar 9, 2026
…rover-community#36132)

- This PR defines sets of infinite places of `L` that lie above a fixed infinite place `v` of `K`, as well as subsets of ramified/unramified such places.
- Also define analogous concepts  for complex embeddings.
- Fix order of parameters in `ComplexEmbedding.LiesOver` to match the order of params in `AbsoluteValue.LiesOver` make it protected.
xroblot pushed a commit to xroblot/mathlib4 that referenced this pull request Mar 10, 2026
…rover-community#36132)

- This PR defines sets of infinite places of `L` that lie above a fixed infinite place `v` of `K`, as well as subsets of ramified/unramified such places.
- Also define analogous concepts  for complex embeddings.
- Fix order of parameters in `ComplexEmbedding.LiesOver` to match the order of params in `AbsoluteValue.LiesOver` make it protected.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). FLT part of the ongoing formalization of Fermat's Last Theorem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants