enhance(algolia): rank by viewTitleIndexWithinExplorer

owid · Apr 2, 2024 · 3feb186 · 3feb186
1 parent 77be58b
commit 3feb186
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 6 deletions.
diff --git a/baker/algolia/configureAlgolia.ts b/baker/algolia/configureAlgolia.ts
@@ -140,10 +140,17 @@ export const configureAlgolia = async () => {
         searchableAttributes: [
             "unordered(viewTitle)",
             "unordered(viewSettings)",
+            "unordered(explorerTitle)",
         ],
-        customRanking: ["desc(score)", "asc(viewIndexWithinExplorer)"],
-        attributeForDistinct: "viewTitleAndExplorerSlug",
-        distinct: true,
+        customRanking: [
+            // For multiple explorer views with the same title, we want to avoid surfacing duplicates.
+            // So, rank a result with viewTitleIndexWithinExplorer=0 way more highly than one with 1, 2, etc.
+            "asc(viewTitleIndexWithinExplorer)",
+            "desc(score)",
+            "asc(viewIndexWithinExplorer)",
+        ],
+        attributeForDistinct: "explorerSlug",
+        distinct: 4,
         minWordSizefor1Typo: 6,
     })
 

diff --git a/baker/algolia/indexExplorerViewsToAlgolia.ts b/baker/algolia/indexExplorerViewsToAlgolia.ts
@@ -64,10 +64,13 @@ const explorerChoiceToViewSettings = (
     })
 }
 
-const computeScore = (record: Partial<ExplorerViewEntryWithExplorerInfo>) =>
+const computeScore = (
+    record: Omit<ExplorerViewEntry, "viewTitleIndexWithinExplorer"> &
+        Partial<ExplorerViewEntryWithExplorerInfo>
+) =>
     (record.explorerViews_7d ?? 0) * 10 -
-    (record.numNonDefaultSettings ?? 0) * 50 -
-    (record.titleLength ?? 0)
+    record.numNonDefaultSettings * 50 -
+    record.titleLength
 
 const getExplorerViewRecordsForExplorerSlug = async (
     trx: db.KnexReadonlyTransaction,