Skip to content

Finite topology property + simple theorems#1803

Open
artemetra wants to merge 11 commits into
mainfrom
artem/finite-topology-1
Open

Finite topology property + simple theorems#1803
artemetra wants to merge 11 commits into
mainfrom
artem/finite-topology-1

Conversation

@artemetra

Copy link
Copy Markdown
Collaborator

Initial PR for #1800.
Defines Finite topology and shows finite topology => Artinian and Noetherian, needs more work

@artemetra

artemetra commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator Author

@prabau I did what you mentioned, however I don't think this PR is useful as-is currently, and I suggest at least adding a space that has finite topology but isn't finite or indiscrete; and replacing (deleting?) the finite => Artinian and finite => Noetherian theorems.

@artemetra

Copy link
Copy Markdown
Collaborator Author

I added a couple more pretty simple theorems and it seems like the property has been established for all spaces now. The above comment still holds, tho.

Comment thread properties/P000245.md
Comment thread properties/P000245.md
Comment thread properties/P000245.md Outdated
Comment thread theorems/T000909.md Outdated
Comment thread theorems/T000910.md
Comment thread theorems/T000912.md Outdated
Comment thread properties/P000245.md Outdated
Comment thread properties/P000245.md Outdated
- Has finitely many open sets
---

The cardinality of the topology on $(X, \tau)$ is finite, i.e. $|\tau| < \infty$.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
The cardinality of the topology on $(X, \tau)$ is finite, i.e. $|\tau| < \infty$.
$X$ has finitely many open sets.

simpler

Comment thread properties/P000245.md Outdated
Comment thread properties/P000245.md
Comment thread properties/P000245.md Outdated
@prabau

prabau commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

T911 is not needed here. Since indiscrete spaces are Artinian and Noetherian (already known in pi-base), the theorem will be a direct consequence of the second PR.
see #1803 (comment)

@prabau

prabau commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

T909, T910: we don't want to introduce theorems showing that finite topology implies Artinian and Noetherian. Instead the existing corresponding theorems for finite spaces should be modified (T198, T825).

(When we generalize theorems we usually try to modify the less general theorem in place, except in some cases where we want to preserve the previous less general result for didactic reasons. Here, Artinian and Noetherian are not very basic properties, so it's fine to modify the results directly.)

Comment thread theorems/T000908.md Outdated
@prabau

prabau commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

I changed the name from Finite topology to Has a finite topology to make the discussion easier.
Now deciding between (1) Has a finite topology and (2) Has finitely many open sets.

I like (2). But the reason I had suggested the shorter name (1) was that a shorter name makes things easier to grasp when used in theorems. Long property names that are basically a description of the meaning seem harder to handle in that context.

But maybe (2) is not that bad after all. So why don't we do the following: change the name to (2) and see how we feel about it after the second PR.

Note: we do not need an "alias" in this case. Aliases are meant to be terminology commonly used in the literature, so that someone looking for the property under that name would be able to more easily find the property in pi-base. Since the current property usually does not appear in the literature and certainly not with a well-known name, aliases serve no purpose here. So bottom line: don't add Has a finite topology as an alias.

@StevenClontz fyi

@prabau

prabau commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

@artemetra T912: I find this a little cryptic maybe. Can you explain here in words what the reasoning is?

Wait, I think I get it:
Kol(X) is finite, and is equal to X since X is T0. So X is finite.

@prabau

prabau commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

T911 indiscrete => finite topology.
Thinking some more about it, we can keep this. But we can and should remove T251 indiscrete => compact then.
Not sure if other results involving indiscrete can be cleaned up further. I have not looked at it.

@felixpernegger

Copy link
Copy Markdown
Collaborator

We can improve T823

@prabau

prabau commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

and remove T658 at the same time. There must be more cleanup of that kind lying around. Not sure we want to do it all necessarily in this PR. But maybe ok if it's kind of trivial.

Co-authored-by: Patrick Rabau <70125716+prabau@users.noreply.github.com>
Comment thread properties/P000245.md Outdated
@felixpernegger

Copy link
Copy Markdown
Collaborator

@artemetra I think once you apply all the suggestions we are ready to merge

artemetra and others added 3 commits June 19, 2026 19:35
Co-authored-by: Patrick Rabau <70125716+prabau@users.noreply.github.com>
Co-authored-by: Patrick Rabau <70125716+prabau@users.noreply.github.com>
Co-authored-by: Felix Pernegger <s59fpern@uni-bonn.de>
Comment thread theorems/T000912.md Outdated

@felixpernegger felixpernegger left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Also, I fixed a small typo and indendation error and applied it directly, I hope you mind

Comment thread properties/P000245.md Outdated
@prabau

prabau commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

@artemetra I think once you apply all the suggestions we are ready to merge

@felixpernegger You often say exactly the same thing, and often something comes up after the suggestions are applied. Would be good to first have the suggestion applied and then analyze the results further. As there may be further suggestions that have not been given yet. (Sorry for the rant)

Comment thread properties/P000245.md
Comment on lines +4 to +5
aliases:
- Has a finite topology

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
aliases:
- Has a finite topology

per #1803 (comment)

Comment thread theorems/T000912.md
P000078: true
---

Follows from the Kolmogorov quotient being finite.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Follows from the Kolmogorov quotient being finite.
$\text{Kol}(X)$ is finite, and is equal to $X$ since $X$ is {P1}. So $X$ is finite.

#1803 (comment)

@prabau

prabau commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

@artemetra There were several further suggestions in the discussion above that need to be applied, regarding T909, T910, and other related theorems. Still working on it, I assume?

@prabau prabau left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

need more work (ensures that the PR does not get prematurely approved and merged)

@felixpernegger

Copy link
Copy Markdown
Collaborator

@artemetra I think once you apply all the suggestions we are ready to merge

@felixpernegger You often say exactly the same thing, and often something comes up after the suggestions are applied. Would be good to first have the suggestion applied and then analyze the results further. As there may be further suggestions that have not been given yet. (Sorry for the rant)

I have been making PRs in this repo for ~8 months now (and in that timespan almost the majority of all PRs were made by me) as well reviewed many others. I think am I qualified to make judgement calls.
It is not necessary at every single PR for you to give the final approval to go ahead, especially if you have shared suggestions already (which were implemented). You don't do the same in other PRs as well.

@prabau

prabau commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

@artemetra I think once you apply all the suggestions we are ready to merge

@felixpernegger You often say exactly the same thing, and often something comes up after the suggestions are applied. Would be good to first have the suggestion applied and then analyze the results further. As there may be further suggestions that have not been given yet. (Sorry for the rant)

I have been making PRs in this repo for ~8 months now (and in that timespan almost the majority of all PRs were made by me) as well reviewed many others. I think am I qualified to make judgement calls. It is not necessary at every single PR for you to give the final approval to go ahead, especially if you have shared suggestions already (which were implemented). You don't do the same in other PRs as well.

Of course not. But for this particular PR there were several things that still needed to be applied. And the PR should not have been approved without further changes and discussion.

@prabau

prabau commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

@felixpernegger I guess for next time, if I have further pending comments, I should say it up front. Sometimes I don't say it so that the discussion can be focused on one thing at a time. My bad.

@felixpernegger

Copy link
Copy Markdown
Collaborator

@felixpernegger I guess for next time, if I have further pending comments, I should say it up front. Sometimes I don't say it so that the discussion can be focused on one thing at a time. My bad.

Good idea, no worries

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants