Add summary table ID

Author: ElliottKasoar

mlip_testing/app/build_app.py

@@ -76,7 +76,7 @@ def build_summary_table(tables: dict[str, DataTable]) -> DataTable:
     columns_headers = ("MLIP",) + tuple(tables.keys()) + ("Score", "Rank")
     columns = [{"name": headers, "id": headers} for headers in columns_headers]
 
-    return DataTable(data=data, columns=columns)
+    return DataTable(data=data, columns=columns, id="summary-table")
 
 
 def build_tabs(
@@ -152,6 +152,6 @@ def build_full_app(full_app: Dash) -> None:
         header="Benchmark weights",
         labels=tables.keys(),
         ids=tables.keys(),
-        reset_prefix="summary",
+        table_prefix="summary",
     )
     build_tabs(full_app, layouts, summary_table, weight_components)

mlip_testing/app/utils/build.py

@@ -63,16 +63,16 @@ def build_slider(
     )
 
 
-def register_weight_callbacks(weight_prefix: str, reset_prefix: str) -> None:
+def register_weight_callbacks(weight_prefix: str, table_prefix: str) -> None:
     """
     Register all callbacks for weight inputs.
 
     Parameters
     ----------
     weight_prefix
         Prefix for weight component IDs.
-    reset_prefix
-        Prefix for reset component ID.
+    table_prefix
+        Prefix for table and reset component ID.
     """
 
     # Callback to sync weights between slider, text, reset, and Store
@@ -83,7 +83,7 @@ def register_weight_callbacks(weight_prefix: str, reset_prefix: str) -> None:
         Input(f"{weight_prefix}-input", "value"),
         Input(f"{weight_prefix}-slider", "value"),
         Input(f"{weight_prefix}-weight-store", "data"),
-        Input(f"{reset_prefix}-reset-weights-button", "n_clicks"),
+        Input(f"{table_prefix}-reset-weights-button", "n_clicks"),
         prevent_initial_call=False,
     )
     def store_slider_value(
@@ -116,15 +116,15 @@ def store_slider_value(
             weight = input_value
         elif trigger_id == f"{weight_prefix}-slider":
             weight = slider_value
-        elif trigger_id == f"{reset_prefix}-reset-weights-button":
+        elif trigger_id == f"{table_prefix}-reset-weights-button":
             weight = 1
         else:
             raise ValueError("Invalid trigger. trigger_id: ", trigger_id)
         return weight, weight, weight
 
 
 def build_weight_components(
-    header: str, labels: list[str], ids: list[str], reset_prefix: str
+    header: str, labels: list[str], ids: list[str], table_prefix: str
 ) -> Div:
     """
     Build weight sliders, text boxes and reset button.
@@ -137,8 +137,8 @@ def build_weight_components(
         Names for each weight slider.
     ids
         Prefix for slider and input box IDs.
-    reset_prefix
-        Label for reset button used for all weights.
+    table_prefix
+        Label for table and reset button used for all weights.
 
     Returns
     -------
@@ -160,7 +160,7 @@ def build_weight_components(
     layout.append(
         Button(
             "Reset Weights",
-            id=f"{reset_prefix}-reset-weights-button",
+            id=f"{table_prefix}-reset-weights-button",
             n_clicks=0,
             style={"marginTop": "20px"},
         ),
@@ -170,7 +170,7 @@ def build_weight_components(
         layout.append(
             Store(id=f"{weight_prefix}-weight-store", storage_type="session", data=1.0)
         )
-        register_weight_callbacks(weight_prefix, reset_prefix)
+        register_weight_callbacks(weight_prefix, table_prefix)
 
     return Div(layout)