Skip to content

Commit 14f4853

Browse files
geetu040PGijsbers
andauthored
Update openml/_api/resources/setup.py
#1619 (comment) Co-authored-by: Pieter Gijsbers <p.gijsbers@tue.nl>
1 parent bc10ac6 commit 14f4853

1 file changed

Lines changed: 7 additions & 11 deletions

File tree

openml/_api/resources/setup.py

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -133,20 +133,16 @@ def _create_setup(self, result_dict: dict) -> OpenMLSetup:
133133

134134
xml_parameters = result_dict["oml:setup_parameters"]["oml:parameter"]
135135
if isinstance(xml_parameters, dict):
136-
parameters = {
137-
int(xml_parameters["oml:id"]): self._create_setup_parameter_from_xml(
138-
xml_parameters
139-
),
140-
}
141-
elif isinstance(xml_parameters, builtins.list):
142-
parameters = {
143-
int(xml_parameter["oml:id"]): self._create_setup_parameter_from_xml(xml_parameter)
144-
for xml_parameter in xml_parameters
145-
}
146-
else:
136+
xml_parameters = [xml_parameters]
137+
if not isinstance(xml_parameters, list):
147138
raise ValueError(
148139
f"Expected None, list or dict, received something else: {type(xml_parameters)!s}",
149140
)
141+
142+
parameters = {
143+
int(xml_parameter["oml:id"]): self._create_setup_parameter_from_xml(xml_parameter)
144+
for xml_parameter in xml_parameters
145+
}
150146
return OpenMLSetup(setup_id, flow_id, parameters)
151147

152148
def _create_setup_parameter_from_xml(self, result_dict: dict[str, str]) -> OpenMLParameter:

0 commit comments

Comments
 (0)