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: infinite normed field is noncompact #8349

Closed
wants to merge 3 commits into from

Conversation

urkud
Copy link
Member

@urkud urkud commented Nov 11, 2023

Generalize from nontrivially normed fields to normed fields.
Also prove that a nontrivially normed field is infinite.


Open in Gitpod

@urkud urkud added awaiting-review The author would like community review of the PR t-analysis Analysis (normed *, calculus) labels Nov 11, 2023
exact ⟨fun h ↦ NormedSpace.unbounded_univ 𝕜 E h.isBounded⟩
· push_neg at H
rcases exists_ne (0 : E) with ⟨x, hx⟩
suffices ClosedEmbedding (Infinite.natEmbedding 𝕜 · • x) from this.noncompactSpace
Copy link
Collaborator

Choose a reason for hiding this comment

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

That would be good to have these closed embeddings in general? and even in finite dimension.

Copy link
Member Author

Choose a reason for hiding this comment

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

What general statement are you talking about? https://leanprover-community.github.io/mathlib4_docs/Mathlib/Topology/Algebra/Module/FiniteDimension.html#closedEmbedding_smul_left ? Another statement is that a trivially normed field has discrete topology (so, in fact we can skip Infinite.natEmbedding 𝕜).

Copy link
Collaborator

Choose a reason for hiding this comment

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

I was thinking of the suffices that you prove on the next lines. I guess it also holds when the field is nontrivially valued.
Actually, I am not sure you need to make the cases disjoint: once you reduce to the case of a line by these closed embeddings, what matters is that k is noncompact.

Copy link
Member Author

Choose a reason for hiding this comment

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

If k = Q, then this scalar multiplication is not a closed embedding.

@sgouezel
Copy link
Contributor

bors r+
Thanks!

@github-actions github-actions 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 Nov 12, 2023
mathlib-bors bot pushed a commit that referenced this pull request Nov 12, 2023
Generalize from nontrivially normed fields to normed fields.
Also prove that a nontrivially normed field is infinite.
@mathlib-bors
Copy link

mathlib-bors bot commented Nov 12, 2023

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: infinite normed field is noncompact [Merged by Bors] - feat: infinite normed field is noncompact Nov 12, 2023
@mathlib-bors mathlib-bors bot closed this Nov 12, 2023
@mathlib-bors mathlib-bors bot deleted the YK-noncompact branch November 12, 2023 13:54
Copy link
Collaborator

@YaelDillies YaelDillies left a comment

Choose a reason for hiding this comment

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

Looks good!

maintainer merge

alexkeizer pushed a commit that referenced this pull request Nov 17, 2023
Generalize from nontrivially normed fields to normed fields.
Also prove that a nontrivially normed field is infinite.
alexkeizer pushed a commit that referenced this pull request Nov 21, 2023
Generalize from nontrivially normed fields to normed fields.
Also prove that a nontrivially normed field is infinite.
grunweg pushed a commit that referenced this pull request Dec 15, 2023
Generalize from nontrivially normed fields to normed fields.
Also prove that a nontrivially normed field is infinite.
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-analysis Analysis (normed *, calculus)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants