Zulip Chat Archive
Stream: mathlib4
Topic: Naming around local homeomorphisms
Michael Rothgang (Dec 10 2023 at 17:41):
Right now, mathlib has LocalHomeomorph
and IsLocallyHomeomorph
: these sound very similar, but are in fact different.
The latter is called a "local homeomorphism" in informal, textbook mathematics. The former could rather be called a partial homeomorphism: it's a homeomorphism on an open set of the domain (extended to the whole space, by the junk value pattern). I would prefer renaming these.
- @Floris van Doorn suggested renaming
LocalHomeomorph
toPartialHomeomorph
(I have a wip branch performing this rename) - I would prefer renaming
IsLocallyHomeomorph
toIsLocalHomeomorph
. I don't see a good reason to deviate from math terminology. (This would matchIsLocalStructomorphWithinAt
. It is different fromIsLocallyInjective
; I'd argue this is different since one would always speak of a "locally injective map".)
@Thomas Browning You wrote this file; do you see a reason not to do this?
Yury G. Kudryashov (Dec 10 2023 at 18:20):
AFAIR, the main reason for naming docs#IsLocallyHomeomorph was to avoid renaming of LocalHomeomorph
mid-port.
Yury G. Kudryashov (Dec 10 2023 at 18:22):
And the main issue with PartialHomeomorph
(and, I guess, PartialEquiv
) is that it's too similar to docs#PEquiv (AFAICT, mostly unused in Mathlib).
Thomas Browning (Dec 10 2023 at 18:54):
Yes, I agree that the naming is confusing right now. If a better pair of names is found, I don't see any reason not to rename. But it would be good to keep the names of LocalEquiv
and LocalHomeomorph
similar to each other.
Floris van Doorn (Dec 11 2023 at 01:00):
I'm in favor of renaming LocalEquiv
to PartialEquiv
and LocalHomeomorph
to PartialHomeomorph
. I think it's fine to have similar names PEquiv
and PartialEquiv
, since they also mean almost the same thing.
Michael Rothgang (Dec 11 2023 at 16:56):
Sounds like there is consensus (among those speaking up) for the new names, great.
I've submitted #8982 for renaming LocalHomeomorph; LocalEquiv is up for volunteers (or I'll do it later this week).
Michael Rothgang (Dec 11 2023 at 17:04):
And #8983 for IsLocallyHomeomorph{On}
.
Michael Rothgang (Dec 11 2023 at 20:40):
Actually, I've just done LocalEquiv
also: #8984
Doesn't lint (line length); feel free to jump in and fix these (or I'll do it later).
Last updated: Dec 20 2023 at 11:08 UTC