Merge pull request #414 from c-bata/improve-human-in-the-loop-ui

Minor improvements on human-in-the-loop UI
optuna · Mar 14, 2023 · 2e0baba · 2e0baba
2 parents 51f0ddd + cad8870
commit 2e0baba
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 11 deletions.
diff --git a/optuna_dashboard/ts/components/BestTrialsCard.tsx b/optuna_dashboard/ts/components/BestTrialsCard.tsx
@@ -32,16 +32,22 @@ export const BestTrialsCard: FC<{
     header = `Best Trial (number=${bestTrial.number})`
     content = (
       <>
-        <Typography
-          variant="h3"
-          sx={{
-            fontWeight: theme.typography.fontWeightBold,
-            marginBottom: theme.spacing(2),
-          }}
-          color="secondary"
-        >
-          {bestTrial.values}
-        </Typography>
+        {bestTrial.values === undefined || bestTrial.values.length === 1 ? (
+          <Typography
+            variant="h3"
+            sx={{
+              fontWeight: theme.typography.fontWeightBold,
+              marginBottom: theme.spacing(2),
+            }}
+            color="secondary"
+          >
+            {bestTrial.values}
+          </Typography>
+        ) : (
+          <Typography>
+            Objective Values = [{bestTrial.values?.join(", ")}]
+          </Typography>
+        )}
         <Typography>
           Params = [
           {bestTrial.params

diff --git a/optuna_dashboard/ts/components/TrialList.tsx b/optuna_dashboard/ts/components/TrialList.tsx
@@ -158,7 +158,7 @@ const TrialListDetail: FC<{
 
   const params = trial.state === "Waiting" ? trial.fixed_params : trial.params
   const info: [string, string | null | ReactNode][] = [
-    ["Value", trial.values?.map((v) => v.toString()).join(" ") || "None"],
+    ["Value", trial.values?.map((v) => v.toString()).join(", ") || "None"],
     [
       "Intermediate Values",
       <Box>