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.

Sign up for GitHub

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

Jump to bottom

use mixin class for cp2k #193

Open

smoors wants to merge 3 commits into EESSI:main from smoors:cp2k_mixin

Collaborator

smoors commented Oct 12, 2024 •

edited

Loading

additional improvements to the mixin class:

add hook set_tag_ci() to the mixin class, which is used if both bench_name_ci and bench_name are set. we could also make this a mandatory attribute, to be discussed.
make measure_memory_usage a reframe variable that can be set on the cmd line with reframe -S measure_memory_usage=True

cp2k.py file reduced from 114 lines to 76 \o/

Samuel Moors added 3 commits

October 12, 2024 15:06


          use mixin class for cp2k

7557de7


          remove unused imports

bb83565


          make measure_memory_usage a reframe variable

20631a4

smoors mentioned this pull request

use mixin class for gromacs #194

Open

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

Labels

None yet