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

{lib}[GCCcore/10.3.0] Mesa v21.1.1, LLVM v11.1.0, Mako v1.1.4, ... w/ Python 3.9.5 #12943

Merged
merged 2 commits into from
May 28, 2021

Conversation

boegel
Copy link
Member

@boegel boegel commented May 24, 2021

(created using eb --new-pr)
requires #12939 (X11)

…e-10.3.0.eb, Mako-1.1.4-GCCcore-10.3.0.eb, libdrm-2.4.106-GCCcore-10.3.0.eb, libglvnd-1.3.3-GCCcore-10.3.0.eb, libunwind-1.4.0-GCCcore-10.3.0.eb
@boegel boegel added the update label May 24, 2021
@boegel
Copy link
Member Author

boegel commented May 24, 2021

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

@boegel boegel added this to the 4.4.0 milestone May 24, 2021
@boegel
Copy link
Member Author

boegel commented May 24, 2021

@boegelbot please test @ generoso

@boegelbot
Copy link
Collaborator

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

PR test command 'EB_PR=12943 EB_ARGS= /apps/slurm/default/bin/sbatch --job-name test_PR_12943 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

- notification for comment with ID 847268139 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).

Copy link
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

Do we consider holding back the LLVM version a bit for better compatibility with e.g. numba (I don't see that we use LLVM for anything else really)

@boegel
Copy link
Member Author

boegel commented May 24, 2021

Do we consider holding back the LLVM version a bit for better compatibility with e.g. numba (I don't see that we use LLVM for anything else really)

We would need to use LLVM 10.x then (or even 9.x on aarch64, cfr. https://github.com/numba/llvmlite#compatibility), which is a downgrade compared to 2020b...

Someone in Slack was recently looking into statically linking LLVM into llvmlite, which seems like a more promising approach since that makes LLVM just a build dependency for llvmlite+numba?

@boegel
Copy link
Member Author

boegel commented May 24, 2021

It looks like @fizwit got numba to work on top of LLVM 11.0.1 though, see https://github.com/FredHutch/easybuild-life-sciences/blob/master/fh_easyconfigs/n/numba/numba-0.53.1-fosscuda-2020b.eb ...

@fizwit
Copy link
Contributor

fizwit commented May 24, 2021 via email

@boegel
Copy link
Member Author

boegel commented May 24, 2021

Test report by @boegel
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in total)
easybuild2.novalocal - Linux centos linux 8.3.2011, POWER, IBM pSeries (emulated by qemu) (power9le), Python 3.6.8
See https://gist.github.com/edf33f2740ee5a4d0c13ff8a6c02638f for a full test report.

@boegel
Copy link
Member Author

boegel commented May 24, 2021

Test report by @boegel
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in total)
select-pika-c6gd-2xlarge-0001 - Linux centos linux 8.3.2011, AArch64, ARM UNKNOWN (graviton2), Python 3.6.8
See https://gist.github.com/a71dfd1ff4bbdfc01143f768cc46c999 for a full test report.

@boegel
Copy link
Member Author

boegel commented May 25, 2021

Note that using LLVM 11.1.0 here is (deliberately) conservative, since LLVM 12.0.0 is available already too...

@fizwit Did you run any test suites for llvmlite + numba on top of LLVM 11.x?
It seems like the patch may not be enough, based on the feedback in numba/llvmlite#639 ...
(see also numba/llvmlite#715)

@easybuilders easybuilders deleted a comment from boegelbot May 25, 2021
@Micket
Copy link
Contributor

Micket commented May 25, 2021

I should mention that llvmlite has a tracking issue for llvm 12 support already as well, so maybe they plan to skip 11

@fizwit
Copy link
Contributor

fizwit commented May 25, 2021 via email

@verdurin
Copy link
Member

Test report by @verdurin
SUCCESS
Build succeeded for 9 out of 9 (6 easyconfigs in total)
easybuild-c7.novalocal - Linux centos linux 7.9.2009, x86_64, Intel Xeon Processor (Skylake, IBRS), Python 3.6.8
See https://gist.github.com/265b831fd2ffd3577636368bd2d73db6 for a full test report.

Copy link
Member

@verdurin verdurin left a comment

Choose a reason for hiding this comment

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

Looks fine.

@verdurin
Copy link
Member

Going in, thanks @boegel!

@verdurin verdurin merged commit 06bd207 into easybuilders:develop May 28, 2021
@boegel boegel deleted the 20210524190740_new_pr_Mesa2111 branch May 28, 2021 20:43
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.

5 participants