fix zo build sorting (#2611)

frzyc · Jan 25, 2025 · 5598e99 · 5598e99
1 parent 3d446c7
commit 5598e99
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 12 deletions.
diff --git a/libs/sr/solver/src/parentWorker.ts b/libs/sr/solver/src/parentWorker.ts
@@ -188,15 +188,10 @@ async function start({
     })
   )
 
-  // Trigger spinner on UI
-  if (results.length > MAX_BUILDS) {
-    results.sort((a, b) => b.value - a.value)
-    results = results.slice(0, MAX_BUILDS)
-  }
   // Send back results, which can take a few seconds
   postMessage({
     resultType: 'done',
-    buildResults: results,
+    buildResults: results.sort((a, b) => b.value - a.value).slice(0, 10), // TODO: take numBuilds from opt UI
   })
 }
 

diff --git a/libs/zzz/solver/src/parentWorker.ts b/libs/zzz/solver/src/parentWorker.ts
@@ -176,15 +176,10 @@ async function start({
     })
   )
 
-  // Trigger spinner on UI
-  if (results.length > MAX_BUILDS) {
-    results.sort((a, b) => b.value - a.value)
-    results = results.slice(0, MAX_BUILDS)
-  }
   // Send back results, which can take a few seconds
   postMessage({
     resultType: 'done',
-    buildResults: results.slice(0, 10), // TODO: take numBuilds from opt UI
+    buildResults: results.sort((a, b) => b.value - a.value).slice(0, 10), // TODO: take numBuilds from opt UI
   })
 }