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

Update to OpenBLAS 0.3.18 as dependency for FlexiBLAS 3.0.4 (used in foss/2021.07) #14167

Conversation

SebastianAchilles
Copy link
Member

(created using eb --new-pr)

@SebastianAchilles
Copy link
Member Author

Test report by @SebastianAchilles
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
centos8-eb - Linux centos linux 8.3.2011, x86_64, Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz (skylake), Python 3.6.8
See https://gist.github.com/5b9952a3c9c4121c5df9593b57e0e225 for a full test report.

@boegel
Copy link
Member

boegel commented Oct 18, 2021

@SebastianAchilles Hmm, that doesn't exactly match with the toolchain versioning scheme anymore, since OpenBLAS 0.3.18 was released Oct 2nd...

@SebastianAchilles
Copy link
Member Author

@SebastianAchilles Hmm, that doesn't exactly match with the toolchain versioning scheme anymore, since OpenBLAS 0.3.18 was released Oct 2nd...

That is true, but the same argument applies for #14164 as libfabric v1.13.1 was released on Aug 25th.

Currently the EasyConfig for OpenBLAS v0.3.17 using GCC 11.2.0 contains a patch which is part of the v0.3.18 release (https://github.com/easybuilders/easybuild-easyconfigs/blob/develop/easybuild/easyconfigs/o/OpenBLAS/OpenBLAS-0.3.17-GCC-11.2.0.eb#L18). That is why I think it makes more sense to use OpenBLAS v0.3.18 in the upcoming foss/2021b toolchain.

@boegel
Copy link
Member

boegel commented Oct 18, 2021

@SebastianAchilles For foss/2021b, we should definitely use latest OpenBLAS, yes.

@boegel
Copy link
Member

boegel commented Oct 20, 2021

Maybe we should just promote foss/2021.07 in place to foss/2021b for EasyBuild 4.5.0?

If we'll go forward like this, merging this PR is fine imho...

@boegel boegel added the update label Oct 20, 2021
@boegel
Copy link
Member

boegel commented Oct 20, 2021

@boegelbot please test @ generoso

@boegelbot
Copy link
Collaborator

@boegel: Request for testing this PR well received on login1

PR test command 'EB_PR=14167 EB_ARGS= /opt/software/slurm/bin/sbatch --job-name test_PR_14167 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 7119

Test results coming soon (I hope)...

- notification for comment with ID 947927486 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegel
Copy link
Member

boegel commented Oct 20, 2021

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node2624.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/86a3d580495edb1432a53dab9eee2cd6 for a full test report.

@boegel
Copy link
Member

boegel commented Oct 20, 2021

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3511.doduo.os - Linux RHEL 8.2, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/e73785778d3890f5900c0400a0b0edcb for a full test report.

@boegel
Copy link
Member

boegel commented Oct 20, 2021

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3905.accelgor.os - Linux RHEL 8.4, x86_64, AMD EPYC 7413 24-Core Processor (zen2), 1 x NVIDIA NVIDIA A100-SXM-80GB, 470.57.02, Python 3.6.8
See https://gist.github.com/ed444a72c516d87f30bb7800905fcefa for a full test report.

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
cnx1 - Linux rocky linux 8.4, x86_64, Intel(R) Xeon(R) CPU E5-2690 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/e2ba5354c6d823e82bf4afc7de72ead1 for a full test report.

@boegel
Copy link
Member

boegel commented Oct 20, 2021

Going in, thanks @SebastianAchilles!

@boegel boegel merged commit b100aaf into easybuilders:develop Oct 20, 2021
@verdurin
Copy link
Member

Test report by @verdurin
SUCCESS
Build succeeded for 36 out of 36 (1 easyconfigs in total)
centos7.lan - Linux centos linux 7.9.2009, x86_64, Intel Core Processor (Skylake, IBRS), Python 3.6.8
See https://gist.github.com/42d674ecad45681669ea09ca1fd8f54e for a full test report.

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

Successfully merging this pull request may close these issues.

4 participants