experiment: fix various eta issues#12636
experiment: fix various eta issues#12636arthur-adjedj wants to merge 21 commits intoleanprover:masterfrom
Conversation
1a20ba7 to
efa54a6
Compare
|
Reference manual CI status:
|
|
Mathlib CI status (docs):
|
|
Mathlib CI status (docs):
|
what's a good commit tree worth anyway
|
Mathlib CI status (docs):
|
|
!bench |
|
Benchmark results for 063264c against 527a07b are in! @arthur-adjedj
No significant changes detected. |
|
!bench |
|
Benchmark results for 3411e8c against 527a07b are in! @arthur-adjedj
Large changes (2🟥)
Medium changes (10🟥)
Small changes (151🟥) Too many entries to display here. View the full report on radar instead. |
adabe5f to
df736ce
Compare
|
Mathlib CI status (docs):
|
|
!bench |
|
Benchmark results for 2b6be12 against 527a07b are in! @arthur-adjedj Warning These warnings may indicate that the benchmark results are not directly comparable, for example due to changes in the runner configuration or hardware.
Large changes (2🟥)
Medium changes (9🟥)
Small changes (156🟥) Too many entries to display here. View the full report on radar instead. |
|
!bench |
|
Benchmark results for 5422814 against 527a07b are in! @arthur-adjedj Warning These warnings may indicate that the benchmark results are not directly comparable, for example due to changes in the runner configuration or hardware.
Large changes (2🟥)
Medium changes (5🟥)
Small changes (88🟥) Too many entries to display here. View the full report on radar instead. |
|
!bench |
|
Benchmark results for 467c610 against 527a07b are in! @arthur-adjedj Warning These warnings may indicate that the benchmark results are not directly comparable, for example due to changes in the runner configuration or hardware.
Large changes (1🟥)
Medium changes (1✅, 1🟥)
Small changes (48🟥) Too many entries to display here. View the full report on radar instead. |
|
!bench |
|
Benchmark results for b586ad1 against 527a07b are in! @arthur-adjedj Warning These warnings may indicate that the benchmark results are not directly comparable, for example due to changes in the runner configuration or hardware.
Large changes (1🟥)
Medium changes (1✅, 1🟥)
Small changes (49🟥) Too many entries to display here. View the full report on radar instead. |
|
!bench |
|
Benchmark results for 4c08fdd against 527a07b are in! @arthur-adjedj Warning These warnings may indicate that the benchmark results are not directly comparable, for example due to changes in the runner configuration or hardware.
Large changes (1🟥)
Medium changes (2🟥)
Small changes (78🟥) Too many entries to display here. View the full report on radar instead. |
4c08fdd to
03a20c4
Compare
This PR is an experiment in fixing various issues defeq issues related to eta-expansions, and should not be merged. The fixes are minimal in that they do not try to alter much of the codebase, and should incur some performance penalty. Some of them could be alleviated through various refactors. One goal of this draft is to measure the perf impact of the most naive approach to those fixes.
Closes #2258, #3213, #12520 and, more unexpectedly, #2237.
The fixes are as follows:
For #2258 : Expand the unit-like criteria to handle Pi types whose codomain is unit-like, and structures whose fields are all unit-like or proof-irrelevant.
For #3213: Expand the condition for which the major of a recursor can be eta-expanded. Namely, subsingleton structures such as
Andshould be eta-expanded in this situation.For #12520:
isDefEqEta, eta-expand partial recursor applications when the major is missing and is an eta-expandable structure.isDefEqEtaStruct, eta-expand partial constructor applications.