apply formatting

Author: pbalcer

devops/scripts/benchmarks/benches/base.py

@@ -110,7 +110,7 @@ def unstable(self) -> str:
 
     def get_metadata(self) -> BenchmarkMetadata:
         return BenchmarkMetadata(
-            type='benchmark',
+            type="benchmark",
             description=self.description(),
             notes=self.notes(),
             unstable=self.unstable(),

devops/scripts/benchmarks/benches/compute.py

@@ -56,21 +56,20 @@ def setup(self):
 
     def additionalMetadata(self) -> dict[str, BenchmarkMetadata]:
         return {
-            "SubmitKernel" : BenchmarkMetadata(
+            "SubmitKernel": BenchmarkMetadata(
                 type="group",
                 description="Measures CPU time overhead of submitting kernels through different APIs.",
                 notes="Each layer builds on top of the previous layer, adding functionality and overhead. "
-                      "The first layer is the Level Zero API, the second is the Unified Runtime API, and the third is the SYCL API. "
-                      "The UR v2 adapter noticeably reduces UR layer overhead, also improving SYCL performance."
-                      "Work is ongoing to reduce the overhead of the SYCL API",
+                "The first layer is the Level Zero API, the second is the Unified Runtime API, and the third is the SYCL API. "
+                "The UR v2 adapter noticeably reduces UR layer overhead, also improving SYCL performance."
+                "Work is ongoing to reduce the overhead of the SYCL API",
             ),
-            "SinKernelGraph" : BenchmarkMetadata(
+            "SinKernelGraph": BenchmarkMetadata(
                 type="group",
                 unstable="This benchmark combines both eager and graph execution, and may not be representative of real use cases.",
             ),
         }
 
-
     def benchmarks(self) -> list[Benchmark]:
         if options.sycl is None:
             return []

devops/scripts/benchmarks/benches/test.py

@@ -39,18 +39,20 @@ def benchmarks(self) -> list[Benchmark]:
                 value = base_value * value_multiplier
                 diff = base_diff * value_multiplier
 
-                result.append(TestBench(self, name, value, diff, group, notes, unstable))
+                result.append(
+                    TestBench(self, name, value, diff, group, notes, unstable)
+                )
 
         return result
 
     def additionalMetadata(self) -> dict[str, BenchmarkMetadata]:
         return {
-            "Foo Group" : BenchmarkMetadata(
+            "Foo Group": BenchmarkMetadata(
                 type="group",
                 description="This is a test benchmark for Foo Group.",
                 notes="This is a test note for Foo Group.",
             ),
-            "Bar Group" : BenchmarkMetadata(
+            "Bar Group": BenchmarkMetadata(
                 type="group",
                 description="This is a test benchmark for Bar Group.",
                 unstable="This is an unstable note for Bar Group.",

devops/scripts/benchmarks/html/scripts.js

@@ -205,20 +205,20 @@ function createChartContainer(data, canvasId, type) {
     container.className = 'chart-container';
     container.setAttribute('data-label', data.label);
     container.setAttribute('data-suite', data.suite);
-    
+
     // Check if this benchmark is marked as unstable
     const metadata = metadataForLabel(data.label, type);
     if (metadata && metadata.unstable) {
         container.setAttribute('data-unstable', 'true');
-        
+
         // Add unstable warning
         const unstableWarning = document.createElement('div');
         unstableWarning.className = 'benchmark-unstable';
         unstableWarning.textContent = metadata.unstable;
         unstableWarning.style.display = showUnstable ? 'block' : 'none';
         container.appendChild(unstableWarning);
     }
-    
+
     // Add notes if present
     if (metadata && metadata.notes) {
         const noteElement = document.createElement('div');
@@ -227,7 +227,7 @@ function createChartContainer(data, canvasId, type) {
         noteElement.style.display = showNotes ? 'block' : 'none';
         container.appendChild(noteElement);
     }
-    
+
     // Add description if present in metadata, but only for groups
     if (metadata && metadata.description && metadata.type === "group") {
         const descElement = document.createElement('div');
@@ -274,7 +274,7 @@ function metadataForLabel(label, type) {
             return metadata;
         }
     }
-    
+
     return null;
 }
 
@@ -305,27 +305,27 @@ function generateExtraInfo(latestRunsLookup, data) {
     return labels.map(label => {
         const metadata = metadataForLabel(label);
         const latestRun = latestRunsLookup.get(label);
-        
+
         let html = '<div class="extra-info-entry">';
-        
+
         if (metadata) {
             html += `<strong>${label}:</strong> ${formatCommand(latestRun.result)}<br>`;
-            
+
             if (metadata.description) {
                 html += `<em>Description:</em> ${metadata.description}`;
             }
-            
+
             if (metadata.notes) {
                 html += `<br><em>Notes:</em> <span class="note-text">${metadata.notes}</span>`;
             }
-            
+
             if (metadata.unstable) {
                 html += `<br><em class="unstable-warning">⚠️ Unstable:</em> <span class="unstable-text">${metadata.unstable}</span>`;
             }
         } else {
             html += `<strong>${label}:</strong> No data available`;
         }
-        
+
         html += '</div>';
         return html;
     }).join('');
@@ -407,9 +407,9 @@ function filterCharts() {
         const isUnstable = container.getAttribute('data-unstable') === 'true';
 
         // Hide unstable benchmarks if showUnstable is false
-        const shouldShow = regex.test(label) && 
-                          activeSuites.includes(suite) && 
-                          (showUnstable || !isUnstable);
+        const shouldShow = regex.test(label) &&
+            activeSuites.includes(suite) &&
+            (showUnstable || !isUnstable);
 
         container.style.display = shouldShow ? '' : 'none';
     });
@@ -471,7 +471,7 @@ function processBarChartsData(benchmarkRuns) {
             if (!groupedResults[result.explicit_group]) {
                 // Look up group metadata
                 const groupMetadata = metadataForLabel(result.explicit_group);
-                
+
                 groupedResults[result.explicit_group] = {
                     label: result.explicit_group,
                     suite: result.suite,
@@ -550,7 +550,7 @@ function setupSuiteFilters() {
 function setupToggles() {
     const notesToggle = document.getElementById('show-notes');
     const unstableToggle = document.getElementById('show-unstable');
-    
+
     notesToggle.addEventListener('change', function() {
         showNotes = this.checked;
         // Update all note elements visibility
@@ -559,7 +559,7 @@ function setupToggles() {
         });
         filterCharts();
     });
-    
+
     unstableToggle.addEventListener('change', function() {
         showUnstable = this.checked;
         // Update all unstable warning elements visibility
@@ -568,16 +568,16 @@ function setupToggles() {
         });
         filterCharts();
     });
-    
+
     // Initialize from URL params if present
     const notesParam = getQueryParam('notes');
     const unstableParam = getQueryParam('unstable');
-    
+
     if (notesParam !== null) {
         showNotes = notesParam === 'true';
         notesToggle.checked = showNotes;
     }
-    
+
     if (unstableParam !== null) {
         showUnstable = unstableParam === 'true';
         unstableToggle.checked = showUnstable;

devops/scripts/benchmarks/utils/result.py

@@ -47,7 +47,7 @@ class BenchmarkRun:
 @dataclass_json
 @dataclass
 class BenchmarkMetadata:
-    type: str = 'benchmark' # or 'group'
+    type: str = "benchmark"  # or 'group'
     description: Optional[str] = None
     notes: Optional[str] = None
     unstable: Optional[str] = None