Merge branch 'fix_compact_process_binding' into pytorch

casparvl · May 3, 2024 · 760cd59 · 760cd59
2 parents b868cc1 + 6ecad3a
commit 760cd59
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/.github/workflows/scorecards.yml b/.github/workflows/scorecards.yml
@@ -22,6 +22,7 @@ permissions: read-all
 
 jobs:
   analysis:
+    if: github.repository == 'EESSI/test-suite'   # Prevent running on forks
     name: Scorecards analysis
     runs-on: ubuntu-latest
     permissions:

diff --git a/eessi/testsuite/hooks.py b/eessi/testsuite/hooks.py
@@ -483,7 +483,7 @@ def set_compact_process_binding(test: rfm.RegressionTest):
     # Other launchers may or may not do the correct binding
     test.env_vars['I_MPI_PIN_CELL'] = 'core'  # Don't bind to hyperthreads, only to physcial cores
     test.env_vars['I_MPI_PIN_DOMAIN'] = '%s:compact' % physical_cpus_per_task
-    test.env_vars['OMPI_MCA_rmaps_base_mapping_policy'] = 'node:PE=%s' % physical_cpus_per_task
+    test.env_vars['OMPI_MCA_rmaps_base_mapping_policy'] = 'slot:PE=%s' % physical_cpus_per_task
     # Default binding for SLURM. Only effective if the task/affinity plugin is enabled
     # and when number of tasks times cpus per task equals either socket, core or thread count
     test.env_vars['SLURM_CPU_BIND'] = 'verbose'