Skip to content

Commit

Permalink
Merge pull request #2022 from boegel/numexpr_extension
Browse files Browse the repository at this point in the history
fix numexpr easyblock to allow (correct) installing it as extension
  • Loading branch information
zao authored Apr 12, 2020
2 parents 3100bae + 6444279 commit faa67df
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions easybuild/easyblocks/n/numexpr.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,16 +52,12 @@ def __init__(self, *args, **kwargs):

self.imkl_root = None

def prepare_step(self, *args, **kwargs):
"""Prepare environment for building and installing numexpr."""
super(EB_numexpr, self).prepare_step(*args, **kwargs)

self.imkl_root = get_software_root('imkl')

def configure_step(self):
"""Custom configuration procedure for numexpr."""
super(EB_numexpr, self).configure_step()

self.imkl_root = get_software_root('imkl')

# if Intel MKL is available, set up site.cfg such that the right VML library is used;
# this makes a *big* difference in terms of performance;
# see also https://github.com/pydata/numexpr/blob/master/site.cfg.example
Expand Down Expand Up @@ -103,4 +99,4 @@ def sanity_check_step(self):
if self.imkl_root:
custom_commands.append("python -c 'import numexpr; assert(numexpr.use_vml)'")

super(EB_numexpr, self).sanity_check_step(custom_commands=custom_commands)
return super(EB_numexpr, self).sanity_check_step(custom_commands=custom_commands)

0 comments on commit faa67df

Please sign in to comment.