Skip to content

Commit 2e73452

Browse files
committed
test fixes
1 parent 151295b commit 2e73452

11 files changed

Lines changed: 21 additions & 30 deletions

libensemble/alloc_funcs/persistent_aposmm_alloc.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@ def persistent_aposmm_alloc(W, H, sim_specs, gen_specs, alloc_specs, persis_info
2121
if libE_info["sim_max_given"] or not libE_info["any_idle_workers"]:
2222
return {}, persis_info
2323

24+
if not persis_info:
25+
persis_info = {i: {} for i in range(len(W))}
2426
user = {**gen_specs, **alloc_specs.get("user", {})}
2527
init_sample_size = user["initial_batch_size"]
2628
manage_resources = libE_info["use_resource_sets"]
@@ -70,7 +72,7 @@ def persistent_aposmm_alloc(W, H, sim_specs, gen_specs, alloc_specs, persis_info
7072
if persis_info.get("gen_started") is None:
7173
for wid in support.avail_worker_ids(persistent=False, gen_workers=True):
7274
# Finally, call a persistent generator as there is nothing else to do.
73-
persis_info.get(wid)["nworkers"] = len(W)
75+
persis_info[wid]["nworkers"] = len(W)
7476
try:
7577
Work[wid] = support.gen_work(
7678
wid, gen_specs.get("in", []), range(len(H)), persis_info.get(wid), persistent=True

libensemble/tests/functionality_tests/test_runlines_adaptive_workers.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -63,9 +63,6 @@
6363

6464
alloc_specs = {
6565
"alloc_f": give_sim_work_first,
66-
"user": {
67-
"batch_mode": False,
68-
},
6966
}
7067

7168
comms = libE_specs["comms"]
@@ -85,7 +82,7 @@
8582
exit_criteria = {"sim_max": 40, "wallclock_max": 300}
8683

8784
# Perform the run
88-
H, persis_info, flag = libE(sim_specs, gen_specs, exit_criteria, alloc_specs=libE_specs, libE_specs=libE_specs)
85+
H, persis_info, flag = libE(sim_specs, gen_specs, exit_criteria, alloc_specs=alloc_specs, libE_specs=libE_specs)
8986

9087
if is_manager:
9188
assert flag == 0

libensemble/tests/functionality_tests/test_uniform_sampling_cancel.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
from libensemble.libE import libE
3131
from libensemble.sim_funcs.six_hump_camel import six_hump_camel
3232
from libensemble.tests.regression_tests.support import six_hump_camel_minima as minima
33-
from libensemble.tools import parse_args
33+
from libensemble.tools import get_rng, parse_args
3434

3535

3636
def create_H0(persis_info, gen_specs, sim_max):
@@ -42,7 +42,8 @@ def create_H0(persis_info, gen_specs, sim_max):
4242
b = sim_max
4343

4444
H0 = np.zeros(b, dtype=[("x", float, 2), ("sim_id", int), ("sim_started", bool), ("cancel_requested", bool)])
45-
H0["x"] = persis_info[0]["rand_stream"].uniform(lb, ub, (b, n))
45+
rng = get_rng(gen_specs, {})
46+
H0["x"] = rng.uniform(lb, ub, (b, n))
4647
H0["sim_id"] = range(b)
4748
H0["sim_started"] = False
4849
for i in range(b):
@@ -126,6 +127,7 @@ def create_H0(persis_info, gen_specs, sim_max):
126127
if is_manager:
127128
print("Testing cancellations with non-persistent gen functions")
128129

130+
persis_info = {0: {}}
129131
for testnum in range(1, 6):
130132
alloc_specs = allocs[testnum]
131133
if is_manager:
@@ -142,7 +144,7 @@ def create_H0(persis_info, gen_specs, sim_max):
142144

143145
# Perform the run - do not overwrite persis_info
144146
H, persis_out, flag = libE(
145-
sim_specs, gen_specs, exit_criteria, persis_info, alloc_specs, libE_specs=libE_specs, H0=H0
147+
sim_specs, gen_specs, exit_criteria, persis_info, alloc_specs=alloc_specs, libE_specs=libE_specs, H0=H0
146148
)
147149

148150
if is_manager:

libensemble/tests/regression_tests/support.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,13 +65,16 @@ def remote_write_gen_func(calc_in, persis_info, gen_specs, libE_info):
6565
return H_o, persis_info
6666

6767

68-
def write_uniform_gen_func(H, persis_info, gen_specs, _):
68+
def write_uniform_gen_func(H, persis_info, gen_specs, libE_info):
69+
from libensemble.tools import get_rng
70+
6971
ub = gen_specs["user"]["ub"]
7072
lb = gen_specs["user"]["lb"]
7173
n = len(lb)
7274
b = gen_specs["batch_size"]
7375
H_o = np.zeros(b, dtype=gen_specs["out"])
74-
H_o["x"] = persis_info["rand_stream"].uniform(lb, ub, (b, n))
76+
rng = get_rng(gen_specs, libE_info)
77+
H_o["x"] = rng.uniform(lb, ub, (b, n))
7578
with open("test_gen_out.txt", "a") as f:
7679
f.write(f"gen_f produced: {H_o['x']}\n")
7780
return H_o, persis_info

libensemble/tests/regression_tests/test_asktell_aposmm_nlopt.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,6 @@ def six_hump_camel_func(x):
6262

6363
n = 2
6464

65-
workflow.libE_specs.gen_on_manager = True
66-
6765
vocs = VOCS(
6866
variables={"core": [-3, 3], "edge": [-2, 2], "core_on_cube": [-3, 3], "edge_on_cube": [-2, 2]},
6967
objectives={"energy": "MINIMIZE"},

libensemble/tests/regression_tests/test_ensemble_platform_workdir.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@
5050
"persis_in": ["f", "x", "sim_id"],
5151
"out": [("priority", float), ("resource_sets", int), ("x", float, n)],
5252
"initial_batch_size": ensemble.nworkers - 1,
53-
"give_all_with_same_priority": False,
53+
"batch_evaluate_same_priority": False,
5454
"async_return": False,
5555
"user": {
5656
"max_resource_sets": ensemble.nworkers - 1, # Any sim created can req. 1 worker up to all.

libensemble/tests/regression_tests/test_persistent_aposmm_ibcdfo_manifold_sampling.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -113,12 +113,10 @@ def synthetic_beamline_mapping(H, _, sim_specs):
113113

114114
alloc_specs = {"alloc_f": alloc_f}
115115

116-
persis_info = {}
117-
118116
exit_criteria = {"sim_max": 500}
119117

120118
# Perform the run
121-
H, persis_info, flag = libE(sim_specs, gen_specs, exit_criteria, persis_info, alloc_specs, libE_specs)
119+
H, persis_info, flag = libE(sim_specs, gen_specs, exit_criteria, alloc_specs=alloc_specs, libE_specs=libE_specs)
122120

123121
if is_manager:
124122
assert np.min(H["f"]) == 2.0, "The best is 2"

libensemble/tests/regression_tests/test_persistent_aposmm_ibcdfo_pounders.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -122,12 +122,10 @@ def synthetic_beamline_mapping(H, _, sim_specs):
122122

123123
alloc_specs = {"alloc_f": alloc_f}
124124

125-
persis_info = {}
126-
127125
exit_criteria = {"sim_max": 500}
128126

129127
# Perform the run
130-
H, persis_info, flag = libE(sim_specs, gen_specs, exit_criteria, persis_info, alloc_specs, libE_specs)
128+
H, persis_info, flag = libE(sim_specs, gen_specs, exit_criteria, alloc_specs=alloc_specs, libE_specs=libE_specs)
131129

132130
if is_manager:
133131
assert persis_info[0].get("run_order"), "Run_order should have been given back"

libensemble/tests/regression_tests/test_persistent_aposmm_ibcdfo_pounders_jax.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -119,12 +119,10 @@ def synthetic_beamline_mapping(H, _, sim_specs):
119119

120120
alloc_specs = {"alloc_f": alloc_f}
121121

122-
persis_info = {}
123-
124122
exit_criteria = {"sim_max": 500}
125123

126124
# Perform the run
127-
H, persis_info, flag = libE(sim_specs, gen_specs, exit_criteria, persis_info, alloc_specs, libE_specs)
125+
H, persis_info, flag = libE(sim_specs, gen_specs, exit_criteria, alloc_specs=alloc_specs, libE_specs=libE_specs)
128126

129127
if is_manager:
130128
print(H[["x", "f", "local_min"]])

libensemble/tests/regression_tests/test_persistent_aposmm_nlopt.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -81,12 +81,10 @@
8181

8282
alloc_specs = {"alloc_f": alloc_f}
8383

84-
persis_info = {}
85-
8684
exit_criteria = {"sim_max": 2000}
8785

8886
# Perform the run
89-
H, persis_info, flag = libE(sim_specs, gen_specs, exit_criteria, persis_info, alloc_specs, libE_specs)
87+
H, persis_info, flag = libE(sim_specs, gen_specs, exit_criteria, alloc_specs=alloc_specs, libE_specs=libE_specs)
9088

9189
if is_manager:
9290
print("[Manager]:", H[np.where(H["local_min"])]["x"])

0 commit comments

Comments
 (0)