copy numpy array

Author: FBruzzesi

packages/python/plotly/plotly/express/trendline_functions/__init__.py

@@ -111,17 +111,20 @@ def lowess(trendline_options, x_raw, x, y, x_label, y_label, non_missing):
 
 
 def _pandas(mode, trendline_options, x_raw, y, non_missing):
+    import numpy as np
+
+    try:
+        import pandas as pd
+    except ImportError:
+        msg = "Trendline requires pandas to be installed"
+        raise ImportError(msg)
+
     modes = dict(rolling="Rolling", ewm="Exponentially Weighted", expanding="Expanding")
     trendline_options = trendline_options.copy()
     function_name = trendline_options.pop("function", "mean")
     function_args = trendline_options.pop("function_args", dict())
 
-    pd = nw.dependencies.get_pandas()
-    if pd is None:
-        msg = "Trendline requires pandas to be installed"
-        raise ImportError(msg)
-
-    series = pd.Series(y, index=x_raw.to_pandas())
+    series = pd.Series(np.copy(y), index=x_raw.to_pandas())
 
     # TODO: If narwhals were to support rolling, ewm and expanding then we could go around these
     agg = getattr(series, mode)  # e.g. series.rolling

packages/python/plotly/plotly/figure_factory/_hexbin_mapbox.py

@@ -427,7 +427,7 @@ def create_hexbin_mapbox(
         agg_data_frame_list.append(
             nw.from_dict(
                 {
-                    "frame": [int(key[0])] * len(hexagons_ids),
+                    "frame": [key[0]] * len(hexagons_ids),
                     "locations": hexagons_ids,
                     "color": aggregated_value,
                 },

packages/python/plotly/plotly/tests/test_optional/test_figure_factory/test_figure_factory.py

@@ -4474,7 +4474,7 @@ def test_build_dataframe(self):
         lon = np.random.randn(N)
         color = np.ones(N)
         frame = np.random.randint(0, n_frames, N)
-        df = pd.DataFrame(
+        df = pd.DataFrame(  # TODO: Test other constructors?
             np.c_[lat, lon, color, frame],
             columns=["Latitude", "Longitude", "Metric", "Frame"],
         )