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

CI errors on master in LinearAlgebra.mul! #32924

Closed
fredrikekre opened this issue Aug 16, 2019 · 4 comments · Fixed by #33843
Closed

CI errors on master in LinearAlgebra.mul! #32924

fredrikekre opened this issue Aug 16, 2019 · 4 comments · Fixed by #33843
Labels
linear algebra Linear algebra test This change adds or pertains to unit tests
Milestone

Comments

@fredrikekre
Copy link
Member

Probably from the mul! PR, cc @tkf

Error in testset LinearAlgebra/addmul:
Test Failed at /buildworker/worker/tester_linux32/build/share/julia/stdlib/v1.3/LinearAlgebra/test/addmul.jl:178
  Expression: ���(collect(returned_mat), collect(�� * A * B + �� * Cc), rtol=rtol)
   Evaluated: [0.6384605169296265] ��� [0.5298598366189116] (rtol=0.0003452669770922512) 

See e.g. https://build.julialang.org/#/builders/13/builds/4527

@fredrikekre fredrikekre added the linear algebra Linear algebra label Aug 16, 2019
@fredrikekre fredrikekre added this to the 1.3 milestone Aug 16, 2019
@fredrikekre fredrikekre changed the title CI errors on mater in LinearAlgebra.mul! CI errors on master in LinearAlgebra.mul! Aug 16, 2019
@tkf
Copy link
Member

tkf commented Aug 16, 2019

There are some bug fixes in #32901.

By the way, I realized that the test log is not very helpful since @testset titles are not rendered in the CI. Is there a way to get a similar output as in Pkg.test?

@JeffBezanson JeffBezanson added the test This change adds or pertains to unit tests label Aug 16, 2019
@JeffBezanson
Copy link
Sponsor Member

Hopefully fixed by #32901; can reopen if not.

@vtjnash
Copy link
Sponsor Member

vtjnash commented Nov 13, 2019

Still seems to be happening (or at least, I assume related), e.g. https://build.julialang.org/#/builders/29/builds/5947/steps/2/logs/stdio

@tkf
Copy link
Member

tkf commented Nov 13, 2019

This is probably due to #33743. cc @dkarrasch

Just FYI, to debug this locally (without #32938), comment out this line

# n_samples = Inf # to try all combinations

and run the file. IIRC, it takes some hours to finish with --compile=min if there is no failure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
linear algebra Linear algebra test This change adds or pertains to unit tests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants