refactor: unify summary table naming and fix review issues

Author: zzhfz

dashboard/pages/communication.py

@@ -12,9 +12,9 @@
     plot_metric_vs_size,
     plot_comparison_matrix,
     create_gauge_chart,
-    create_summary_table,
     plot_timeseries_auto,
     create_summary_table_infer,
+    create_summary_table_comm,
 )
 
 init_page("通信测试分析 | InfiniMetrics", "🔗")
@@ -293,7 +293,7 @@ def main():
                     f"{run['operation']} ({run['device_used']} GPUs) - 配置详情"
                 ):
                     # Create summary table
-                    summary_df = create_summary_table(run.get("data", {}))
+                    summary_df = create_summary_table_comm(run.get("data", {}))
                     st.dataframe(
                         summary_df,
                         use_container_width=True,

dashboard/pages/operator.py

@@ -95,7 +95,7 @@ def main():
                         st.write(f"- 加速卡: {acc.get('model', 'Unknown')}")
                         st.write(f"- 显存: {acc.get('memory_gb_per_card', '?')} GB")
                         st.write(f"- CUDA版本: {acc.get('cuda', 'Unknown')}")
-                    except:
+                    except Exception:
                         st.json(env)
 
     with tab3:

dashboard/utils/visualizations/__init__.py

@@ -31,9 +31,9 @@
 
 # Summary tables
 from .summary_tables import (
-    create_comm_summary_table,
-    create_infer_summary_table,
-    create_ops_summary_table,
+    create_summary_table_comm,
+    create_summary_table_infer,
+    create_summary_table_ops,
 )
 
 # Hardware functions
@@ -57,11 +57,6 @@
     render_config_details,
 )
 
-# Backward-compatible aliases
-create_summary_table = create_comm_summary_table
-create_summary_table_infer = create_infer_summary_table
-create_summary_table_ops = create_ops_summary_table
-
 __all__ = [
     # Base
     "create_gauge_chart",
@@ -73,10 +68,7 @@
     "render_inference_metrics",
     "render_memory_gauge",
     # Summary tables
-    "create_comm_summary_table",
-    "create_infer_summary_table",
-    "create_ops_summary_table",
-    "create_summary_table",
+    "create_summary_table_comm",
     "create_summary_table_infer",
     "create_summary_table_ops",
     # Hardware

dashboard/utils/visualizations/operator.py

@@ -4,7 +4,6 @@
 import streamlit as st
 import pandas as pd
 import plotly.graph_objects as go
-import plotly.express as px
 
 from .base import create_gauge_chart
 

dashboard/utils/visualizations/summary_tables.py

@@ -5,7 +5,7 @@
 from typing import Dict, Any
 
 
-def create_comm_summary_table(test_result: Dict[str, Any]) -> pd.DataFrame:
+def create_summary_table_comm(test_result: Dict[str, Any]) -> pd.DataFrame:
     """Create summary table for communication tests."""
     summary_data = []
 
@@ -86,7 +86,7 @@ def create_comm_summary_table(test_result: Dict[str, Any]) -> pd.DataFrame:
     return pd.DataFrame(summary_data)
 
 
-def create_infer_summary_table(test_result: dict) -> pd.DataFrame:
+def create_summary_table_infer(test_result: dict) -> pd.DataFrame:
     """Create summary table for inference tests."""
     rows = []
 
@@ -141,7 +141,7 @@ def create_infer_summary_table(test_result: dict) -> pd.DataFrame:
     return pd.DataFrame(rows)
 
 
-def create_ops_summary_table(test_result: dict) -> pd.DataFrame:
+def create_summary_table_ops(test_result: dict) -> pd.DataFrame:
     """Create summary table for operator tests."""
     rows = []
     cfg = test_result.get("config", {})