Zulip Chat Archive

Stream: mathlib4

Topic: LinearAlgebra.Matrix.ToLin !4#3552


Scott Morrison (Apr 21 2023 at 04:52):

LinearAlgebra.Matrix.ToLin !4#3552 is now error-free, but I am sad about the porting notes about simp not working in places rw does. If anyone would like to look at this (whether to decide to merge as is, or indicate that we should investigate further), that would be great!

Ruben Van de Velde (Apr 21 2023 at 05:35):

It would be really great to have someone debug the simp issue - it's bothered me in a bunch of places already. I'd prefer not to let this sit for too long if nobody picks it up, though

Ruben Van de Velde (Apr 21 2023 at 07:35):

@Scott Morrison but we'll have to wait until you push your fixes anyway :)

Scott Morrison (Apr 21 2023 at 07:35):

Ugh, I can't find them??

Ruben Van de Velde (Apr 21 2023 at 07:36):

If you committed them, git reflog should have them

Johan Commelin (Apr 21 2023 at 07:37):

You will need to know in which clone you committed them (-;

Eric Wieser (Apr 21 2023 at 07:42):

I'll take a look once the fixes are found

Scott Morrison (Apr 21 2023 at 07:45):

I am completely mystified. I think at this point I should just try again, and hope I remember how the fixes looked.

Scott Morrison (Apr 21 2023 at 08:02):

Now the file looks unfamiliar to me, and I have no idea how I came to post this message. I am seriously feeling as if the matrix glitched....?!? Sorry for the noise.

Reid Barton (Apr 22 2023 at 11:14):

Johan Commelin said:

You will need to know in which clone you committed them (-;

Are we talking about clones of mathlib, or clones of Scott?

Eric Wieser (Apr 26 2023 at 10:59):

I've pushed some more fixes to this PR

Eric Wieser (Apr 26 2023 at 11:00):

There's one really weird error where congr didn't try to apply congr_fun

Eric Wieser (Apr 26 2023 at 11:00):

And a bunch of timeouts near the bottom of the file


Last updated: Dec 20 2023 at 11:08 UTC