Fix histogram scaling

Unscored user bin was accidentally included in calculating the maximum bin count
shiruken · Aug 10, 2024 · 2d7a027 · 2d7a027
1 parent 5fb001f
commit 2d7a027
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/devvit.yaml b/devvit.yaml
@@ -1,2 +1,2 @@
 name: user-scorer
-version: 1.2.1
+version: 1.2.2
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
   "private": true,
   "name": "user-scorer",
-  "version": "1.2.1",
+  "version": "1.2.2",
   "license": "BSD-3-Clause",
   "type": "module",
   "scripts": {

diff --git a/src/handlers.ts b/src/handlers.ts
@@ -330,7 +330,7 @@ export async function generateReport(_event: MenuItemOnPressEvent, context: Cont
   }
 
   // Generate the histogram visualization
-  const bin_max = Math.max(...histogram.bins.map(bin => bin.count));
+  const bin_max = Math.max(...(histogram.bins.slice(1)).map(bin => bin.count));
   let chart = "";
   if (bin_max > 0) {
     histogram.bins.slice(1).forEach(bin => {