feat: Exclude error requests from latency stats and plots

index.html

@@ -694,16 +694,23 @@
                 const successRate = totalRequests > 0 ? ((successfulRequests / totalRequests) * 100).toFixed(1) : 0;
 
                 const validLatencies = rows
+                    .map(r => r.duration_ms)
+                    .filter(d => d !== null && d >= 0); // Keep this filter for null/negative durations
+
+                // Filter for successful requests *before* calculating average latency
+                const successfulRows = rows.filter(r => r.response_status_code === 200);
+                const successfulLatencies = successfulRows
                     .map(r => r.duration_ms)
                     .filter(d => d !== null && d >= 0);
-                const avgLatency = validLatencies.length > 0
-                    ? (validLatencies.reduce((a, b) => a + b, 0) / validLatencies.length).toFixed(0)
+
+                const avgLatency = successfulLatencies.length > 0
+                    ? (successfulLatencies.reduce((a, b) => a + b, 0) / successfulLatencies.length).toFixed(0)
                     : 0;
 
-                // Calculate median latency per provider for the filtered data
+                // Calculate median latency per provider using only successful requests
                 const latencyByProvider = {};
-                 rows.forEach(row => {
-                    if (row.duration_ms !== null && row.duration_ms >= 0) {
+                 successfulRows.forEach(row => { // Iterate over successful rows only
+                    if (row.duration_ms !== null && row.duration_ms >= 0) { // Still check for valid duration
                         if (!latencyByProvider[row.provider_name]) {
                             latencyByProvider[row.provider_name] = [];
                         }
@@ -740,9 +747,10 @@
              function createLatencyByProviderPlot(rows, selectedModelId) {
                 const titleContext = selectedModelId === 'all' ? '' : `for ${selectedModelId.split('/').pop()}`;
                 const dataByProvider = {};
-                rows.forEach(row => {
+                // Filter for successful requests before processing
+                rows.filter(r => r.response_status_code === 200).forEach(row => {
                     if (!dataByProvider[row.provider_name]) dataByProvider[row.provider_name] = [];
-                    if (row.duration_ms !== null && row.duration_ms >= 0) {
+                    if (row.duration_ms !== null && row.duration_ms >= 0) { // Still check for valid duration
                          dataByProvider[row.provider_name].push(row.duration_ms);
                     }
                 });
@@ -802,10 +810,11 @@
 
             function createLatencyByModelPlot(rows) { // Only shown for 'all'
                 const dataByModel = {};
-                 rows.forEach(row => {
+                 // Filter for successful requests before processing
+                 rows.filter(r => r.response_status_code === 200).forEach(row => {
                     const model = row.model_id;
                     if (!dataByModel[model]) dataByModel[model] = [];
-                    if (row.duration_ms !== null && row.duration_ms >= 0) {
+                    if (row.duration_ms !== null && row.duration_ms >= 0) { // Still check for valid duration
                          dataByModel[model].push(row.duration_ms);
                     }
                 });
@@ -866,8 +875,9 @@
                 const allProviders = new Set();
                 const allModels = new Set();
 
-                rows.forEach(row => {
-                    if (row.duration_ms !== null && row.duration_ms >= 0) {
+                // Filter for successful requests before processing
+                rows.filter(r => r.response_status_code === 200).forEach(row => {
+                    if (row.duration_ms !== null && row.duration_ms >= 0) { // Still check for valid duration
                         const provider = row.provider_name;
                         const model = row.model_id;
                         allProviders.add(provider);
@@ -923,9 +933,16 @@
                     const provider = row.provider_name;
                     if (!providerStats[provider]) providerStats[provider] = { total: 0, success: 0, errors: 0, latencies: [] };
                     providerStats[provider].total++;
-                    if (row.response_status_code === 200) providerStats[provider].success++;
-                    else if (row.response_status_code !== null) providerStats[provider].errors++;
-                    if (row.duration_ms !== null && row.duration_ms >= 0) providerStats[provider].latencies.push(row.duration_ms);
+                    if (row.response_status_code === 200) {
+                        providerStats[provider].success++;
+                        // Only add latency for successful requests with valid duration
+                        if (row.duration_ms !== null && row.duration_ms >= 0) {
+                            providerStats[provider].latencies.push(row.duration_ms);
+                        }
+                    } else if (row.response_status_code !== null) {
+                         providerStats[provider].errors++;
+                    }
+                    // Note: We no longer add latency if the request was not successful (status != 200)
                 });
 
                 // Create Header (only if it doesn't exist)