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 custom easyblock for CP2K >= v2024 #3433

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

beeebiii
Copy link

@beeebiii beeebiii commented Sep 4, 2024

Hi, as discussed in Slack, and being based in in these three PRs:
easybuilders/easybuild-easyconfigs#17718
#2349
#2918
I have written easyblock for CP2K, during the compilation of **CP2K-2024.2-foss-2023a-CUDA-12.2.0.eb **. Almost all the tests pass except for around 50 tests of total 4000 tests fail.

@boegel boegel changed the title This is the cp2k.py(Easyblock) that i have written being based on PR(2918) update custom easyblock for CP2K >= v2024 Sep 11, 2024
@boegel boegel added the update label Sep 11, 2024
Copy link

@gkaf89 gkaf89 left a comment

Choose a reason for hiding this comment

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

Modification relevant, fix CUDA builds. Tested with CP2K 2024.3 with and without CUDA (12.1.1 with gpuver=V100).

if cp2k_version >= LooseVersion('2024'):
options['DFLAGS'] += ' -D__OFFLOAD_CUDA -D__DBCSR_ACC '
options['LIBS'] += ' -lcufft -lcudart -lnvrtc -lcuda -lcublas'
options['OFFLOAD_CC'] = 'nvcc'
Copy link

@gkaf89 gkaf89 Nov 20, 2024

Choose a reason for hiding this comment

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

The OFFLOAD_CC variable is used in the makefile of CP2K to build files with OpenACC offloading, even in versions older than 2024. For instance 2023.1 fails to build with CUDA support with the old easyblock.

Maybe enable the modification for some versions earlier than 2024 if they are still relevant?

Copy link
Author

@beeebiii beeebiii Nov 21, 2024

Choose a reason for hiding this comment

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

Yeah it can surely be modified, but i have not tested them with old versions, when i test them will definitely modify if required. Thanks!!

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.

3 participants