plotagents 0.7.0 → 0.8.0

package/src/charts/network.js

@@ -276,13 +276,16 @@ function spanningTree(nodes, edges, byId, root) {
   // Hojas numeradas en orden DFS; cada padre queda en el promedio de sus hijos.
   const pos = new Map();
   let leaf = 0;
-  const visit = (id) => {
-    const ch = children.get(id);
-    if (!ch.length) { pos.set(id, leaf++); return; }
-    for (const c of ch) visit(c);
-    pos.set(id, ch.reduce((a, c) => a + pos.get(c), 0) / ch.length);
-  };
-  visit(root.id);
+  // Iterativo: una cadena larga (miles de niveles) desbordaría la pila.
+  const stack = [[root.id, 0]];
+  while (stack.length) {
+    const top = stack[stack.length - 1];
+    const ch = children.get(top[0]);
+    if (!ch.length) { pos.set(top[0], leaf++); stack.pop(); continue; }
+    if (top[1] < ch.length) { stack.push([ch[top[1]++], 0]); continue; }
+    pos.set(top[0], ch.reduce((a, c) => a + pos.get(c), 0) / ch.length);
+    stack.pop();
+  }
   const maxDepth = Math.max(0, ...depth.values());
   const rest = nodes.filter((n) => !depth.has(n.id)).sort(byGroup);
   return { depth, pos, leaves: leaf, maxDepth, rest, treeEdges, children };
@@ -504,7 +507,9 @@ export function network(spec, theme) {
 
   // Foco (clic en un nodo): va al centro y se destacan sus vecinos, con nombre.
   const focusNode = spec.focus !== undefined && spec.focus !== null ? byId.get(String([].concat(spec.focus)[0])) ?? null : null;
-  const hl = focusNode ? new Set([focusNode.id]) : spec.highlight === undefined || spec.highlight === null ? null : new Set([].concat(spec.highlight).map(String));
+  // Un `highlight` que no existe en la red se ignora (si no, todo quedaría atenuado).
+  const hlIds = spec.highlight === undefined || spec.highlight === null ? [] : [].concat(spec.highlight).map(String).filter((id) => byId.has(id));
+  const hl = focusNode ? new Set([focusNode.id]) : hlIds.length ? new Set(hlIds) : null;
   const near = new Set();
   if (hl) { for (const id of hl) near.add(id); for (const e of edges) { if (hl.has(e.s)) near.add(e.t); if (hl.has(e.t)) near.add(e.s); } }
   const on = (n) => !hl || near.has(n.id);

package/src/charts/pointmap.js

@@ -23,6 +23,9 @@ const MAX_LAT = 85.0511287798;
 const TILE = 256;
 const MAX_TILES = 48;
 const clamp = (v, lo, hi) => Math.max(lo, Math.min(hi, v));
+// Mínimo y máximo sin `...` (con cientos de miles de puntos desborda la pila).
+const lo = (a) => a.reduce((m, v) => (v < m ? v : m), Infinity);
+const hi = (a) => a.reduce((m, v) => (v > m ? v : m), -Infinity);
 
 // ── Web Mercator (EPSG:3857) en coordenadas normalizadas [0, 1] ─────────────
 /** Longitud → x de Web Mercator en [0, 1] (0 = 180° O). */
@@ -235,10 +238,10 @@ export function pointmap(spec, theme) {
   let unwrapped = false;
   {
     const lons = pts.map((p) => p.lon);
-    const span = Math.max(...lons) - Math.min(...lons);
+    const span = hi(lons) - lo(lons);
     if (span > 180) {
       const alt = lons.map((l) => (l < 0 ? l + 360 : l));
-      if (Math.max(...alt) - Math.min(...alt) < span) { for (const p of pts) if (p.lon < 0) p.lon += 360; unwrapped = true; }
+      if (hi(alt) - lo(alt) < span) { for (const p of pts) if (p.lon < 0) p.lon += 360; unwrapped = true; }
     }
   }
   for (const p of pts) { p.mx = mercX(p.lon); p.my = mercY(p.lat); }
@@ -267,7 +270,7 @@ export function pointmap(spec, theme) {
     colorOf = (p) => cmap.get(p.k) || theme.other;
     if (keys.length > 1) legend = keys.map((k) => ({ label: k, color: cmap.get(k) }));
   } else if (valueF && values.length) {
-    const vmin = Math.min(...values), vmax = Math.max(...values);
+    const vmin = lo(values), vmax = hi(values);
     const mid = spec.midpoint ?? (spec.diverging ? 0 : undefined);
     let stops, at;
     if (mid !== undefined) {
@@ -295,7 +298,7 @@ export function pointmap(spec, theme) {
   let area = frame.layout(legend);
 
   // Radios: área proporcional a `size` (raíz cuadrada), o uno fijo según la densidad.
-  const maxS = Math.max(0, ...sizes.map(Math.abs));
+  const maxS = sizes.reduce((m, v) => Math.max(m, Math.abs(v)), 0);
   const wk = Math.max(0.6, Math.min(1.2, area.w / 672));
   const rMax = sizeF && maxS ? spec.maxRadius ?? Math.round((n <= 40 ? 22 : n <= 150 ? 15 : n <= 600 ? 10 : 6) * wk) : 0;
   const rBase = spec.radius ?? (dense ? 2.2 : n > 500 ? 3 : n > 120 ? 4 : 5.5);
@@ -513,8 +516,8 @@ function describePointmap({ pts, n, provider, world, hueF, keys, sizeF, valueF,
   const en = String(locale).startsWith('en');
   const W = en ? 'W' : 'O';
   const lats = pts.map((p) => p.lat), lons = pts.map((p) => (p.lon > 180 ? p.lon - 360 : p.lon));
-  const la0 = Math.min(...lats), la1 = Math.max(...lats);
-  const lo0 = Math.min(...lons), lo1 = Math.max(...lons);
+  const la0 = lo(lats), la1 = hi(lats);
+  const lo0 = lo(lons), lo1 = hi(lons);
   const base = provider ? provider.name || provider.attribution : world ? (en ? 'a world outline' : 'un contorno del mundo') : '';
   const where = n === 1 || (la1 - la0 < 0.05 && lo1 - lo0 < 0.05)
     ? (en ? ` at ${deg(la0, 'N', 'S', locale)}, ${deg(lo0, 'E', W, locale)}` : ` en ${deg(la0, 'N', 'S', locale)}, ${deg(lo0, 'E', W, locale)}`)

package/src/charts/regplot.js

@@ -56,7 +56,7 @@ function fitGroup(gp, spec, { xLog, xMin, xMax, steps = 90 }) {
     const pts = lw.map(([t, v]) => [spec.logx ? 10 ** t : t, v]);
     return { line: pts, band: null, r2: null, n: gp.length };
   }
-  const fit = polyfit(xs, ys, { order: spec.order ?? 1, level: (spec.ci ?? 95) / 100 });
+  const fit = polyfit(xs, ys, { order: spec.order ?? 1, level: typeof spec.ci === 'number' ? (spec.ci > 1 ? spec.ci / 100 : spec.ci) : 0.95 });
   const line = grid.map((v) => [v, fit.predict(fx(v))]);
   const band = spec.ci === false || spec.ci === null || gp.length < (spec.order ?? 1) + 3 ? null : grid.map((v) => [v, ...fit.band(fx(v))]);
   return { line, band, r2: fit.r2, n: fit.n, fit };
@@ -123,7 +123,7 @@ export function regplot(spec, theme) {
   });
   if (spec.scatter !== false) {
     const order = [...pts].sort((a, b) => (focusKey(a.k) ? 1 : 0) - (focusKey(b.k) ? 1 : 0));
-    const dots = order.map((p) => ({
+    const dots = order.filter((p) => visible(p, xS, yS, inner)).map((p) => ({
       class: 'pa-m', cx: xS(p.x), cy: yS(p.y), r, fill: colorOf(p.k), fillOpacity: K === 1 ? (M.dotOpacity ?? 0.85) * 0.75 : M.dotOpacity ?? 0.85,
       stroke: pts.length > 400 ? 'none' : theme.surface, strokeWidth: 1.25,
       'data-tip': tip(p.name || p.k || '', `${humanize(P.x)}: ${fx(p.x)}`, `${humanize(P.y)}: ${fy(p.y)}`, p.name && p.k ? p.k : ''),
@@ -351,7 +351,7 @@ export function jointplot(spec, theme) {
       body.push(el('path', { d: L(f.line), fill: 'none', stroke: K === 1 ? theme.ink : colorOf(k), strokeWidth: M.lineWidth ?? 2, opacity: 0.9 }));
     });
   }
-  body.push(g({}, dotLayer(pts.map((p) => ({ class: 'pa-m', cx: xS(p.x), cy: yS(p.y), r, fill: colorOf(p.k), fillOpacity: M.dotOpacity ?? 0.85, stroke: pts.length > 400 ? 'none' : theme.surface, strokeWidth: 1.25, 'data-tip': tip(p.name || p.k, `${humanize(P.x)}: ${fx(p.x)}`, `${humanize(P.y)}: ${fy(p.y)}`, p.name && p.k ? p.k : '') })))));
+  body.push(g({}, dotLayer(pts.filter((p) => visible(p, xS, yS, inner)).map((p) => ({ class: 'pa-m', cx: xS(p.x), cy: yS(p.y), r, fill: colorOf(p.k), fillOpacity: M.dotOpacity ?? 0.85, stroke: pts.length > 400 ? 'none' : theme.surface, strokeWidth: 1.25, 'data-tip': tip(p.name || p.k, `${humanize(P.x)}: ${fx(p.x)}`, `${humanize(P.y)}: ${fy(p.y)}`, p.name && p.k ? p.k : '') })))));
   body.push(pointLabels(pts, spec, xS, yS, r, inner, theme, P));
   body.push(gridX(xAx.ticks, xS, inner, theme, tickFmt(fx, xAx), { lines: false }));
   body.push(yTickLabels(yAx.ticks, yS, inner, theme, yt));
@@ -363,6 +363,12 @@ export function jointplot(spec, theme) {
   return frame.finish(body, { desc, type: 'jointplot' });
 }
 
+/** Con xDomain/yDomain, los puntos fuera del rango no se dibujan fuera del área. */
+function visible(p, xS, yS, inner) {
+  const cx = xS(p.x), cy = yS(p.y);
+  return cx >= inner.x - 1 && cx <= inner.x + inner.w + 1 && cy >= inner.y - 1 && cy <= inner.y + inner.h + 1;
+}
+
 function hashStr(s) {
   let h = 0;
   for (let i = 0; i < s.length; i++) h = (Math.imul(31, h) + s.charCodeAt(i)) | 0;

package/src/charts/sankey.js

@@ -120,7 +120,7 @@ export function sankey(spec, theme) {
     const x0 = l.sn.x + NW, x1 = l.tn.x, xm = (x0 + x1) / 2, w = l.v * ky;
     const d = `M${x0},${l.y0}C${xm},${l.y0} ${xm},${l.y1} ${x1},${l.y1}L${x1},${l.y1 + w}C${xm},${l.y1 + w} ${xm},${l.y0 + w} ${x0},${l.y0 + w}Z`;
     const share = l.sn.value ? Math.round((l.v / l.sn.value) * 100) : 0;
-    linkG.push(el('path', { class: 'pa-m', d, fill: linkColor(l), fillOpacity: theme.link, 'data-tip': `${l.s} → ${l.t}\n${fmt(l.v)} · ${share}% de ${l.s}` }));
+    linkG.push(el('path', { class: 'pa-m', d, fill: linkColor(l), fillOpacity: theme.link, 'data-tip': `${l.s} → ${l.t}\n${fmt(l.v)} · ${share}% ${en ? 'of' : 'de'} ${l.s}` }));
   }
   body.push(g({ class: 'pa-links' }, linkG));
   for (const [ci, c] of cols.entries()) for (const n of c) {

package/src/charts/scatter.js

@@ -138,7 +138,9 @@ export function scatter(spec, theme) {
   // Tooltips: la capa densa guarda a lo más MAX_TIPS; no se arman los que se descartan.
   const MAX_TIPS = 20000;
   const hx = humanize(x), hy = humanize(y), hs = size ? humanize(size) : '';
-  const circles = sorted.map((p, i) => {
+  // Con xDomain/yDomain, los puntos fuera del rango no se dibujan fuera del área.
+  const vis = (p) => { const cx = xS(p.x), cy = yS(p.y); return cx >= inner.x - 1 && cx <= inner.x + inner.w + 1 && cy >= inner.y - 1 && cy <= inner.y + inner.h + 1; };
+  const circles = sorted.filter(vis).map((p, i) => {
     let t = '';
     if (i < MAX_TIPS || sorted.length <= 1500) {
       const lines = [p.name || '', `${hx}: ${fx(p.x)}`, `${hy}: ${fy(p.y)}`];
@@ -195,7 +197,7 @@ export function scatter(spec, theme) {
     }
     const placed = [];
     for (const p of chosen) {
-      if (!p.name) continue;
+      if (!p.name || !vis(p)) continue;
       const w = measure(p.name, 12, 500);
       const r = rOf(p);
       let lx = xS(p.x) + r + 4, ly = yS(p.y) + 4, anchor = 'start';

package/src/charts/sunburst.js

@@ -7,7 +7,7 @@ import { makeFormat } from '../format.js';
 import { textOn } from '../theme.js';
 import { humanize } from '../data.js';
 import { createFrame } from '../frame.js';
-import { buildHierarchy, findNode, assignColors, shareFmt, breadcrumb, height, SEP } from '../hierarchy.js';
+import { buildHierarchy, focusNode, assignColors, shareFmt, breadcrumb, height, SEP } from '../hierarchy.js';
 
 const TAU = Math.PI * 2;
 const f = (n) => n.toFixed(2);
@@ -24,7 +24,7 @@ export function sunburst(spec, theme) {
   const locale = spec.locale || 'es';
   const en = String(locale).startsWith('en');
   const { root, fields, valueField } = buildHierarchy(spec);
-  const focus = findNode(root, spec.focus);
+  const focus = focusNode(root, spec.focus);
   const levels = Math.max(1, height(focus));
   assignColors(root, theme, spec); // colores estables al navegar
   const vals = [];