-
Notifications
You must be signed in to change notification settings - Fork 15
Update LAMMPS pre-configure hook to disable SIMD for ARM + CUDA #87
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
Conversation
|
testing new hook in #88 |
Signed-off-by: laraPPr <[email protected]>
I guess you're doing this to try and avoid that the PR is merged with easystack file included? For future cases: I think it's better to temporarily add an easystack in this PR, test, then remove and rebuild to only get tarball for updated hooks? |
|
Yes I like to be carefull. I have still have not made a proposal for the policy on how we will handle this. It is somewhere on my todo. |
Signed-off-by: laraPPr <[email protected]>
Signed-off-by: laraPPr <[email protected]>
Signed-off-by: laraPPr <[email protected]>
Signed-off-by: laraPPr <[email protected]>
Signed-off-by: laraPPr <[email protected]>
Signed-off-by: laraPPr <[email protected]>
Signed-off-by: laraPPr <[email protected]>
Signed-off-by: laraPPr <[email protected]>
Signed-off-by: laraPPr <[email protected]>
|
Will need to be build into the easyblock because of how the block handles generic builds. |
Signed-off-by: laraPPr <[email protected]>
Signed-off-by: laraPPr <[email protected]>
Signed-off-by: laraPPr <[email protected]>
Signed-off-by: laraPPr <[email protected]>
Signed-off-by: laraPPr <[email protected]>
ocaisa
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.
This looks good to me, I'm going to merge it and then it will get heavily tested in the software layer PR
|
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws for:arch=x86_64/amd/zen2 |
|
New job on instance
|
|
New job on instance
|
No description provided.