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.

Sign up for GitHub

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

Jump to bottom

fix linking numexpr with Intel MKL's VML library for imkl >= 2021.x #2897

Merged
merged 4 commits into from
Feb 28, 2023
Merged

fix linking numexpr with Intel MKL's VML library for imkl >= 2021.x #2897

merged 4 commits into from
Feb 28, 2023

Conversation

boegel
Copy link
Member

@boegel boegel commented Feb 25, 2023 •
edited
Loading

(created using eb --new-pr)

WIP, because this currently only changes the enhanced sanity check catches the problem:

$ python -c "import numexpr; numexpr.set_vml_accuracy_mode('low')"
INTEL MKL ERROR: /software/imkl/2022.1.0/mkl/2022.1.0/lib/intel64/libmkl_vml_def.so.2: undefined symbol: mkl_lapack_dspevd.
        Intel MKL FATAL ERROR: cannot load libmkl_vml_def.so.2.)

@boegel boegel marked this pull request as draft February 25, 2023 18:22
@boegel boegel added the bug fix label Feb 25, 2023
@boegel boegel added this to the next release (4.7.1?) milestone Feb 25, 2023
@boegel
Copy link
Member Author

boegel commented Feb 25, 2023

Same problem as in https://bugs.gentoo.org/828021 (but no solution there either, so far)

@boegel
Copy link
Member Author

boegel commented Feb 25, 2023

This error is also mentioned at https://intel.github.io/intel-extension-for-pytorch/xpu/1.13.10+xpu/tutorials/performance_tuning/known_issues.html

@boegel boegel marked this pull request as ready for review February 25, 2023 22:02
@boegel
Copy link
Member Author

boegel commented Feb 26, 2023

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS SciPy-bundle-2022.05-intel-2022a.eb
  • SUCCESS SciPy-bundle-2021.10-intel-2021b.eb
  • SUCCESS SciPy-bundle-2021.05-intel-2021a.eb
  • SUCCESS SciPy-bundle-2020.11-intel-2020b.eb
  • SUCCESS SciPy-bundle-2022.05-foss-2022a.eb
  • SUCCESS gomkl-2021a.eb
  • SUCCESS SciPy-bundle-2021.05-gomkl-2021a.eb

Build succeeded for 7 out of 7 (6 easyconfigs in total)
node3121.skitty.os - Linux RHEL 8.6, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/8487addbc64b3693a133c73c4122b248 for a full test report.

This was referenced Feb 28, 2023
Closed
Merged
smoors
smoors approved these changes Feb 28, 2023
View reviewed changes
Copy link
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@smoors
Copy link
Contributor

smoors commented Feb 28, 2023

Going in, thanks @boegel!

@smoors smoors merged commit e51c1bd into easybuilders:develop Feb 28, 2023
@boegel boegel deleted the fix_numexpr branch February 28, 2023 20:18
@boegel boegel mentioned this pull request Mar 14, 2023
Merged
@casparvl casparvl mentioned this pull request Mar 15, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smoors smoors smoors approved these changes

Assignees
No one assigned
Labels
bug fix
Projects
None yet
Milestone
4.7.1
Development

Successfully merging this pull request may close these issues.
2 participants
@boegel @smoors