-
Notifications
You must be signed in to change notification settings - Fork 19
Test lammps hook #186
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
Test lammps hook #186
Changes from 11 commits
78d6a88
6041094
1f7dce2
b3bc87f
db5b854
0fc7eb8
c595f7c
83d415f
e80b897
13ddf4e
a45ad67
87596c8
8bd8c41
a76c78d
f8541da
ac865ed
31eb3c3
5ad22cb
7b211de
773613b
2ff1929
d3d0eaa
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
| easyconfigs: | ||
| - kim-api-2.4.1-GCC-13.3.0.eb: | ||
| options: | ||
| # See https://github.com/easybuilders/easybuild-easyconfigs/pull/25133 | ||
| from-commit: c484c12aaad5da0e27cd9269d59b4ecaa89927ab | ||
| - LAMMPS-22Jul2025-foss-2024a-kokkos.eb: | ||
| options: | ||
| # See https://github.com/easybuilders/easybuild-easyconfigs/pull/25133 | ||
| # and https://github.com/easybuilders/easybuild-easyconfigs/pull/25593 | ||
| from-commit: e2dedae93022d7e3f10bf2983ca8a03b03b0dca0 | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -1575,6 +1575,31 @@ def pre_test_hook_ignore_failing_tests_OpenBabel_a64fx(self, *args, **kwargs): | |
| self.cfg['testopts'] = "|| echo ignoring failing tests" | ||
|
|
||
|
|
||
| def pre_test_hook_ignore_failing_tests_LAMMPS_ARM_generic(self, *args, **kwargs): | ||
| """ | ||
| Pre-test hook for LAMMPS: skip failing ctest for selected LAMMPS version on ARM generic. | ||
|
|
||
| See: https://github.com/lammps/lammps/issues/4926 | ||
| """ | ||
| print('build option before pre_test_hooki: ', build_option('ignore_test_failure')) | ||
| if self.name == 'LAMMPS' and self.version in ('22Jul2025'): | ||
| cpu_target = get_eessi_envvar('EESSI_SOFTWARE_SUBDIR') | ||
| print('Debug: ', cpu_target) | ||
| if cpu_target == CPU_TARGET_AARCH64_GENERIC: | ||
| print('I will reset ignore_test_failure') | ||
| self.orig_ignore_test_failure = build_option('ignore_test_failure') | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You should do this outside of the |
||
| update_build_option('ignore_test_failure', True) | ||
| print(build_option('ignore_test_failure')) | ||
|
|
||
|
|
||
| def post_test_hook(self, *args, **kwargs): | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @laraPPr Due to where Try doing it in |
||
| """ | ||
| If self.orig_ignore_test_failure exists reset ignore_test_failure to its original value. | ||
| """ | ||
| if hasattr(self, "orig_ignore_test_failure") and self.orig_ignore_test_failure != build_option('ignore_test_failure'): | ||
| update_build_option('ignore_test_failure', self.orig_ignore_test_failure) | ||
|
|
||
|
|
||
| def pre_single_extension_hook(ext, *args, **kwargs): | ||
| """Main pre-extension: trigger custom functions based on software name.""" | ||
| if ext.name in PRE_SINGLE_EXTENSION_HOOKS: | ||
|
|
@@ -1986,6 +2011,7 @@ def post_easyblock_hook(self, *args, **kwargs): | |
| 'netCDF': pre_test_hook_ignore_failing_tests_netCDF, | ||
| 'OpenBabel': pre_test_hook_ignore_failing_tests_OpenBabel_a64fx, | ||
| 'PyTorch': pre_test_hook_increase_max_failed_tests_arm_PyTorch, | ||
| 'LAMMPS': pre_test_hook_ignore_failing_tests_LAMMPS_ARM_generic, | ||
| } | ||
|
|
||
| PRE_SINGLE_EXTENSION_HOOKS = { | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.