make comparison page more performant

- only pre-check known exes
- query only checked benchmarks and exes

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: mattip

codespeed/static/js/comparison.js

@@ -64,14 +64,22 @@ function refreshContent() {
 function savedata(data) {
   if (data.error !== "None") {
     var h = $("#content").height();//get height for error message
-    $("#cplot").html(getLoadText(data.error, h));
+    $("#plotwrapper").html(getLoadText(data.error, h));
     return 1;
   }
   delete data.error;
   compdata = data;
   refreshContent();
 }
 
+function loadData() {
+  var conf = getConfiguration();
+  if (!conf.exe || !conf.ben) { return; }
+  var h = $("#content").height();
+  $("#plotwrapper").html(getLoadText("Loading...", h));
+  $.getJSON("json/", {exe: conf.exe, ben: conf.ben}, savedata);
+}
+
 function abortRender(plotid, message) {
     $("#" + plotid).css("border", "dashed 1px grey");
     $("#" + plotid).css("padding", "1em");
@@ -411,19 +419,18 @@ function init(defaults) {
         sel.filter("[value='" + env + "']").prop('checked', true);
     });
 
-    $("#chart_type, #baseline, #direction, input[name='executables']," +
-      "input[name='benchmarks'], input[name='environments']").change(refreshContent);
+    // Re-fetch when exe or benchmark selection changes
+    $("input[name='executables'], input[name='benchmarks']").change(loadData);
+    $('.checkall, .uncheckall').click(loadData);
 
-    $('.checkall, .uncheckall').click(refreshContent);
+    // Re-render without re-fetching for other controls
+    $("#chart_type, #baseline, #direction, input[name='environments']").change(refreshContent);
 
     $.ajaxSetup ({
       cache: false
     });
 
-    // Get comparison data
-    var h = $("#content").height();//get height for loading text
-    $("#cplot").html(getLoadText("Loading...", h));
-    $.getJSON("json/", savedata);
+    loadData();
 
     $("#permalink").click(function() {
         window.location = "?" + $.param(getConfiguration());

codespeed/views.py

@@ -196,13 +196,32 @@ def gethistoricaldata(request):
 @require_GET
 def getcomparisondata(request):
     executables, exekeys = getcomparisonexes()
+
+    requested_exes = None
+    if 'exe' in request.GET:
+        requested_exes = set(
+            i for i in request.GET['exe'].split(",") if i and i in exekeys
+        )
+
     benchmarks = Benchmark.objects.all()
+    if 'ben' in request.GET:
+        bench_ids = set()
+        for i in request.GET['ben'].split(","):
+            try:
+                bench_ids.add(int(i))
+            except ValueError:
+                pass
+        if bench_ids:
+            benchmarks = benchmarks.filter(id__in=bench_ids)
+
     environments = Environment.objects.all()
 
     compdata = {}
     compdata['error'] = "Unknown error"
     for proj in executables:
         for exe in executables[proj]:
+            if requested_exes is not None and exe['key'] not in requested_exes:
+                continue
             compdata[exe['key']] = {}
             for env in environments:
                 compdata[exe['key']][env.id] = {}
@@ -271,6 +290,30 @@ def comparison(request):
             except Revision.DoesNotExist:
                 #TODO: log
                 pass
+    if not checkedexecutables:
+        if hasattr(settings, 'DEF_EXECUTABLES') and settings.DEF_EXECUTABLES:
+            for exe_spec in settings.DEF_EXECUTABLES:
+                try:
+                    proj = Project.objects.get(name=exe_spec['project'])
+                    exe = Executable.objects.get(name=exe_spec['name'], project=proj)
+                    for key in exekeys:
+                        if key.startswith(str(exe.id) + "+L+"):
+                            checkedexecutables.append(key)
+                            break
+                except (Executable.DoesNotExist, Project.DoesNotExist):
+                    pass
+        if (not checkedexecutables and
+                hasattr(settings, 'DEF_BASELINES') and settings.DEF_BASELINES):
+            baselines = getbaselineexecutables()
+            for base_spec in settings.DEF_BASELINES:
+                for base in baselines:
+                    if base['key'] == "none":
+                        continue
+                    if (base['executable'].name == base_spec['executable'] and
+                            base['revision'].commitid == base_spec['revision']):
+                        if base['key'] in exekeys:
+                            checkedexecutables.append(base['key'])
+                        break
     if not checkedexecutables:
         checkedexecutables = exekeys