[Plot] Revert grid alpha and add TODOs (#67)

Author: CatherineSue

genai_bench/analysis/flexible_plot_report.py

@@ -481,7 +481,7 @@ def _plot_multi_line_metric(
         ax.set_xlabel(plot_spec.x_label or self._generate_label(plot_spec.x_field))
         ax.set_ylabel(plot_spec.y_label or "Value")
         ax.set_title(plot_spec.title)
-        ax.grid(True, alpha=0.1)
+        ax.grid(True, alpha=0.3)
 
         # Position legend outside plot area for multi-line plots to avoid overlap
         ax.legend(bbox_to_anchor=(1.05, 1), loc="upper left", fontsize="small")
@@ -593,7 +593,7 @@ def _save_individual_subplots_multiline(
             # Copy grid
             ax_temp.grid(
                 ax.get_xgridlines()[0].get_visible() if ax.get_xgridlines() else True,
-                alpha=0.1,
+                alpha=0.3,
             )
             ax_temp.minorticks_on()
 

genai_bench/analysis/plot_report.py

@@ -278,6 +278,7 @@ def plot_metrics(
         )
 
 
+# TODO: Remove this function when flexible plot report is fully tested.
 def plot_experiment_data(
     run_data_list: List[Tuple[ExperimentMetadata, ExperimentMetrics]],
     group_key: str,