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

misc + iota axiom saves #4492

Merged
merged 8 commits into from
Dec 29, 2024
Merged

misc + iota axiom saves #4492

merged 8 commits into from
Dec 29, 2024

Conversation

icecream17
Copy link
Contributor

Commit by commit (a lot more compartmentalized than my last pr)

About 40 to 50 miscallaneous downstream axiom saves are from the first few commits, and then iotaval/iotaex is responsible for the rest in the axiom diff.

@icecream17 icecream17 force-pushed the develop branch 2 times, most recently from 24b760d to d6f17c9 Compare December 23, 2024 23:17
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.

My suggestions are non-blocking.

The only thing in here which really caught my eye was the changes to iota related theorems. If someone who has thought more about iota than I has an opinion, speak up. If not, I think we can go with these changes, as they seem like improvements in terms of axiom usage.

set.mm Outdated Show resolved Hide resolved
set.mm Outdated Show resolved Hide resolved
set.mm Outdated Show resolved Hide resolved
@icecream17 icecream17 marked this pull request as draft December 27, 2024 19:38
@icecream17
Copy link
Contributor Author

icecream17 commented Dec 27, 2024

blocked until #4503 is merged

set.mm Outdated Show resolved Hide resolved
matches recent changes to ~ el and ~ dtru
rgenw + ss2rab + mpbir = a1i + ss2rabi
ralrimivw + ss2rab + sylibr = adantr + ss2rabdv

Ideally ss2rabi and ss2rabdv would be renamed with an "a" suffix,
then new ss2rabi and ss2rabdv would be made.

For now it is implemented manually but it is equivalent to an
automatic shortening.

mptexgf: a1i + ss2rabi
scottex: adantr + ss2rabdv
wwlksnfi: a1i + ss2rabi
disjxwwlkn: a1i + ss2rabi
occon: adantr + ss2rabdv
rmxyelqirr: (more extensive)
itgoss: adantr + ss2rabdv
ovnsslelem: adantr + ss2rabdv (+auto minimize)
ovolval5lem3: a1i + ss2rabi (+auto minimize)

----
Add ss2ab1 and ss2abdv to mathbox, shorten abssdv
dfopifOLD no longer uses ax-10 to 12 so it is restored
rebase: changes-set.txt, iotauni2 now uses unisnv
@icecream17 icecream17 marked this pull request as ready for review December 29, 2024 20:01
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.

Thanks @icecream17 for the careful updates !

@benjub benjub merged commit 2d3605b into metamath:develop Dec 29, 2024
10 checks passed
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.

3 participants