-
Notifications
You must be signed in to change notification settings - Fork 773
{phys}[foss/2023a] HOOMD-blue v4.9.1 #22013
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
{phys}[foss/2023a] HOOMD-blue v4.9.1 #22013
Conversation
…-blue-4.9.1.patch
Updated software
|
|
@boegelbot: please test @ generoso |
|
@sassy-crick: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 2537207234 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
@boegelbot: please test @ generoso |
|
@sassy-crick: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 2537393806 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Something seems to be odd here: the test jobs are all working yet the |
| ('Eigen', '3.4.0'), | ||
| ('Cereal', '1.3.2', '', SYSTEM), | ||
| ('LLVM', '16.0.6'), | ||
| ('Clang', '16.0.6'), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clang dependency will no longer be required with an LLVM built with easybuilders/easybuild-easyblocks#3373
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That might be true for newer versions but I am stuck with the 16.0.6 version and I need Clang. So in the end, the way forward was to use the Clang included LLVM but build Clang with shared libraries. Then it was very easy.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And just to add: I actually just have tried it, rebuild LLVM-16.0.6 with the EasyBlock you mentioned, removed Clang and that configuration fails right at the beginning.
So I guess here we are stuck with the PR as it is, unless somebody shows me a better way of doing it. :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've tried quite a lot of things and it simply isn't possible to build HOOMD-blue with the existing statical libarary Clang 16.0.6
Once the _jit...so python library is linked with all libraries it needs it fails with
== FAILED: Installation ended unsuccessfully: Sanity check failed: sanity check command python -c 'import hoomd' failed with exit code -6 (output: : CommandLine Error: Option 'scalarize-variable-insert-extract' registered more than once!
That comes from LLVM somewhere, and it just isn't possible to get rid of with that build. It do need a shared library Clang build.
…d with shared libs
|
Test report by @sassy-crick |
…asyconfigs into 20241211204258_new_pr_HOOMD-blue491
…asyconfigs into 20241211204258_new_pr_HOOMD-blue491
|
@boegelbot please test @ jsc-zen3 |
|
@Crivella: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3023565454 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @Crivella |
|
For the CI we might need an exception in easybuild-easyconfigs/test/easyconfigs/easyconfigs.py Lines 362 to 365 in 1853c80
|
|
Test report by @boegelbot |
…asyconfigs into 20241211204258_new_pr_HOOMD-blue491
Crivella
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Going in, thanks @sassy-crick! |
(created using
eb --new-pr)See also here for some of the encountered problems.
Requires: