Skip to content

Commit aa679ad

Browse files
committed
Replace gridMode and randomMode by definition_mode in pretty repr of NumericalHyperparameterSettings
1 parent a72a381 commit aa679ad

File tree

1 file changed

+12
-16
lines changed

1 file changed

+12
-16
lines changed

dataikuapi/dss/ml.py

Lines changed: 12 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -602,29 +602,25 @@ def _pretty_repr(self):
602602
class NumericalHyperparameterSettings(HyperparameterSettings):
603603

604604
def _pretty_repr(self):
605-
active_dict = self._get_active_settings_dict()
606-
return self.__class__.__name__ + "(hyperparameter=\"{}\", settings={})".format(self.name, json.dumps(active_dict, indent=4))
605+
raw_hyperparam = self._algo_settings[self.name]
606+
pretty_hyperparam = dict()
607+
if self._algo_settings.strategy == "GRID":
608+
pretty_hyperparam["definition_mode"] = raw_hyperparam["gridMode"]
609+
else:
610+
# RANDOM and BAYESIAN strategies
611+
pretty_hyperparam["definition_mode"] = raw_hyperparam["randomMode"]
612+
if self.definition_mode == "EXPLICIT":
613+
pretty_hyperparam["values"] = raw_hyperparam["values"]
614+
else:
615+
pretty_hyperparam["range"] = raw_hyperparam["range"]
616+
return self.__class__.__name__ + "(hyperparameter=\"{}\", settings={})".format(self.name, json.dumps(pretty_hyperparam, indent=4))
607617

608618
def __repr__(self):
609619
raw_dict = self._algo_settings[self.name]
610620
return self.__class__.__name__ + "(hyperparameter=\"{}\", settings={})".format(self.name, json.dumps(raw_dict))
611621

612622
__str__ = __repr__
613623

614-
def _get_active_settings_dict(self):
615-
clean_hyperparam = dict()
616-
raw_hyperparam = self._algo_settings[self.name]
617-
if self.definition_mode == "EXPLICIT":
618-
clean_hyperparam["values"] = raw_hyperparam["values"]
619-
else:
620-
clean_hyperparam["range"] = raw_hyperparam["range"]
621-
if self._algo_settings.strategy == "GRID":
622-
clean_hyperparam["gridMode"] = raw_hyperparam["gridMode"]
623-
else:
624-
# RANDOM and BAYESIAN strategies
625-
clean_hyperparam["randomMode"] = raw_hyperparam["randomMode"]
626-
return clean_hyperparam
627-
628624
@property
629625
def definition_mode(self):
630626
"""

0 commit comments

Comments
 (0)