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 fails due to dynamic linking errors with Intel MKL during doctests. #58

Open
olson-sean-k opened this issue Jun 5, 2020 · 4 comments
Labels

Comments

@olson-sean-k
Copy link
Owner

eccebcc disables testing on the beta Rust toolchain due to dynamic linking errors when running doctests. The lapack feature from Theon uses Intel MKL on Linux and for some reason the library fails to load while executing doctests. See this issue.

CI should test against the beta toolchain. Once this issue is resolved, include the beta toolchain in the test matrix again (revert eccebcc).

@olson-sean-k
Copy link
Owner Author

This issues had graduated to the stable toolchain. See this failed CI run, for example. There is an upstream issue tracking this against ndarray-linalg. For now, I may restore CI against the beta toolchain. CI will be noisy, but that's probably okay; disabling CI against the beta toolchain was a debatable response in the first place. 😛

@olson-sean-k
Copy link
Owner Author

0237224 re-enables CI against the beta Rust toolchain.

@olson-sean-k olson-sean-k changed the title Re-enable CI testing on the beta Rust toolchain. CI fails due to dynamic linking errors with Intel MKL during doctests. Jul 25, 2020
@olson-sean-k
Copy link
Owner Author

I've renamed this issue so that it can be used to track ongoing CI failures with the context gathered thus far. I plan to close it once the upstream issue is closed and the fix propagates downstream to Theon and Plexus.

@olson-sean-k olson-sean-k pinned this issue Sep 14, 2020
@olson-sean-k
Copy link
Owner Author

It's unclear when a fix in ndarray-linalg will be released and I have seen dynamic linking errors when directly executing the example binaries. To reduce noise in CI and avoid propagating linking errors, 81a7468 disables LAPACK features (disables Theon's lapack feature and the FacePlane trait implementation). This means that FaceView::plane is no longer available, but I don't consider that an essential API. This should be temporary and these features will be re-enabled once a fix is available.

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

No branches or pull requests

1 participant