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(topology/connected): sufficient conditions for the preimage of a connected set to be connected #10289

Closed
wants to merge 3 commits into from

Conversation

YaelDillies
Copy link
Collaborator

and other simple connectedness results


From LTE
Co-authored-by: Kevin Buzzard k.buzzard@imperial.ac.uk
Open in Gitpod

@YaelDillies YaelDillies added the awaiting-review The author would like community review of the PR label Nov 12, 2021
@AntoineChambert-Loir
Copy link
Collaborator

There is a more general theorem to prove : If a continuous map f is strict (the quotient topology on the image coincides with its topology of a subspace) and if the fibers of f are preconnected, then the preimage of a preconnected set is preconnected.

Sufficient conditions for a map to be strict are to be closed, or to be open.

An injective open map is strict, and the fibers are (sub)singletons hence are preconnected.

@YaelDillies
Copy link
Collaborator Author

That sounds significantly more advanced. Do we even have strict maps?
I think it's better to stay with this proof now and if someone comes along and proves your theorem we can shorten the already short proof of my lemmas.

@AntoineChambert-Loir
Copy link
Collaborator

Probably, strict maps are not in mathlib yet.
Two cases which are already worth being done are when the map is open or closed.

@jcommelin jcommelin changed the title feat(topology/connected): The preimage of a connected set is connected feat(topology/connected): sufficient conditions for the preimage of a connected set to be connected Nov 15, 2021
@jcommelin
Copy link
Member

I changed the PR title to something that isn't strictly false 😜

@YaelDillies Could you please turn the comments by @AntoineChambert-Loir into a TODO in the code? So that someone who stumbles on them later can start an API for strict maps?

@jcommelin jcommelin added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Nov 15, 2021
@AntoineChambert-Loir
Copy link
Collaborator

I changed the PR title to something that isn't strictly false 😜

Actually, @jcommelin , this was the very reason why I opened the PR and loooked at it !

@YaelDillies YaelDillies added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Nov 20, 2021
src/topology/connected.lean Outdated Show resolved Hide resolved
src/topology/connected.lean Outdated Show resolved Hide resolved
Copy link
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

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

Thanks 🎉

bors merge

@github-actions github-actions bot added ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) and removed awaiting-review The author would like community review of the PR labels Nov 23, 2021
bors bot pushed a commit that referenced this pull request Nov 23, 2021
… connected set to be connected (#10289)

and other simple connectedness results



Co-authored-by: Johan Commelin <johan@commelin.net>
@bors
Copy link

bors bot commented Nov 23, 2021

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Nov 23, 2021
… connected set to be connected (#10289)

and other simple connectedness results



Co-authored-by: Johan Commelin <johan@commelin.net>
@bors
Copy link

bors bot commented Nov 23, 2021

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(topology/connected): sufficient conditions for the preimage of a connected set to be connected [Merged by Bors] - feat(topology/connected): sufficient conditions for the preimage of a connected set to be connected Nov 23, 2021
@bors bors bot closed this Nov 23, 2021
@bors bors bot deleted the connected_preimage branch November 23, 2021 15:05
ericrbg pushed a commit that referenced this pull request Nov 24, 2021
… connected set to be connected (#10289)

and other simple connectedness results



Co-authored-by: Johan Commelin <johan@commelin.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants