Skip to content

Commit 27c68a8

Browse files
author
Vasileios Karakasis
authored
Merge branch 'master' into update-alloc_speed
2 parents a81d740 + ebbd740 commit 27c68a8

File tree

4 files changed

+25
-25
lines changed

4 files changed

+25
-25
lines changed

.github/workflows/main.yml

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,9 @@ jobs:
99
matrix:
1010
python-version: ['3.6', '3.7', '3.8', '3.9', '3.10']
1111
steps:
12-
- uses: actions/checkout@v2
12+
- uses: actions/checkout@v3
1313
- name: Set up Python ${{ matrix.python-version }}
14-
uses: actions/setup-python@v2
14+
uses: actions/setup-python@v3
1515
with:
1616
python-version: ${{ matrix.python-version }}
1717
- name: Install dependencies
@@ -36,9 +36,9 @@ jobs:
3636
unittest-macos:
3737
runs-on: macos-latest
3838
steps:
39-
- uses: actions/checkout@v2
39+
- uses: actions/checkout@v3
4040
- name: Set up Python 3.9
41-
uses: actions/setup-python@v2
41+
uses: actions/setup-python@v3
4242
with:
4343
python-version: 3.9
4444
- name: Install dependencies
@@ -54,7 +54,7 @@ jobs:
5454
matrix:
5555
modules-version: [Lmod, Lmod77, Tmod32, Tmod4]
5656
steps:
57-
- uses: actions/checkout@v2
57+
- uses: actions/checkout@v3
5858
- name: Build Image for ${{ matrix.modules-version }}
5959
run: |
6060
docker build -f ci-scripts/dockerfiles/${{ matrix.modules-version }}.dockerfile -t reframe:${{ matrix.modules-version }} .
@@ -65,7 +65,7 @@ jobs:
6565
tutorialtest:
6666
runs-on: ubuntu-latest
6767
steps:
68-
- uses: actions/checkout@v2
68+
- uses: actions/checkout@v3
6969
- name: Build Image for Tutorial Tests
7070
run: |
7171
docker build -f ci-scripts/dockerfiles/tutorials.dockerfile -t reframe:tutorials .
@@ -76,7 +76,7 @@ jobs:
7676
unusedimports:
7777
runs-on: ubuntu-latest
7878
steps:
79-
- uses: actions/checkout@v2
79+
- uses: actions/checkout@v3
8080
with:
8181
fetch-depth: 0
8282
- name: Lint Code Base
@@ -89,9 +89,9 @@ jobs:
8989
wheelvalidation:
9090
runs-on: ubuntu-latest
9191
steps:
92-
- uses: actions/checkout@v2
92+
- uses: actions/checkout@v3
9393
- name: Setup up Python 3.8
94-
uses: actions/setup-python@v2
94+
uses: actions/setup-python@v3
9595
with:
9696
python-version: 3.8
9797
- name: Generate Wheel
@@ -109,9 +109,9 @@ jobs:
109109
docvalidation:
110110
runs-on: ubuntu-latest
111111
steps:
112-
- uses: actions/checkout@v2
112+
- uses: actions/checkout@v3
113113
- name: Setup up Python 3.8
114-
uses: actions/setup-python@v2
114+
uses: actions/setup-python@v3
115115
with:
116116
python-version: 3.8
117117
- name: Install Doc Requirements

.github/workflows/publish.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,9 @@ jobs:
88
publish:
99
runs-on: ubuntu-latest
1010
steps:
11-
- uses: actions/checkout@v2
11+
- uses: actions/checkout@v3
1212
- name: Setup up Python 3.8
13-
uses: actions/setup-python@v2
13+
uses: actions/setup-python@v3
1414
with:
1515
python-version: 3.8
1616
- name: Generate dist packages

cscs-checks/apps/lammps/lammps_check.py

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,8 @@
1111

1212
class LAMMPSCheck(rfm.RunOnlyRegressionTest):
1313
scale = parameter(['small', 'large'])
14-
variant = parameter(['maint', 'prod'])
1514
modules = ['cray-python', 'LAMMPS']
16-
tags = {'scs', 'external-resources'}
15+
tags = {'external-resources', 'maintenance', 'production'}
1716
maintainers = ['LM']
1817
strict_check = False
1918
extra_resources = {
@@ -78,14 +77,13 @@ def setup_by_scale(self):
7877
self.num_tasks = 32
7978
self.num_tasks_per_node = 2
8079

81-
self.reference = self.references_by_variant[self.variant][self.scale]
82-
self.tags |= {'maintenance', 'production'}
80+
self.reference = self.refs_by_scale[self.scale]
8381

8482

8583
@rfm.simple_test
8684
class LAMMPSCPUCheck(LAMMPSCheck):
8785
valid_systems = ['daint:mc', 'eiger:mc', 'pilatus:mc']
88-
res_by_scale = {
86+
refs_by_scale = {
8987
'small': {
9088
'dom:mc': {'perf': (4394, -0.05, None, 'timesteps/s')},
9189
'daint:mc': {'perf': (3350, -0.30, None, 'timesteps/s')},
@@ -121,5 +119,4 @@ def setup_by_scale(self):
121119
self.num_tasks_per_node = 128
122120
self.num_tasks = 256 if self.scale == 'small' else 512
123121

124-
self.reference = self.references_by_variant[self.variant][self.scale]
125-
self.tags |= {'maintenance', 'production'}
122+
self.reference = self.refs_by_scale[self.scale]

docs/dependencies.rst

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,8 @@ This can be expressed inside :class:`T1` using the :func:`depends_on` method:
2424
valid_systems = ['P0', 'P1']
2525
valid_prog_environs = ['E0', 'E1']
2626
27-
def __init__(self):
27+
@run_after('init')
28+
def setup_deps(self):
2829
self.depends_on('T0')
2930
3031
Conceptually, this dependency can be viewed at the test level as follows:
@@ -58,8 +59,9 @@ You can achieve this as follows:
5859
5960
@rfm.simple_test
6061
class T1(rfm.RegressionTest):
61-
def __init__(self):
62-
...
62+
...
63+
@run_after('init')
64+
def setup_deps(self):
6365
self.depends_on('T0', how=udeps.fully)
6466
6567
@@ -154,8 +156,9 @@ The following code will create dependencies only if the source partition is ``P0
154156
155157
@rfm.simple_test
156158
class T1(rfm.RegressionTest):
157-
def __init__(self):
158-
...
159+
...
160+
@run_after('init')
161+
def setup_deps(self):
159162
self.depends_on('T0', how=myway)
160163
161164

0 commit comments

Comments
 (0)