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

rename identity theorems #4674

Merged
merged 9 commits into from
Feb 26, 2025
Merged

rename identity theorems #4674

merged 9 commits into from
Feb 26, 2025

Conversation

sctfn
Copy link
Contributor

@sctfn sctfn commented Feb 24, 2025

Per #4666 this PR renames all the id1 and id2 theorems remaining in set.mm to rid and lid respectively.

@avekens
Copy link
Contributor

avekens commented Feb 24, 2025

Per #4666 this PR renames all the id1 and id2 theorems remaining in set.mm to rid and lid respectively.

Thank you very much for this janitorial work. We should perform clean ups as soon as possible after they are detected, otherwise they won't be done in most cases.

Copy link
Contributor

@icecream17 icecream17 left a comment

Choose a reason for hiding this comment

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

halfway finished reviewing

set.mm Outdated
@@ -129303,13 +129303,13 @@ it could represent the (meaningless) operation of
$( Subtraction of a number from itself. (Contributed by NM, 8-Oct-1999.)
(Revised by Mario Carneiro, 27-May-2016.) $)
subid $p |- ( A e. CC -> ( A - A ) = 0 ) $=
( cc wcel cc0 caddc co cmin addid1 oveq1d wceq 0cn pncan2 mpan2 eqtr3d ) AB
( cc wcel cc0 caddc co cmin addrid oveq1d wceq 0cn pncan2 mpan2 eqtr3d ) AB
CZADEFZAGFZAAGFDOPAAGAHIODBCQDJKADLMN $.

$( Identity law for subtraction. (Contributed by NM, 9-May-2004.) (Revised
by Mario Carneiro, 27-May-2016.) $)
subid1 $p |- ( A e. CC -> ( A - 0 ) = A ) $=
Copy link
Contributor

Choose a reason for hiding this comment

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

This could be renamed too
metamath.exe show statement *id1,*id2 can check for possible missed theorems

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There are 73 other theorems of the form *id[12] in set.mm, not all of which fit this pattern. This PR would absolutely balloon out of control if I were to rename all of them here. Could somebody else please go through some of them?

Copy link
Contributor

@jkingdon jkingdon left a comment

Choose a reason for hiding this comment

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

OK, I checked them all and found four more errors and the rest correct.

Copy link
Contributor

@avekens avekens left a comment

Choose a reason for hiding this comment

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

  • one blocking review remark (resubidaddrid => resubidaddlid)
  • one nonblocking review remark (mistake in comment)

@sctfn sctfn merged commit 4165aac into develop Feb 26, 2025
20 checks passed
@sctfn sctfn deleted the rename-ids branch February 26, 2025 15:52
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.

4 participants