fix: ignore non-finite values in scatter/plot auto bounding box

scatter(x, y) and the Tables.jl plot(table, :x, :y) extension both
computed boundingbox via extrema(x), which propagates NaN when the
data array contains any NaN/Inf. The classic JSXGraph scatter-via-
curve idiom feeds [x, x, NaN, ...] arrays, which made the auto-axis
path produce an unrenderable boundingbox of (NaN, NaN, NaN, NaN).

Filter non-finite values before extrema and fall back to (-1.0, 1.0)
when no finite values remain. Tests cover both call sites.

Author: s-celles

CHANGELOG.md

@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.5.1] - 2026-05-13
+
+### Fixed
+
+- `scatter(x, y)` and `plot(table, :x, :y)` (Tables.jl extension) no longer produce an invalid `(NaN, NaN, NaN, NaN)` bounding box when the data contains `NaN` or `Inf`. Non-finite values are filtered before computing the auto-padded axis range, with a safe `(-1.0, 1.0)` fallback when no finite values remain. This unblocks the classic JSXGraph scatter-via-curve idiom (`[x, x, NaN, ...]`) when using the auto-axis path.
+
+## [0.5.0] - 2026-05-13
+
 ### Added
 
 - Pluto `@bind` integration via `bindable=true` keyword on `board(...)`. The bound state exposes draggable points (2D + 3D) and sliders by name; helpers `points_xy` and `points_xyz` project to parallel coordinate vectors for spline / regression / array workflows. New optional weak dependency on `AbstractPlutoDingetjes` (loaded automatically by Pluto) routes seed coordinates through Pluto's `published_to_js` channel for efficiency; outside Pluto a JSON-inline fallback keeps the feature working with zero new dependencies. Duplicate element names on a bindable board raise `ArgumentError` at construction. Docs include a runnable Pluto example notebook (`docs/notebooks/pluto_bind_demo.jl`) downloadable from the new **Pluto Integration** page; the page can also embed the rendered notebook via `PlutoStaticHTML.jl` when the docs are built with `BUILD_PLUTO_NOTEBOOKS=true`. Fixes #12.

Project.toml

@@ -1,6 +1,6 @@
 name = "JSXGraph"
 uuid = "97e4c2e4-f0cb-461e-a268-312dec255b76"
-version = "0.5.0"
+version = "0.5.1"
 authors = ["JuliaJSXGraph maintainers"]
 
 [deps]

ext/JSXGraphTablesExt.jl

@@ -117,17 +117,8 @@ function JSXGraph.plot(table, xcol::Symbol, ycol::Symbol;
         throw(ArgumentError("Table columns must not be empty"))
     end
 
-    # Auto-compute axis limits with 10% padding
-    if xlim === nothing
-        xmin, xmax = extrema(x)
-        pad = xmin == xmax ? 1.0 : 0.1 * (xmax - xmin)
-        xlim = (xmin - pad, xmax + pad)
-    end
-    if ylim === nothing
-        ymin, ymax = extrema(y)
-        pad = ymin == ymax ? 1.0 : 0.1 * (ymax - ymin)
-        ylim = (ymin - pad, ymax + pad)
-    end
+    xlim = xlim === nothing ? JSXGraph._padded_extrema(x) : xlim
+    ylim = ylim === nothing ? JSXGraph._padded_extrema(y) : ylim
 
     b = Board(""; xlim=xlim, ylim=ylim)
     # Use curve element with data arrays for a connected line plot

src/composition.jl

@@ -134,6 +134,16 @@ end
 
 # --- Convenience functions (REQ-API-002) ---
 
+# Compute padded (min, max) ignoring non-finite values (NaN, Inf).
+# Returns (-1.0, 1.0) when no finite values remain.
+function _padded_extrema(v::AbstractVector{<:Real})
+    finite = filter(isfinite, v)
+    isempty(finite) && return (-1.0, 1.0)
+    vmin, vmax = extrema(finite)
+    pad = vmin == vmax ? 1.0 : 0.1 * (vmax - vmin)
+    return (vmin - pad, vmax + pad)
+end
+
 """
 $(SIGNATURES)
 
@@ -163,17 +173,8 @@ function scatter(x::AbstractVector{<:Real}, y::AbstractVector{<:Real};
         throw(ArgumentError("x and y must not be empty"))
     end
 
-    # Auto-compute axis limits with 10% padding
-    if xlim === nothing
-        xmin, xmax = extrema(x)
-        pad = xmin == xmax ? 1.0 : 0.1 * (xmax - xmin)
-        xlim = (xmin - pad, xmax + pad)
-    end
-    if ylim === nothing
-        ymin, ymax = extrema(y)
-        pad = ymin == ymax ? 1.0 : 0.1 * (ymax - ymin)
-        ylim = (ymin - pad, ymax + pad)
-    end
+    xlim = xlim === nothing ? _padded_extrema(x) : xlim
+    ylim = ylim === nothing ? _padded_extrema(y) : ylim
 
     board = Board(""; xlim=xlim, ylim=ylim)
     for (xi, yi) in zip(x, y)

test/test_composition.jl

@@ -112,6 +112,16 @@ end
     # HTML output contains points
     html = html_string(scatter([0, 1], [0, 1]))
     @test occursin("create('point'", html)
+
+    # NaN values in data must not poison the auto-computed boundingbox
+    b5 = scatter([1.0, NaN, 3.0], [2.0, NaN, 4.0])
+    bb5 = b5.options["boundingbox"]
+    @test all(isfinite, bb5)
+
+    # All-NaN falls back to a safe default range (not NaN)
+    b6 = scatter([NaN, NaN], [NaN, NaN])
+    bb6 = b6.options["boundingbox"]
+    @test all(isfinite, bb6)
 end
 
 @testset "parametric" begin

test/test_tables.jl

@@ -74,6 +74,13 @@ using Tables
         @test occursin("create('curve'", html)
         # Data arrays appear in JS
         @test occursin("[1.0,2.0,3.0,4.0]", html)
+
+        # NaN-separated data (e.g. JSXGraph scatter trick using a curve)
+        # must not poison the auto-computed boundingbox.
+        nan_data = (x=[1.0, 1.0, NaN, 2.0, 2.0, NaN, 3.0, 3.0, NaN],
+                    y=[1.0, 1.0, NaN, 2.0, 2.0, NaN, 3.0, 3.0, NaN])
+        b6 = plot(nan_data, :x, :y; strokeWidth=3)
+        @test all(isfinite, b6.options["boundingbox"])
     end
 
     @testset "table edge cases" begin