search

blas-lapack-rs / lapack-src

LAPACK source of choice
Other
12 stars 10 forks source link

Update intel-mkl-src 0.4.0 #3

Closed termoshtt closed 5 years ago

termoshtt commented 5 years ago
waywardmonkeys commented 5 years ago

Any chance this could land and have a new release cut, @IvanUkhov ?

IvanUkhov commented 5 years ago

Sorry for the delay!

waywardmonkeys commented 5 years ago

Thanks! Have a PR now into nalgebra-lapack to update to the newly published version.

IvanUkhov commented 5 years ago

@termoshtt, you actually have write access to all -src repositories in this organization, and now you should also be able publish blas-src and lapack-src on crates.io on your own. Please feel free to make changes as you see fit, as I might forget and unintentionally cause delays.

IvanUkhov commented 5 years ago

Any idea why intel-mkl fails on macOS?

https://travis-ci.org/blas-lapack-rs/blas-src/builds/542312996

https://travis-ci.org/blas-lapack-rs/lapack-src/builds/542312616

Should this pull request be reverted?

astraw commented 5 years ago

How is Travis caching?

The errors like 

error[E0277]: the trait bound `rand_hc::Hc128Rng: rand_core::SeedableRng` is not satisfied

look like perhaps something going wrong with rand and its dependencies. I don't see how Intel MKL could have anything to do with these errors. Perhaps putting a cargo update into the .travis.yml will fix this?

IvanUkhov commented 5 years ago

I’ve cleared the cache and restarted the build. Two out of six have already passed.

IvanUkhov commented 5 years ago

All green. Thanks for the tip!

termoshtt commented 5 years ago

Please feel free to make changes as you see fit, as I might forget and unintentionally cause delays.

I see. I will do next time, thanks.