Zulip Chat Archive

Stream: mathlib4

Topic: !4#3552 (LinearAlgebra.Matrix.ToLin)


Scott Morrison (Apr 27 2023 at 04:40):

I have !4#3552 (LinearAlgebra.Matrix.ToLin) working.

I found the before we started using etaExperiment, people were often hacking around the issue by adding (or subtracting!) various instances as needed. These instances were then accumulating, and leading to typeclass search timeouts.

Thus I've done a lot of cleanup: removing hacks adding or removing instances, and more uniformly just using etaExperiment.

To ease review, I've separated out those changes as !4#3668.

Scott Morrison (Apr 27 2023 at 04:40):

ToLin is on the critical path, so it would be great if these could be speedily reviewed. :-)

Eric Wieser (Apr 27 2023 at 08:15):

I was suspicious that adding those instance hacks might make the problem worse, thanks for confirming that suspicion!

Scott Morrison (Apr 27 2023 at 08:42):

It turns out !4#3668 was more complicated than I at first thought (I fixed ToLin, but broke other files that CI picked up). Hopefully ready for review again soon (TM).

Scott Morrison (Apr 27 2023 at 08:44):

Fortunately the instance hacks are all well signposted with porting notes, so easy to track down and exterminate. :-)

Scott Morrison (Apr 27 2023 at 11:03):

Okay, the etaExperiment cleanup branch !4#3668 is finally compiling again.


Last updated: Dec 20 2023 at 11:08 UTC