Skip to content

Commit

Permalink
adapt to changes from review
Browse files Browse the repository at this point in the history
  • Loading branch information
jenshnielsen committed Sep 30, 2022
1 parent cea1665 commit ed9e189
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 9 deletions.
8 changes: 4 additions & 4 deletions qcodes/dataset/dond/do_nd.py
Expand Up @@ -68,7 +68,7 @@ def __init__(
self._sweeps = sweeps
self._setpoints = self._make_setpoints_tuples()
self._setpoints_dict = self._make_setpoints_dict()
self._shapes = self._make_shape(sweeps, additional_setpoints)
self._shape = self._make_shape(sweeps, additional_setpoints)

@property
def setpoints_dict(self) -> dict[str, list[Any]]:
Expand Down Expand Up @@ -174,7 +174,7 @@ def sweep_groupes(self) -> tuple[tuple[ParameterBase, ...], ...]:

# next we generate all valid combinations of picking one parameter from each
# dimension in the setpoints.
setpoint_combinations = itertools.product(*expanded_parameter)
setpoint_combinations = itertools.product(*expanded_param_tuples)

setpoint_combinations_expanded = tuple(
tuple(itertools.chain.from_iterable(setpoint_combination))
Expand Down Expand Up @@ -267,8 +267,8 @@ def __init__(
if dataset_dependencies and len(grouped_parameters) > 1:
raise ValueError(
"Measured parameters have been grouped both in input "
"and in the given dataset dependencies. This is not supported, "
"group measurement parameters either in input or in dataset dependencies"
"and given in dataset dependencies. This is not supported, "
"group measurement parameters either in input or in dataset dependencies."
)

if dataset_dependencies is None:
Expand Down
11 changes: 6 additions & 5 deletions qcodes/tests/dataset/dond/test_doNd.py
Expand Up @@ -969,14 +969,14 @@ def test_dond_together_sweep_sweeper(_param_set, _param_set_2, _param):
assert sweeper.shape == (10,)
assert sweeper.all_setpoint_params == (sweep_1.param, sweep_2.param)

assert list(sweeper.setpoint_dict.keys()) == [
assert list(sweeper.setpoints_dict.keys()) == [
"simple_setter_parameter",
"simple_setter_parameter_2",
]
assert sweeper.setpoint_dict["simple_setter_parameter"] == list(
assert sweeper.setpoints_dict["simple_setter_parameter"] == list(
sweep_1.get_setpoints()
)
assert sweeper.setpoint_dict["simple_setter_parameter_2"] == list(
assert sweeper.setpoints_dict["simple_setter_parameter_2"] == list(
sweep_2.get_setpoints()
)

Expand Down Expand Up @@ -1031,7 +1031,8 @@ def test_dond_together_sweep_sweeper_mixed_splitting():
ValueError,
match=re.escape(
"Measured parameters have been grouped both in input "
"and using dataset dependencies. This is not supported."
"and given in dataset dependencies. This is not supported, "
"group measurement parameters either in input or in dataset dependencies."
),
):
a = ManualParameter("a", initial_value=0)
Expand Down Expand Up @@ -1140,7 +1141,7 @@ def test_dond_together_sweep_sweeper_combined_explict_names_and_single_name():
with pytest.raises(
ValueError,
match=re.escape(
"Creating multiple datasets but one only one measurement name given."
"Creating multiple datasets but only one measurement name given."
),
):
datasets, _, _ = dond(
Expand Down

0 comments on commit ed9e189

Please sign in to comment.