express cloud eval capability with engine property

halo2404 · Nov 27, 2024 · b42c98b · b42c98b
1 parent 540db34
commit b42c98b
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/ui/ceval/src/ctrl.ts b/ui/ceval/src/ctrl.ts
@@ -209,10 +209,7 @@ export default class CevalCtrl {
   }
 
   get isCacheable(): boolean {
-    //>=sf16 or fsf14+nnue
-    return (
-      this.engines.active?.tech === 'NNUE' && this.engines.active.requires.includes('dynamicImportFromWorker')
-    );
+    return !!this.engines.active?.cloudEval;
   }
 
   get showingCloud(): boolean {

diff --git a/ui/ceval/src/engines/engines.ts b/ui/ceval/src/engines/engines.ts
@@ -45,6 +45,7 @@ export class Engines {
         tech: 'NNUE',
         requires: ['sharedMem', 'simd', 'dynamicImportFromWorker'],
         variants: [key],
+        cloudEval: true,
         assets: {
           version: 'sfw006',
           root: 'npm/lila-stockfish-web',
@@ -72,6 +73,7 @@ export class Engines {
           tech: 'NNUE',
           requires: ['sharedMem', 'simd', 'dynamicImportFromWorker'],
           minMem: 1536,
+          cloudEval: true,
           assets: {
             version: 'sfw006',
             root: 'npm/lila-stockfish-web',
@@ -88,6 +90,7 @@ export class Engines {
           tech: 'NNUE',
           requires: ['sharedMem', 'simd', 'dynamicImportFromWorker'],
           minMem: 2560,
+          cloudEval: true,
           assets: {
             version: 'sfw006',
             root: 'npm/lila-stockfish-web',
@@ -123,6 +126,7 @@ export class Engines {
           tech: 'HCE',
           requires: ['sharedMem', 'simd', 'dynamicImportFromWorker'],
           variants: variants.map(v => v[0]),
+          cloudEval: true,
           assets: {
             version: 'sfw006',
             root: 'npm/lila-stockfish-web',

diff --git a/ui/ceval/src/types.ts b/ui/ceval/src/types.ts
@@ -48,6 +48,7 @@ export interface ExternalEngineInfoFromServer extends BaseEngineInfo {
 
 export interface ExternalEngineInfo extends ExternalEngineInfoFromServer {
   tech: 'EXTERNAL';
+  cloudEval?: false;
 }
 
 export interface BrowserEngineInfo extends BaseEngineInfo {
@@ -57,6 +58,7 @@ export interface BrowserEngineInfo extends BaseEngineInfo {
   assets: { root?: string; js?: string; wasm?: string; version?: string; nnue?: string[] };
   requires: Feature[];
   obsoletedBy?: Feature;
+  cloudEval?: boolean;
 }
 
 export type EngineInfo = BrowserEngineInfo | ExternalEngineInfo;