💨 removing todo & fixing small bug

Author: jonasvdd

plotly_resampler/figure_resampler/figure_resampler_interface.py

@@ -236,8 +236,7 @@ def _get_current_graph(self) -> dict:
             "data": [
                 {
                     k: copy(trace[k])
-                    # TODO: why not "text" as well? -> we can use _hf_data_container.fields then
-                    for k in set(trace.keys()).difference({"x", "y", "hovertext"})
+                    for k in set(trace.keys()).difference({_hf_data_container.fields})
                 }
                 for trace in self._data
             ],
@@ -613,7 +612,9 @@ def _parse_get_trace_props(
         hf_y = (
             trace["y"]
             if hasattr(trace, "y") and hf_y is None
-            else hf_y.values if isinstance(hf_y, (pd.Series, pd.Index)) else hf_y
+            else hf_y.values
+            if isinstance(hf_y, (pd.Series, pd.Index))
+            else hf_y
         )
         # NOTE: the if will not be triggered for a categorical series its values
         if not hasattr(hf_y, "dtype"):
@@ -1019,7 +1020,6 @@ def add_trace(
             hf_properties = {}
             for _, _, hf_param_name in DOWNSAMPLABLE_PROPERTIES:
                 if hf_param_name in trace_kwargs:
-                    # TODO -> hf_param name
                     hf_properties[hf_param_name] = trace_kwargs.pop(hf_param_name)
 
             # construct the hf_data_container