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

feat(topology/compact_open): express the compact-open topology as an Inf of topologies #9046

Closed
wants to merge 8 commits into from

Conversation

@hrmacbeth hrmacbeth added the blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. label Sep 7, 2021
@github-actions github-actions bot removed the blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. label Sep 7, 2021
@hrmacbeth hrmacbeth added the blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. label Sep 7, 2021
@github-actions github-actions bot removed the blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. label Sep 7, 2021
@github-actions github-actions bot added the merge-conflict Please `git merge origin/master` then a bot will remove this label. label Sep 8, 2021
@github-actions github-actions bot removed the merge-conflict Please `git merge origin/master` then a bot will remove this label. label Sep 8, 2021
@semorrison semorrison added the awaiting-review The author would like community review of the PR label Sep 9, 2021
@hrmacbeth hrmacbeth removed the awaiting-review The author would like community review of the PR label Sep 9, 2021
@hrmacbeth
Copy link
Member Author

I made a new PR, #9106, to replace this one, and will close this one once @ocfnash thinks through to confirm my doubts about this one.

In this PR it is established that the compact-open topology is equal to

⨅ (s : set α) (hs : is_compact s),
  topological_space.generate_from {m | ∃ (u : set β) (hu : is_open u), m = {f | f '' s ⊆ u}}

but actually, I think that

topological_space.generate_from {m | ∃ (u : set β) (hu : is_open u), m = {f | f '' s ⊆ u}}

is not the same as the topology of uniform convergence on s? In this topology there would be no "small" neighbourhoods of a function with large range.

The result I prove in the other PR is that the compact-open topology is equal to

⨅ (s : set α) (hs : is_compact s),
  topological_space.induced (λ f, f.restrict s) continuous_map.compact_open

@ocfnash
Copy link
Collaborator

ocfnash commented Sep 9, 2021

I made a new PR, #9106, to replace this one, and will close this one once @ocfnash thinks through to confirm my doubts about this one.

Yes I think you're right, though I admit I'm only looking at this from a formal point of view.

I suppose the point is that life will be easiest if we can express the topology as an Inf over compact subsets of K ⊆ α and furthermore that each component in the Inf is the same topology that K gets when regarded as a topological space in its own right (i.e., exactly what you prove in #9106).

I guess that way, if we take the same approach for the uniform structures we should be able to prove is_open_uniformity one K at a time (via https://leanprover-community.github.io/mathlib_docs/topology/uniform_space/basic.html#to_topological_space_Inf).

Of course, unlike this PR, #9106 no longer provides an alternate definition so in a way both are valuable. Similar remarks apply to the uniform structure (assuming I've got the above correct).

Finally I should say that it is likely to be a couple of weeks till I get back to this: I'm going on holiday for 11 days tomorrow and I'm still pushing to get some last bits about convexity finished before I head off. It's a shame because I'm excited to see your work on this but we'll definitely get there.

@hrmacbeth hrmacbeth closed this Sep 9, 2021
@YaelDillies YaelDillies deleted the locally-uniform-tsum branch March 29, 2023 22:11
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

3 participants