Skip to content

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

Merged
smoors merged 4 commits intoeasybuilders:developfrom
boegel:fix_numexpr
Feb 28, 2023
Merged

fix linking numexpr with Intel MKL's VML library for imkl >= 2021.x#2897
smoors merged 4 commits intoeasybuilders:developfrom
boegel:fix_numexpr

Conversation

@boegel
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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