Merge pull request #5367 from voxel51/bug/sortable-table-ME

Fixes sort in model eval display option and enables "display options" in table view
voxel51 · Jan 13, 2025 · 43b874e · 43b874e
2 parents 7fd2b30 + 36bc7e3
commit 43b874e
Showing 1 changed file with 13 additions and 14 deletions.
diff --git a/app/packages/core/src/plugins/SchemaIO/components/NativeModelEvaluationView/Evaluation.tsx b/app/packages/core/src/plugins/SchemaIO/components/NativeModelEvaluationView/Evaluation.tsx
@@ -1059,18 +1059,16 @@ export default function Evaluation(props: EvaluationProps) {
                         );
                       })}
                     </Select>
-                    {classMode === "chart" && (
-                      <IconButton
-                        onClick={() => {
-                          setClassPerformanceDialogConfig((state) => ({
-                            ...state,
-                            open: true,
-                          }));
-                        }}
-                      >
-                        <Settings />
-                      </IconButton>
-                    )}
+                    <IconButton
+                      onClick={() => {
+                        setClassPerformanceDialogConfig((state) => ({
+                          ...state,
+                          open: true,
+                        }));
+                      }}
+                    >
+                      <Settings />
+                    </IconButton>
                   </Stack>
                 </Stack>
                 {classMode === "chart" && (
@@ -1654,9 +1652,10 @@ function formatPerClassPerformance(perClassPerformance, barConfig) {
         return b.value - a.value;
       } else if (sortBy === "worst") {
         return a.value - b.value;
-      } else {
-        return b.property.localeCompare(a.property);
+      } else if (sortBy === "az") {
+        return a.property.localeCompare(b.property);
       }
+      return b.property.localeCompare(a.property);
     });
   }