bench: chmod +x bench scripts

Author: hulxv

.github/workflows/benchmark.yml

@@ -18,7 +18,6 @@ jobs:
   benchmark:
     name: Performance Benchmark
     runs-on: ubuntu-latest
-    timeout-minutes: 30
 
     steps:
       - name: Check out the repo
@@ -112,13 +111,6 @@ jobs:
         run: |
           pip install pandas matplotlib seaborn numpy
 
-      - name: Make scripts executable
-        run: |
-          chmod +x benchmarks/run-benchmarks.sh
-          chmod +x benchmarks/benchmark.sh
-          chmod +x benchmarks/analyze-benchmarks.py
-          chmod +x benchmarks/move-results.sh
-
       - name: Setup benchmark results directory
         run: |
           mkdir -p benchmark-results
@@ -132,9 +124,7 @@ jobs:
           RESULTS_DIR: benchmark-results
 
       - name: Generate PR benchmark summary
-        run: |
-          chmod +x benchmarks/generate-pr-summary.py
-          python3 benchmarks/generate-pr-summary.py
+        run: python3 benchmarks/generate-pr-summary.py
         env:
           GITHUB_SHA: ${{ github.sha }}
           RUNNER_OS: ${{ runner.os }}

benchmarks/README.md

@@ -34,18 +34,22 @@ def parse_latency(latency_str):
     if not latency_str or latency_str == "N/A":
         return 0
 
-    latency_str = str(latency_str).lower()
-    if 'ms' in latency_str:
-        return float(latency_str.replace('ms', ''))
-    elif 's' in latency_str and 'ms' not in latency_str:
-        return float(latency_str.replace('s', '')) * 1000
-    elif 'us' in latency_str:
-        return float(latency_str.replace('us', '')) / 1000
-    else:
-        try:
+    latency_str = str(latency_str).lower().strip()
+    
+    try:
+        # Check for most specific patterns first
+        if 'ms' in latency_str:
+            return float(latency_str.replace('ms', ''))
+        elif 'us' in latency_str:
+            return float(latency_str.replace('us', '')) / 1000
+        elif latency_str.endswith('s'):
+            # Only for pure seconds (not microseconds or milliseconds)
+            return float(latency_str.replace('s', '')) * 1000
+        else:
+            # Try to parse as plain number (assume milliseconds)
             return float(latency_str)
-        except:
-            return 0
+    except (ValueError, TypeError):
+        return 0
 
 def format_rps(rps_str):
     """Format RPS for display"""