Merge branch 'main' into validate-colors-mutating-input

Author: camdecoster

CHANGELOG.md

@@ -16,13 +16,16 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 - Fix `mpl_to_plotly` dropping matplotlib stairs plots by rendering `StepPatch` artists as step lines or filled step areas from their path vertices and edge style [[#5698](https://github.com/plotly/plotly.py/pull/5698)], with thanks to @robertoffmoura for the contribution!
 - Fix `mpl_to_plotly` showing unwanted legends with placeholder or internal label entries (`_nolegend_`, `_childN`) by defaulting `layout.showlegend=False` unless the matplotlib figure explicitly defines a legend [[#5699](https://github.com/plotly/plotly.py/pull/5699)], with thanks to @robertoffmoura for the contribution!
 - Fix `mpl_to_plotly` crashing on figures with fully transparent or alpha-carrying colors, such as boxplot outlier markers (`facecolor="none"`), rgba line colors, and `color="none"` text and axis labels; all matplotlib colors are now exported through a single helper that maps `"none"` to transparent rgba and keeps or overrides alpha as matplotlib does [[#5700](https://github.com/plotly/plotly.py/pull/5700)], with thanks to @robertoffmoura for the contribution!
+- Fix serialization of box/lasso select data for figures with multiple subplots [[#5710](https://github.com/plotly/plotly.py/pull/5710)], with thanks to @flpi-sun for the contribution!
 - Export tick label colors to the plotly tickfont in `mpl_to_plotly` [[#5716](https://github.com/plotly/plotly.py/pull/5716)], with thanks to @robertoffmoura for the contribution!
 - Fix `convert_dash` mapping dotted lines (`linestyle=':'`) to invalid `'circle'` in Plotly, correctly converting them to `'dot'` and preserving custom dash patterns as pixel lists, [[#5719](https://github.com/plotly/plotly.py/pull/5719)] with thanks to @robertoffmoura for the contribution!
 - Fix Plotly Express mutating lists passed to the `x` or `y` arguments in wide mode [[#5727](https://github.com/plotly/plotly.py/pull/5727)], with thanks to @cpruijsen for the contribution!
 - Fix `px.sunburst`, `px.treemap` and `px.icicle` listing sectors in a different order on every run when `path` is used with a Polars DataFrame; sectors now follow their order of first appearance for all dataframe backends [[#5766](https://github.com/plotly/plotly.py/pull/5766)], with thanks to @Irahan2 for the contribution!
 - Fix `mpl_to_plotly` conversion of matplotlib contour lines and line collections: close contour rings ending with `Z` codes, support dash styles, convert date x-axes, separate disjoint subpaths with `None` separators, hide line collection traces from the legend by default, and group consecutive same-style lines into single traces [[#5770](https://github.com/plotly/plotly.py/pull/5770)], with thanks to @robertoffmoura for the contribution!
+- Fix `plotly.io.write_images` ignoring `layout.width`/`layout.height` (and the template's) and exporting every figure at the default 700x500 size; it now uses the same width/height priority as `write_image` and `to_image` [[#5784](https://github.com/plotly/plotly.py/pull/5784)], with thanks to @breken-ai for the contribution!
 - Fix color utilities mutating the list of colors passed to them [[#5836](https://github.com/plotly/plotly.py/pull/5836)], with thanks to @Enesuygurs for the contribution!
 
+
 ## [7.1.0] - 2026-09-15
 
 ### Added

doc/python/3d-mesh.md

@@ -42,7 +42,7 @@ import plotly.graph_objects as go
 import numpy as np
 
 # Download data set from plotly repo
-pts = np.loadtxt(np.DataSource().open('https://raw.githubusercontent.com/plotly/datasets/master/mesh_dataset.txt'))
+pts = np.loadtxt(np.lib.npyio.DataSource().open('https://raw.githubusercontent.com/plotly/datasets/master/mesh_dataset.txt'))
 x, y, z = pts.T
 
 fig = go.Figure(data=[go.Mesh3d(x=x, y=y, z=z, color='lightpink', opacity=0.50)])
@@ -58,7 +58,7 @@ The `alphahull` parameter sets the shape of the mesh. If the value is -1 (defaul
 import plotly.graph_objects as go
 import numpy as np
 
-pts = np.loadtxt(np.DataSource().open('https://raw.githubusercontent.com/plotly/datasets/master/mesh_dataset.txt'))
+pts = np.loadtxt(np.lib.npyio.DataSource().open('https://raw.githubusercontent.com/plotly/datasets/master/mesh_dataset.txt'))
 x, y, z = pts.T
 
 fig = go.Figure(data=[go.Mesh3d(x=x, y=y, z=z,

doc/python/colorscales.md

@@ -443,7 +443,7 @@ fig.show()
 ```python
 import plotly.graph_objects as go
 
-import urllib
+import urllib.request
 import json
 
 response = urllib.request.urlopen(
@@ -535,7 +535,7 @@ Like axes, you can customize the color bar ticks, labels, and values with `ticks
 ```python
 import plotly.graph_objects as go
 
-import urllib
+import urllib.request
 import json
 
 # Load heatmap data
@@ -570,7 +570,7 @@ By default, color bars are displayed vertically. You can change a color bar to b
 ```python
 import plotly.graph_objects as go
 
-import urllib
+import urllib.request
 import json
 
 # Load heatmap data
@@ -647,7 +647,7 @@ See the positioning section of [the figure data structure page](/python/figure-s
 ```python
 import plotly.graph_objects as go
 
-import urllib
+import urllib.request
 import json
 
 # Load heatmap data

doc/python/graphing-multiple-chart-types.md

@@ -173,7 +173,7 @@ import plotly.graph_objects as go
 
 # Load data
 import json
-import urllib
+import urllib.request
 
 response = urllib.request.urlopen(
     "https://raw.githubusercontent.com/plotly/datasets/master/steepest.json")

doc/python/sankey-diagram.md

@@ -66,7 +66,7 @@ fig.show()
 
 ```python
 import plotly.graph_objects as go
-import urllib, json
+import urllib.request, json
 
 url = 'https://raw.githubusercontent.com/plotly/plotly.js/master/test/image/mocks/sankey_energy.json'
 response = urllib.request.urlopen(url)
@@ -126,7 +126,7 @@ This example also uses [hovermode](https://plotly.com/python/reference/layout/#l
 
 ```python
 import plotly.graph_objects as go
-import urllib, json
+import urllib.request, json
 
 url = 'https://raw.githubusercontent.com/plotly/plotly.js/master/test/image/mocks/sankey_energy.json'
 response = urllib.request.urlopen(url)

plotly/io/_kaleido.py

@@ -90,6 +90,31 @@ def infer_format(path: Union[Path, None], format: Union[str, None]) -> Union[str
     return format
 
 
+def _resolve_width_height(
+    fig_dict: dict, width: Union[int, None], height: Union[int, None]
+) -> tuple:
+    """
+    Return the image width and height to request from Kaleido, in priority order:
+    the `width`/`height` arguments, then `layout.width`/`layout.height`, then the
+    template's `layout.width`/`layout.height`, then the plotly.io defaults.
+    """
+    layout = fig_dict.get("layout", {})
+    template_layout = layout.get("template", {}).get("layout", {})
+    width = (
+        width
+        or layout.get("width")
+        or template_layout.get("width")
+        or defaults.default_width
+    )
+    height = (
+        height
+        or layout.get("height")
+        or template_layout.get("height")
+        or defaults.default_height
+    )
+    return width, height
+
+
 def to_image(
     fig: Union[dict, plotly.graph_objects.Figure],
     format: Union[str, None] = None,
@@ -163,24 +188,7 @@ def to_image(
         if defaults.headers:
             kopts["headers"] = defaults.headers
 
-        width = (
-            width
-            or fig_dict.get("layout", {}).get("width")
-            or fig_dict.get("layout", {})
-            .get("template", {})
-            .get("layout", {})
-            .get("width")
-            or defaults.default_width
-        )
-        height = (
-            height
-            or fig_dict.get("layout", {}).get("height")
-            or fig_dict.get("layout", {})
-            .get("template", {})
-            .get("layout", {})
-            .get("height")
-            or defaults.default_height
-        )
+        width, height = _resolve_width_height(fig_dict, width, height)
 
         img_bytes = kaleido.calc_fig_sync(
             fig_dict,
@@ -405,20 +413,23 @@ def write_images(
     # We call infer_format() here rather than above so that the `file` argument
     # has already been cast to a Path object.
     # Also insert defaults for any missing arguments as needed
-    kaleido_specs = [
-        dict(
-            fig=d["fig"],
-            path=d["file"],
-            opts=dict(
-                format=infer_format(d["file"], d["format"]) or defaults.default_format,
-                width=d["width"] or defaults.default_width,
-                height=d["height"] or defaults.default_height,
-                scale=d["scale"] or defaults.default_scale,
-            ),
-            topojson=defaults.topojson,
+    kaleido_specs = []
+    for d in arg_dicts:
+        width, height = _resolve_width_height(d["fig"], d["width"], d["height"])
+        kaleido_specs.append(
+            dict(
+                fig=d["fig"],
+                path=d["file"],
+                opts=dict(
+                    format=infer_format(d["file"], d["format"])
+                    or defaults.default_format,
+                    width=width,
+                    height=height,
+                    scale=d["scale"] or defaults.default_scale,
+                ),
+                topojson=defaults.topojson,
+            )
         )
-        for d in arg_dicts
-    ]
 
     from kaleido.errors import ChromeNotFoundError
 

tests/test_optional/test_kaleido/test_kaleido.py

@@ -335,3 +335,24 @@ def test_width_height_priority():
     assert height == pio.defaults.default_height, (
         "Default height should be used when no layout or argument"
     )
+
+
+def test_write_images_width_height_priority(tmp_path):
+    """write_images uses the same width/height priority as to_image:
+    arguments > layout.width/height > defaults."""
+    paths = [tmp_path / f"fig{i}.svg" for i in range(3)]
+    pio.write_images(
+        [
+            create_figure(width=800, height=600),
+            create_figure(width=800, height=600),
+            create_figure(),
+        ],
+        paths,
+        width=[None, 1200, None],
+    )
+    dimensions = [parse_svg_dimensions(path.read_bytes()) for path in paths]
+    assert dimensions[0] == (800, 600), "Layout dimensions should be used"
+    assert dimensions[1] == (1200, 600), (
+        "Width argument should override layout, height should use layout"
+    )
+    assert dimensions[2] == (pio.defaults.default_width, pio.defaults.default_height)