dearwolf/aider
1
0
Fork 0
mirror of https://github.com/Aider-AI/aider.git synced 2025-06-01 02:05:00 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

refactor: Use constant for max display cost cap in leaderboard table

Browse source
This commit is contained in:
Paul Gauthier (aider) 2025-04-16 19:00:18 -07:00
parent 80909e17c7
commit ed14be4e70
1 changed files with 5 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

9
aider/website/_includes/leaderboard_table.js
View file

@ -1,6 +1,7 @@
document.addEventListener('DOMContentLoaded', function() { document.addEventListener('DOMContentLoaded', function() {
let currentMode = 'view'; // 'view', 'select', 'detail' let currentMode = 'view'; // 'view', 'select', 'detail'
let selectedRows = new Set(); // Store indices of selected rows let selectedRows = new Set(); // Store indices of selected rows
const MAX_DISPLAY_COST_CAP = 50; // Define the constant here
const allMainRows = document.querySelectorAll('tr[id^="main-row-"]'); const allMainRows = document.querySelectorAll('tr[id^="main-row-"]');
const allDetailsRows = document.querySelectorAll('tr[id^="details-"]'); const allDetailsRows = document.querySelectorAll('tr[id^="details-"]');
@ -219,7 +220,7 @@ document.addEventListener('DOMContentLoaded', function() {
// Function to calculate the appropriate max display cost based on visible/selected entries // Function to calculate the appropriate max display cost based on visible/selected entries
function calculateDisplayMaxCost() { function calculateDisplayMaxCost() {
// Get the appropriate set of rows based on the current mode and selection state // Get the appropriate set of rows based on the current mode and selection state
let rowsToConsider; let rowsToConsider;
if (currentMode === 'view' && selectedRows.size > 0) { if (currentMode === 'view' && selectedRows.size > 0) {
// In view mode with selections, only consider selected rows // In view mode with selections, only consider selected rows
@ -242,8 +243,8 @@ document.addEventListener('DOMContentLoaded', function() {
} }
}); });
// Cap at 50 if any entries exceed that amount, otherwise use actual max // Cap at MAX_DISPLAY_COST_CAP if any entries exceed that amount, otherwise use actual max
return maxCost > 50 ? 50 : Math.max(1, maxCost); // Ensure at least 1 to avoid division by zero return maxCost > MAX_DISPLAY_COST_CAP ? MAX_DISPLAY_COST_CAP : Math.max(1, maxCost); // Ensure at least 1 to avoid division by zero
} }
// Process cost bars with dynamic scale // Process cost bars with dynamic scale
@ -264,7 +265,7 @@ document.addEventListener('DOMContentLoaded', function() {
bar.style.width = Math.max(0, Math.min(100, percent)) + '%'; bar.style.width = Math.max(0, Math.min(100, percent)) + '%';
// Mark bars that exceed the limit (only if our display max is capped at 50) // Mark bars that exceed the limit (only if our display max is capped at 50)
if (currentMaxDisplayCost === 50 && cost > 50) { if (currentMaxDisplayCost === MAX_DISPLAY_COST_CAP && cost > MAX_DISPLAY_COST_CAP) {
// Create a darker section at the end with diagonal stripes // Create a darker section at the end with diagonal stripes
const darkSection = document.createElement('div'); const darkSection = document.createElement('div');
darkSection.className = 'bar-viz dark-section'; darkSection.className = 'bar-viz dark-section';