Zulip Chat Archive

Stream: general

Topic: ZFA namespace


Yury G. Kudryashov (Feb 27 2023 at 04:38):

Should we move ZFA stuff in a namespace? Someone who imports all of mathlib can confuse, e.g., docs#finsets with docs#finset. Also, moving to a NS will allow us to use ZFA.finset instead of finsets.

Mario Carneiro (Feb 27 2023 at 04:41):

no it's not supposed to be finset

Mario Carneiro (Feb 27 2023 at 04:41):

it is hereditary finsets

Yury G. Kudryashov (Feb 27 2023 at 04:41):

Also, the module docstring should give some background about ZFA.

Mario Carneiro (Feb 27 2023 at 04:41):

in fact ZFA is also a bit of a misnomer because ZFA has infinite stuff

Yury G. Kudryashov (Feb 27 2023 at 04:42):

I don't see why the name finsets is better than finset. hfinset?

Mario Carneiro (Feb 27 2023 at 04:42):

this is more like "grade school set theory"

Mario Carneiro (Feb 27 2023 at 04:42):

where you write down examples of sets like {{}, 3, {14}}

Yury G. Kudryashov (Feb 27 2023 at 04:43):

It would be nice to have some references in the module docstring and an explanation of what happens here.

Yury G. Kudryashov (Feb 27 2023 at 04:44):

And I think that hiding a rarely used theory in a namespace is a good idea.

Mario Carneiro (Feb 27 2023 at 04:45):

ok, but ZFA makes it sound scarier than it should be

Mario Carneiro (Feb 27 2023 at 04:46):

the main purpose is to be able to do computable set theory toy examples

Yury G. Kudryashov (Feb 27 2023 at 04:46):

What NS would you suggest?

Yury G. Kudryashov (Feb 27 2023 at 04:48):

The only reason I opened this file was to forward-port a change to the module docstring. Probably, I will never open it in the next year or ten.

Mario Carneiro (Feb 27 2023 at 04:48):

ok, so maybe just leave it?

Mario Carneiro (Feb 27 2023 at 04:48):

It's a leaf file anyway

Yury G. Kudryashov (Feb 27 2023 at 04:49):

My main concern is that someone imports the whole Mathlib, then accidentally writes Finsets α instead of Finset α.

Mario Carneiro (Feb 27 2023 at 04:50):

Maybe put a warning in the docstring?

Mario Carneiro (Feb 27 2023 at 04:51):

That's going to be an issue no matter what it is called

Yury G. Kudryashov (Feb 27 2023 at 04:52):

If it's in a namespace, then code completion won't suggest it after you type Finse

Mario Carneiro (Feb 27 2023 at 04:53):

it should, that's definitely a lean 4 bug (lean 3 would do so)

Yury G. Kudryashov (Feb 27 2023 at 04:54):

Probably, it's not a Lean 4 bug but my wrong guess.

Mario Carneiro (Feb 27 2023 at 04:54):

plus it shows up in searches and such

Yury G. Kudryashov (Feb 27 2023 at 04:54):

I didn't test.

Mario Carneiro (Feb 27 2023 at 04:54):

as long as it has a name vaguely related to finset we will need to address the difference in the docstring


Last updated: Dec 20 2023 at 11:08 UTC