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

Add note about df-clel , df-cleq and df-clab #3375

Merged
merged 1 commit into from
Aug 14, 2023

Conversation

jkingdon
Copy link
Contributor

This is in the conventions where we describe reducing use of axioms.

There's a whole lot more background at #3199 (comment) and in some of the pages a click or two away from the text I'm adding, but the short summary is that whatever we want to do about these definitions/axioms in the future, we might as well describe what is going on now.

set.mm Outdated
and moving them up to the ~ ax-4 through ~ ax-9 section.
and moving them up to the ~ ax-4 through ~ ax-9 section.</p>
<p>For these purposes, ~ df-clel , ~ df-cleq ,
and ~ df-clab should be considered axioms rather than definitions.
Copy link
Contributor

Choose a reason for hiding this comment

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

df-clab is actually conservative

proof: wcel is only used by wel at the time of df-clab. df-clab is not ambiguous with wel, so wcel can be considered to have a piecewise definition with wel and df-clab (at the point of df-clab)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, I wasn't sure whether to include df-clab just because the definition checker can't check it, or omit it because I had no knowledge of an interaction with axiom usage.

I've updated this pull request to omit it.

Copy link
Contributor

@benjub benjub left a comment

Choose a reason for hiding this comment

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

This is indeed the kind of text we would have to write if we keep df-cleq and df-clel unchanged. Introducing particular cases adds complexity, so hopefully the changes mentioned in #3199 (comment) will make this not needed.

Copy link
Contributor

@wlammen wlammen left a comment

Choose a reason for hiding this comment

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

I think a rewrap should be applied to improve the alignment of the text.

This is in the conventions where we describe reducing use of axioms.
@jkingdon
Copy link
Contributor Author

I think a rewrap should be applied to improve the alignment of the text.

Ah. I ran the rewrap script but forgot that it doesn't touch anything inside HTML tags.

I have done a manual rewrap.

@wlammen
Copy link
Contributor

wlammen commented Aug 13, 2023

Could you then push the result?

@jkingdon
Copy link
Contributor Author

Could you then push the result?

I pushed up everything I did.

The text I added is pretty well wrapped, isn't it?

In general, I think we should mostly limit our concern for style and formatting to what we can check automatically.

@GinoGiotto
Copy link
Contributor

Feel free to close #3199 after this is merged.

@jkingdon
Copy link
Contributor Author

Whether this text ends up being here for a short time or for longer depends on whether we make other changes to df-clel and df-cleq, but that doesn't seem like a reason not to merge it.

@jkingdon jkingdon merged commit d765262 into metamath:develop Aug 14, 2023
@jkingdon jkingdon deleted the conventions-clel branch August 14, 2023 12:29
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.

6 participants