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

Test error on 0.4 release branch #15733

Closed
eschnett opened this issue Apr 1, 2016 · 6 comments
Closed

Test error on 0.4 release branch #15733

eschnett opened this issue Apr 1, 2016 · 6 comments
Labels
bug Indicates an unexpected problem or unintended behavior priority This should be addressed urgently

Comments

@eschnett
Copy link
Contributor

eschnett commented Apr 1, 2016

After updating on the 0.4 release branch, I see this error in make -j8testall on Darwin:

Exception running test linalg/svd :
On worker 7:
LoadError: MethodError: `getindex` has no method matching getindex(::Base.RefValue{Int64}, ::Int64)
Closest candidates are:
  getindex(::Base.RefValue{T})
 in ggsvd3! at linalg/lapack.jl:1733
 in svdfact! at linalg/svd.jl:84
 in svdfact at linalg/svd.jl:88
 [inlined code] from /Users/eschnett/src/julia/test/linalg/svd.jl:49
 in anonymous at no file:0
 in include_string at loading.jl:282
 in include_from_node1 at ./loading.jl:323
 [inlined code] from util.jl:179
 in runtests at /Users/eschnett/src/julia/test/testdefs.jl:7
 in anonymous at /Users/eschnett/src/julia/test/runtests.jl:36
 in anonymous at multi.jl:923
 in run_work_thunk at multi.jl:661
 [inlined code] from multi.jl:923
 in anonymous at task.jl:63
while loading /Users/eschnett/src/julia/test/linalg/svd.jl, in expression starting on line 21
ERROR: LoadError: Some tests exited with errors.
 in anonymous at /Users/eschnett/src/julia/test/runtests.jl:64
 in cd at ./file.jl:22
 in include at ./boot.jl:261
 in include_from_node1 at ./loading.jl:320
 in process_options at ./client.jl:280
 in _start at ./client.jl:378
while loading /Users/eschnett/src/julia/test/runtests.jl, in expression starting on line 13
make[1]: *** [all] Error 1
make: *** [testall] Error 2
@yuyichao
Copy link
Contributor

yuyichao commented Apr 1, 2016

I believe this is due to #14444 and was fixed on master by #14522 . We can probably fix it on 0.4 with a smaller patch by replacing info[1] with info[].

@andreasnoack
Copy link
Member

Why haven't we caught this in CI?

@yuyichao
Copy link
Contributor

yuyichao commented Apr 1, 2016

Openblas version

@JeffBezanson JeffBezanson added bug Indicates an unexpected problem or unintended behavior priority This should be addressed urgently backport pending 0.4 labels Apr 12, 2016
@tkelman
Copy link
Contributor

tkelman commented Apr 15, 2016

Bump @nalimilan your backport was buggy and it'll need to be fixed before we can do 0.4.6. Can you take a look?

@nalimilan
Copy link
Member

Sorry for the slow reaction. #15883 fixes the failure with 0.2.16 in my tests (I wonder how I introduced it in the first place -- maybe I changed something right after testing).

@tkelman
Copy link
Contributor

tkelman commented Apr 15, 2016

Should be fixed, but wouldn't mind a confirmation from anyone who was able to reproduce this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior priority This should be addressed urgently
Projects
None yet
Development

No branches or pull requests

6 participants