fix #278; robust log domain

src/defined.js

@@ -27,6 +27,10 @@ export function positive(x) {
   return x > 0 ? x : NaN;
 }
 
+export function negative(x) {
+  return x < 0 ? x : NaN;
+}
+
 export function firstof(...values) {
   for (const v of values) {
     if (v !== undefined) {

src/scales/quantitative.js

@@ -50,7 +50,7 @@ import {
 } from "d3";
 import {scaleDiverging, scaleLinear, scaleLog, scalePow, scaleSymlog} from "d3";
 import {registry, radius, color} from "./index.js";
-import {positive} from "../defined.js";
+import {positive, negative} from "../defined.js";
 
 const constant = x => () => x;
 const flip = i => t => i(1 - t);
@@ -179,8 +179,8 @@ export function ScalePow(key, channels, {exponent = 1, ...options}) {
   return ScaleQ(key, scalePow().exponent(exponent), channels, options);
 }
 
-export function ScaleLog(key, channels, {base = 10, ...options}) {
-  return ScaleQ(key, scaleLog().base(base), channels, options);
+export function ScaleLog(key, channels, {base = 10, domain = inferLogDomain(channels), ...options}) {
+  return ScaleQ(key, scaleLog().base(base), channels, {domain, ...options});
 }
 
 export function ScaleSymlog(key, channels, {constant = 1, ...options}) {
@@ -215,10 +215,10 @@ export function ScaleDiverging(key, channels, {
   return {type: "quantitative", invert, domain, scale};
 }
 
-function inferDomain(channels) {
+function inferDomain(channels, f) {
   return [
-    min(channels, ({value}) => value === undefined ? value : min(value)),
-    max(channels, ({value}) => value === undefined ? value : max(value))
+    min(channels, ({value}) => value === undefined ? value : min(value, f)),
+    max(channels, ({value}) => value === undefined ? value : max(value, f))
   ];
 }
 
@@ -232,3 +232,16 @@ function inferRadialRange(channels, domain) {
   const h25 = quantile(channels, 0.5, ({value}) => value === undefined ? NaN : quantile(value, 0.25, positive));
   return domain.map(d => 3 * Math.sqrt(d / h25));
 }
+
+function inferLogDomain(channels) {
+  for (const {value} of channels) {
+    if (value !== undefined) {
+      for (let v of value) {
+        v = +v;
+        if (v > 0) return inferDomain(channels, positive);
+        if (v < 0) return inferDomain(channels, negative);
+      }
+    }
+  }
+  return [1, 10];
+}

test/plots/index.js

@@ -35,6 +35,7 @@ export {default as letterFrequencyBar} from "./letter-frequency-bar.js";
 export {default as letterFrequencyColumn} from "./letter-frequency-column.js";
 export {default as letterFrequencyDot} from "./letter-frequency-dot.js";
 export {default as letterFrequencyLollipop} from "./letter-frequency-lollipop.js";
+export {default as logDegenerate} from "./log-degenerate.js";
 export {default as metroInequality} from "./metro-inequality.js";
 export {default as metroInequalityChange} from "./metro-inequality-change.js";
 export {default as metroUnemployment} from "./metro-unemployment.js";

test/plots/log-degenerate.js

@@ -0,0 +1,12 @@
+import * as Plot from "@observablehq/plot";
+
+export default async function() {
+  return Plot.plot({
+    x: {
+      type: "log"
+    },
+    marks: [
+      Plot.dotX([0, 0.1, 1, 2, 10])
+    ]
+  });
+}