Skip to content

update PETSc easyblock to take into account that ScaLAPACK installation may not have header files + fix building in parallel - #2348

Merged
akesandgren merged 1 commit into
easybuilders:developfrom
boegel:petsc_fixes
Feb 26, 2021
Merged

update PETSc easyblock to take into account that ScaLAPACK installation may not have header files + fix building in parallel#2348
akesandgren merged 1 commit into
easybuilders:developfrom
boegel:petsc_fixes

Conversation

@boegel

@boegel boegel commented Feb 24, 2021

Copy link
Copy Markdown
Member

ScaLAPACK/2.1.0-gompi-2020b does not provide any header files, because the installation was done via CMake.

Apparently PETSc doesn't require the ScaLAPACK header files at all, so there's no point in requiring them.

In addition, building in parallel requires passing MAKE_NP=... to the make command as an argument, just defining the $MAKE_NP doesn't make any difference...

…on may not have header files + fix building in parallel
@boegel

boegel commented Feb 25, 2021

Copy link
Copy Markdown
Member Author

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS PETSc-3.12.4-foss-2020a-Python-3.8.2.eb
  • SUCCESS PETSc-3.12.4-intel-2020a-Python-3.8.2.eb

Build succeeded for 2 out of 2 (2 easyconfigs in total)
node3152.skitty.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 3.6.8
See https://gist.github.com/cec1ca24dda716a17113cea2626e10fc for a full test report.

@boegel

boegel commented Feb 25, 2021

Copy link
Copy Markdown
Member Author

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS PETSc-3.11.0-foss-2018b-downloaded-deps.eb
  • SUCCESS PETSc-3.12.4-foss-2019b-Python-3.7.4.eb
  • SUCCESS PETSc-3.12.4-foss-2020a-Python-3.8.2.eb
  • SUCCESS PETSc-3.12.4-intel-2019b-Python-2.7.16.eb
  • SUCCESS PETSc-3.12.4-intel-2019b-Python-3.7.4.eb
  • SUCCESS PETSc-3.12.4-intel-2020a-Python-3.8.2.eb
  • SUCCESS PETSc-3.9.3-foss-2018a.eb

Build succeeded for 7 out of 7 (7 easyconfigs in total)
node2685.swalot.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/f64c3d0f200ee1848f99a78217a9f1a9 for a full test report.

@boegel
boegel requested a review from akesandgren February 25, 2021 10:35

@akesandgren akesandgren left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@akesandgren

Copy link
Copy Markdown
Contributor

Going in, thanks @boegel!

@akesandgren
akesandgren merged commit 9d86862 into easybuilders:develop Feb 26, 2021
@boegel
boegel deleted the petsc_fixes branch February 26, 2021 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants