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: define ProperConstSMul #6675

Closed
wants to merge 12 commits into from
Closed

Conversation

urkud
Copy link
Member

@urkud urkud commented Aug 19, 2023


I need this as a technical assumption in an upcoming PR about continuity of the action of DomMulAct M on Lp.

Open in Gitpod

@urkud urkud added awaiting-review The author would like community review of the PR t-topology Topological spaces, uniform spaces, metric spaces, filters labels Aug 19, 2023
@urkud urkud requested a review from ADedecker August 19, 2023 02:41
@ADedecker
Copy link
Member

I don't like CocompactSMul as a name due to the existence of https://en.wikipedia.org/wiki/Cocompact_group_action

This is really a technical condition that would never be talked about in usual maths (since everyone just talks about group actions) so I think it's best if the name is a bit technical 😅 but I have no good suggestion.

By the way, do you have a use case for the Lp result for monoid actions?

Comment on lines +1314 to +1316
theorem IsCompact.preimage_continuous [CompactSpace α] [T2Space β] {f : α → β} {s : Set β}
(hs : IsCompact s) (hf : Continuous f) : IsCompact (f ⁻¹' s) :=
(hs.isClosed.preimage hf).isCompact
Copy link
Member

Choose a reason for hiding this comment

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

While you're at it, could you link this to the IsProperMap API by showing that any map from a compact space to a Hausdorff space is proper?

Copy link
Member Author

Choose a reason for hiding this comment

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

BTW, we have IsProperMap and CocompactMap.

Copy link
Member

Choose a reason for hiding this comment

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

I know, I added IsProperMap recently. I claim that it is the right concept mathematically, so I think we should expand this API rather than that of CocompactMap.

Copy link
Member Author

Choose a reason for hiding this comment

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

... right concept mathematically

I wouldn't be surprised if both are useful (or even all 3 if you add isCompact_preimage). BTW, I didn't look closely at the definitions; which one is weaker?

Copy link
Member

Choose a reason for hiding this comment

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

In general spaces (and for continuous functions of course), IsProperMap -> isCompact_preimage -> CocompactMap . The equivalence between isCompact_preimage and CocompactMap works as soon as the codomain is Hausdorff, and everything is equivalent when the codomain is Hausdorff and locally compact.

Regarding isCompact_preimage vs CocompactMap, I think we can safely redefine CocompactMap in terms of compact preimages, the tendsto definition was a nice trick but I don't think it's mathematically meaningful.

IsProperMap on the other hand is indeed significantly stronger than CocompactMap, so I'm not suggesting removing the latter, but I know that some key results that are usually proved for isCompact_preimage maps only work in locally compact space unless you assume the stronger IsProperMap. I don't have a good example in mathlib of such a theorem about CocompactMap, but we have some in the context of group actions. For example you can remove the locally compactness assumption in t2Space_of_properlyDiscontinuousSMul_of_t2Space if you strengthen the definition of ProperlyDiscontinuousSMul to use IsProperMap.

Copy link
Member Author

Choose a reason for hiding this comment

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

Done. I also added a version of isProperMap_iff_ultrafilter for a T₂ codomain.

Copy link
Member Author

@urkud urkud Aug 25, 2023

Choose a reason for hiding this comment

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

Do you think I should use IsProperMap instead of IsCompact s → IsCompact (f ⁻¹' s)? UPD: done.

@ADedecker ADedecker 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 Aug 19, 2023
@urkud
Copy link
Member Author

urkud commented Aug 19, 2023

This is really a technical condition that would never be talked about in usual maths (since everyone just talks about group actions) so I think it's best if the name is a bit technical 😅 but I have no good suggestion.

CompactPreimageSMul (renamed)?

By the way, do you have a use case for the Lp result for monoid actions?

Yes, more details later today. I'm going to use it to prove the following fact (under some topological assumptions on G which I'm trying to minimize - WIP): if H : Subgroup G is topologically dense and s : Set G is a.e. invariant under multiplication of all g ∈ H, then s is a.e. invariant under multiplications of all g : G. In particular, if Subgroup.closure {g} is topologically dense, then multiplication by g is ergodic.

@urkud urkud changed the title feat: define CocompactSMul feat: define CompactPreimageSMul Aug 19, 2023
@urkud urkud 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 Aug 25, 2023
@urkud urkud changed the title feat: define CompactPreimageSMul feat: define ProperConstSMul Aug 25, 2023
Copy link
Member

@ADedecker ADedecker left a comment

Choose a reason for hiding this comment

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

Thanks a lot, and sorry for the slow review!
I have no opinion about proper map / compact preimages in this case, but since properness covers your use cases I'd say it's completely fine to rely on the (small) existing API for IsProperMap.

bors d+


/-- A mixin typeclass saying that the `(c +ᵥ ·)` is a proper map for all `c`.

Note that this is **not** the same as a proper additive action (not yet in `Mathlib`). -/
Copy link
Member

Choose a reason for hiding this comment

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

Could you add the same comment on the multiplicative version?

@bors
Copy link

bors bot commented Aug 30, 2023

✌️ urkud can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added delegated and removed awaiting-review The author would like community review of the PR labels Aug 30, 2023
@urkud
Copy link
Member Author

urkud commented Aug 30, 2023

bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Aug 30, 2023
bors bot pushed a commit that referenced this pull request Aug 30, 2023
@bors
Copy link

bors bot commented Aug 30, 2023

Pull request successfully merged into master.

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot changed the title feat: define ProperConstSMul [Merged by Bors] - feat: define ProperConstSMul Aug 30, 2023
@bors bors bot closed this Aug 30, 2023
@bors bors bot deleted the YK-cocompact-smul branch August 30, 2023 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated ready-to-merge This PR has been sent to bors. t-topology Topological spaces, uniform spaces, metric spaces, filters
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants