fix makefile lint & typo (#7166)

Author: yangw-dev

aws/lambda/benchmark_regression_summary_report/Makefile

@@ -1,20 +1,20 @@
 all: run-local
 
 clean:
-    rm -rf deployment
-    rm -rf venv
-    rm -rf deployment.zip
+	rm -rf deployment
+	rm -rf venv
+	rm -rf deployment.zip
 
 venv/bin/python:
-    virtualenv venv
-    venv/bin/pip install -r requirements.txt
+	virtualenv venv
+	venv/bin/pip install -r requirements.txt
 
 deployment.zip:
-    mkdir -p deployment
-    cp lambda_function.py ./deployment/.
+	mkdir -p deployment
+	cp lambda_function.py ./deployment/.
 	cp -r common ./deployment/common
-    pip3.10 install -r requirements.txt -t ./deployment/. --platform manylinux2014_x86_64 --only-binary=:all: --implementation cp --python-version 3.10 --upgrade
-    cd ./deployment && zip -q -r ../deployment.zip .
+	pip3.10 install -r requirements.txt -t ./deployment/. --platform manylinux2014_x86_64 --only-binary=:all: --implementation cp --python-version 3.10 --upgrade
+	cd ./deployment && zip -q -r ../deployment.zip .
 
 .PHONY: create-deployment-package
 create-deployment-package: deployment.zip

aws/lambda/benchmark_regression_summary_report/lambda_function.py

@@ -49,9 +49,7 @@ def get_clickhouse_client(
     host: str, user: str, password: str
 ) -> clickhouse_connect.driver.client.Client:
     # for local testing only, disable SSL verification
-    return clickhouse_connect.get_client(
-        host=host, user=user, password=password, secure=True, verify=False
-    )
+    # return clickhouse_connect.get_client(host=host, user=user, password=password, secure=True, verify=False)
     return clickhouse_connect.get_client(
         host=host, user=user, password=password, secure=True
     )

clickhouse_db_schema/benchmark_regression_report/schema.sql

@@ -1,4 +1,4 @@
-CREATE TABLE fortesting.benchmark_regression_report
+CREATE TABLE benchmark.benchmark_regression_report
 (
     `id` UUID DEFAULT generateUUIDv4(),
     `report_id` String,

torchci/lib/benchmark/api_helper/utils.ts

@@ -26,11 +26,11 @@ type Subgroup<T> = {
  */
 type GroupedItem<T> = {
   /** Metadata fields for this group (e.g., dtype, arch, compiler). */
-  group_Info: GroupInfo;
+  group_info: GroupInfo;
 
   /**
    * Rows keyed by a unique identifier string,
-   * derived from a distinct combination of subgroup `group_Info` fields.
+   * derived from a distinct combination of subgroup `group_info` fields.
    * Each entry corresponds to one subgroup that contains data points.
    */
   rows: Record<string, Subgroup<T>>;
@@ -124,7 +124,7 @@ export function groupByBenchmarkData<T>(
   for (const [mainKey, subMap] of groups.entries()) {
     const rowsObj = Object.fromEntries(subMap.entries());
     result.push({
-      group_Info: mainInfo.get(mainKey)!,
+      group_info: mainInfo.get(mainKey)!,
       rows: rowsObj,
     });
   }
@@ -249,7 +249,7 @@ export function to_time_series_data(
   let diffs: any[] = [];
 
   const result = tsd.map((group) => {
-    const group_info = group.group_Info;
+    const group_info = group.group_info;
     const sub_group_data = group.rows;
     // extract the first data point for each sub group
     // since we only have one datapoint for each unique workflow id with the same group info