Skip to content

feat: three misc lemmas about Hausdorff distance#10303

Closed
grunweg wants to merge 7 commits intomasterfrom
MR-sphere-eversion-Hausdorff-distance2
Closed

feat: three misc lemmas about Hausdorff distance#10303
grunweg wants to merge 7 commits intomasterfrom
MR-sphere-eversion-Hausdorff-distance2

Conversation

@grunweg
Copy link
Copy Markdown
Contributor

@grunweg grunweg commented Feb 6, 2024

From sphere-eversion, I'm just submitting these.


Open in Gitpod

From sphere-eversion, I'm just submitting these.
@grunweg grunweg added awaiting-review t-topology Topological spaces, uniform spaces, metric spaces, filters labels Feb 6, 2024
open Metric in
theorem IsCompact.exists_thickening_image [PseudoEMetricSpace α] [PseudoEMetricSpace β] {f : α → β}
{K : Set α} {U : Set β} (hK : IsCompact K)
(ho : IsOpen U) (hf : Continuous f) (hKU : MapsTo f K U) :
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The assumption hf seems too strong here: isn't it enough to assume that f is continuous on K?

Copy link
Copy Markdown
Contributor Author

@grunweg grunweg Feb 10, 2024

Choose a reason for hiding this comment

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

It would seem so, but I don't think this actually holds. You need at least continuity in a neighbourhood containing K.

I presume continuity on a neighbourhood of K suffices. (I have proven an analogue of tendsto_nhdsSet with this assumption.) I can try to finish this lemma.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I pushed the helper lemma: I'm not sure where the best place for it is.
NhdsSet is not possible (that file cannot import ContinuousOn; #find_home says ContinuousOn (which doesn't have other lemmas about NhdsSet yet). I put my new lemma at the end, and moved two related lemmas with it - and am not sure if this is the best call.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think that continuity at every point of K should be sufficient, no?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

(That's equivalent to ContinuousOn f K.)
Are you sure? The current proof in ContinuousOn.tendsto_nhdsSet requires more than that; I'm skeptical that this can be weakened.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

(I have finished proving generalising the "main lemma", requiring only continuity in a neighbourhood of K.)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No, that's not equivalent to ContinuousOn f K, which only requires continuity within K while I'm requiring full continuity at those points.

Here is a sketch: at every point x of K, the map is continuous, so a small neighborhood is mapped to a small neighborhood, and in particular one can find a neighborhood of x whose image has a thickening contained in U, say by \epsilon_x. Then cover K by finitely many of such sets, call V their union, and let epsilon be the minimum of epsilon_x over these finitely many x. (I'd probably do the Lean proof using IsCompact.induction_on).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

💡 I will try implementing this, not before next weekend.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have now addressed this, and opened a new PR for simplicity: #20585.
Would you like to take a look again?

@sgouezel sgouezel added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Feb 9, 2024
Co-authored-by: sgouezel <sebastien.gouezel@univ-rennes1.fr>
@grunweg grunweg added awaiting-review and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Feb 10, 2024
@sgouezel sgouezel added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Feb 11, 2024
@grunweg grunweg added awaiting-review awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-author A reviewer has asked the author a question or requested changes. awaiting-review labels Feb 11, 2024
@ghost ghost added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 4, 2024
@grunweg grunweg added the help-wanted The author needs attention to resolve issues label Mar 18, 2024
@grunweg
Copy link
Copy Markdown
Contributor Author

grunweg commented Mar 18, 2024

I will get to this PR eventually, but have other higher-priority things to do right now. If you're reading this, help jumping in and fixing the above is welcome.

@grunweg
Copy link
Copy Markdown
Contributor Author

grunweg commented Jan 9, 2025

An updated version is available at #20585; let me close this PR in favour of that one.

@grunweg grunweg closed this Jan 9, 2025
@grunweg grunweg deleted the MR-sphere-eversion-Hausdorff-distance2 branch January 9, 2025 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-author A reviewer has asked the author a question or requested changes. help-wanted The author needs attention to resolve issues merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) t-topology Topological spaces, uniform spaces, metric spaces, filters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants