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

Add nonempty in definition of hyperconnected and ultraconnected #55

Merged
merged 2 commits into from
Aug 31, 2020

Conversation

prabau
Copy link
Collaborator

@prabau prabau commented Aug 25, 2020

The empty set is disjoint from any other set. So to be strictly correct, the "nonempty" qualifier should be added in the definition of hyperconnected and ultraconnected spaces.

Copy link
Collaborator Author

@prabau prabau left a comment

Choose a reason for hiding this comment

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

The change here generalizes the existing Theorem 110 by weakening the T4 hypothesis to normal with the same conclusion. As a result of this, three spaces that previously had "weakly countably compact" asserted will now have it deduced. This would allow to remove the corresponding property files for these spaces. What is the policy when a property for a space is both asserted and deduced?

The three spaces in question are:

  • Right ordered reals (S42)
  • Nested interval topology (S44)
  • Divisor topology (S49)

@StevenClontz StevenClontz self-requested a review August 31, 2020 18:53
Copy link
Member

@StevenClontz StevenClontz left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

@StevenClontz StevenClontz merged commit 04f15f4 into pi-base:master Aug 31, 2020
@prabau
Copy link
Collaborator Author

prabau commented Sep 1, 2020

@StevenClontz @jamesdabbs See the comment in PR #55 with this question from Aug 26. We now have some cases where a property for a given space is both asserted from an item in Steen & Seebach, and deduced from various theorems in the database. For example space S000042 (right ordered reals) is weakly countably compact, but this can now be deduced from T000110. In the generated data on the web site, assertions seem to take precedence over deductions.

What is your preference about such cases? If we remove the files that assert these properties, that will convert it into a deduction and will slightly streamline the data in the database. If that is the recommended policy, let me know and I can do a pull request.

@StevenClontz
Copy link
Member

There are pedagogical reasons to avoid the policy of strictly forbidding redundant assertions; even if a property can be deduced automatically, there might be a classic textbook reference for it as well. That said, if we implemented #16 we could simply omit the value boolean and add the annotation that way.

But if you've found a way to remove redundant assertions and they don't say anything beyond e.g. "Asserted in Counterexamples page XX" then I definitely don't mind removing them. Ideally this is something we should be able to do automatically down the road. I'll open an issue if we don't already have one to that effect.

@prabau
Copy link
Collaborator Author

prabau commented Sep 1, 2020

Makes sense about the pedagogical reasons. I'll take a closer look at a few cases.

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.

None yet

2 participants