Merge pull request #413 from knshnb/fix-dominated-trials

Fix `getDominatedTrials`
optuna · Mar 14, 2023 · 51f0ddd · 51f0ddd
2 parents b7334bf + cd485c1
commit 51f0ddd
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/optuna_dashboard/ts/dominatedTrials.ts b/optuna_dashboard/ts/dominatedTrials.ts
@@ -27,7 +27,7 @@ export const getDominatedTrials = (
   const dominatedTrials: boolean[] = []
   normalizedValues.forEach((values0: number[], i: number) => {
     const dominated = normalizedValues.some((values1: number[], j: number) => {
-      if (i === j) {
+      if (i === j || values0.every((v, i) => v == values1[i])) {
         return false
       }
       return values0.every((value0: number, k: number) => {