Skip to content

Update intel-mkl-src 0.4.0 #3

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

Merged
merged 1 commit into from
Jun 6, 2019
Merged

Conversation

termoshtt
Copy link
Member

  • Windows support
  • pkg-config support

termoshtt added a commit to rust-ndarray/ndarray-linalg that referenced this pull request May 10, 2019
To bypass the blas-src/lapack-src until their PR are merged

- blas-lapack-rs/blas-src#3
- blas-lapack-rs/lapack-src#3
@waywardmonkeys
Copy link

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

@IvanUkhov
Copy link
Member

Sorry for the delay!

@IvanUkhov IvanUkhov merged commit 0574860 into blas-lapack-rs:master Jun 6, 2019
@waywardmonkeys
Copy link

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

@IvanUkhov
Copy link
Member

@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
Copy link
Member

IvanUkhov commented Jun 9, 2019

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
Copy link
Member

astraw commented Jun 9, 2019

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
Copy link
Member

IvanUkhov commented Jun 9, 2019

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

@IvanUkhov
Copy link
Member

All green. Thanks for the tip!

@termoshtt
Copy link
Member Author

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.

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

Successfully merging this pull request may close these issues.

4 participants