Closed IainNZ closed 10 years ago
Not caused by changes to Grid
(we had no commits between May 23 and June 27) so I suspect this has to do with something in core Julia. I'll take a look and see if I can figure out what the problem is.
Edit: I think I found the problem, although I haven't located the change that broke it.
On line 294, which the error refers to, we define a = zeros(5,5)
. Or so we think - but a
was already used on line 255 to hold a = zeros(5)
. Changing the variable name should fix this.
By the way, is there a reason PkgEval didn't pick this up and file an issue automatically?
Re: why no auto-issue, same question occurred to me. I'm really not sure, probably an error on my part (its not fully automatic, I have to approve it, perhaps I forgot)
@IainNZ Could it be that PkgEval only tests tagged versions? I don't think we've tagged anything since May either.
Well, I started running the test suite and switching out variable names, but there were quite a lot of other ones that didn't work out either. Lots of variables in the test script are named things like A
, a
, x
etc, and re-used with no concerns about type stability. I think the best way out might be to encapsulate parts of the script in various functions and then call them all at the end.
Let's just make sure we don't tag a new version until this is fixed.
That test error is occuring on the tagged version - it simply uses Pkg.add(). People have been messing with this stuff in Base so I'm not too suprised something broke.
Oh, rats; I fixed this in db7cad0d75f7df4cdd2011c5eea3b10dff3fc8a1 but forgot to tag a new version. Fixed.
@IainNZ, this is part of what I was meaning when at JuliaCon I suggested that it would be great to re-run failing tests that same day to increase your confidence that the failures are real. I know if I were relying on my own brain to remember "failed today, let's see what happens tomorrow and if so I'll approve that email," I would forget 99% of the time. But I'm sure you're much better about this kind of thing than me :smile:.
Hah I was definitely thinking of that conversation when I noticed this!