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

RFC: Bump OpenBLAS to v0.2.13 #9277

Merged
merged 2 commits into from
Dec 10, 2014
Merged

RFC: Bump OpenBLAS to v0.2.13 #9277

merged 2 commits into from
Dec 10, 2014

Conversation

tkelman
Copy link
Contributor

@tkelman tkelman commented Dec 9, 2014

should fix #9084 and a few freebsd build issues found by @nolta, and gets rid of a patch that was upstreamed

should fix #9084, and gets rid of a patch that was upstreamed
currently the only values that will make much sense are "64_" or ""
due to arpack and suitesparse
@staticfloat
Copy link
Member

This multiple-status thing is the coolest thing ever.

@ViralBShah
Copy link
Member

Love the multiple status!

@ViralBShah
Copy link
Member

Can we get a different status line for linux and mac from travis?

@ivarne
Copy link
Member

ivarne commented Dec 9, 2014

Probably not before Travis decides to support that option.

@tkelman
Copy link
Contributor Author

tkelman commented Dec 9, 2014

Yes, I think Github did a really good job with this feature - https://github.com/blog/1935-see-results-from-all-pull-request-status-checks

I disabled the webhook for the temporary version that we had been using.

Annoyingly appveyor seems to be timing out especially frequently over the last few days. Some of the failures could possibly be mitigated by running the tests in serial for win64, trade off longer test running time for better reliability. But some runs are getting "broken pipe" errors, and others appear to just freeze for no reason as if the appveyor worker has died.

Regarding splitting out the Travis matrix into different status items, that would have to be done on Travis' side. If you can find where the Travis builder interacts with the Github API, they'd probably entertain a PR to implement an option for separate statuses per entry in the build matrix.

@StefanKarpinski
Copy link
Member

Since we're paying for AppVeyor, I would expect better service than this. Perhaps I should contact them?

@tkelman
Copy link
Contributor Author

tkelman commented Dec 9, 2014

Sure, maybe Feodor could look into some of the failed builds that weren't the fault of Julia. Let me skim through the list of recent failed builds to classify them.

@tkelman
Copy link
Contributor Author

tkelman commented Dec 9, 2014

Network failure (we can try mirroring these files to more reliable places):
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.357/job/e634e1d9lcu6vpwi
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.356/job/kspcel0u0csxh5t8
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.350
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.349
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.348

Inexplicable hang, probably not Julia's fault:
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.398/job/d4wq8y9p7xh1rr6b
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.395/job/t6t4fkfspkgyjwya
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.393/job/65rr1i2ah5v7q1d1
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.387/job/7klqkcfm6k2rw5am
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.385/job/li8m0orbiyoksjyj
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.369/job/0iwm3k5ae09xoo9l
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.363/job/ku9taem0p83dxn0s
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.354/job/xeoecbt1q0f7i91r
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.347/job/5xng282ebpb19l8y
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.336/job/luyv9dyvlf8q755e
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.333/job/4niko1kmu1gmugjc
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.328/job/hjpyjnyiotmup2eh
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.322/job/003q6mvux20kibad

Tests hang, our fault #7942:
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.337/job/74a2o3ib8mgqb1hi
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.330/job/r4qo4qd02c6o69l3
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.329/job/hhvckh589vf855f8
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.327/job/fs75tnw3ve2c7g8v
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.326/job/ka2mgg3p8ay79ssa
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.301/job/ar317i7fvnfnran2
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.300/job/r7tb3083jhnop9g2

Broken pipe, dunno what causes this:
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.325/job/0uok2lssiaqmankd
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.309/job/jsyavfylky6gv9y5

Mysterious git-bash error:
https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.307/job/251hue4hss4he6c9

Also it's kind of silly that https://ci.appveyor.com/project/StefanKarpinski/julia/history doesn't have a scroll bar/button to go more than 10 builds back.

@ViralBShah
Copy link
Member

I assume that bumping openblas here is ok. Perhaps we need a separate issue for Appveyor.

tkelman added a commit that referenced this pull request Dec 10, 2014
@tkelman tkelman merged commit de7339c into master Dec 10, 2014
@tkelman tkelman deleted the tk/openblas-0.2.13 branch December 10, 2014 07:12
@tkelman
Copy link
Contributor Author

tkelman commented Dec 10, 2014

Yeah I think it should be fine. If anything breaks with linking arpack or suitesparse with MKL, let me know. The lines I had here regarding MKL were actually redundant since MKL is always considered USE_SYSTEM_BLAS = 1.

Now you should also be able to explicitly disable symbol renaming if you know you don't want it by setting override OPENBLAS_SYMBOLSUFFIX = "" in your Make.user.

@ViralBShah
Copy link
Member

That is handy. Thanks.

@tkelman
Copy link
Contributor Author

tkelman commented Dec 14, 2014

backported in 11eee39 and b1ed699

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Segfault in svd
5 participants