default diverging based on scheme

Author: mbostock

src/scales.js

@@ -3,6 +3,7 @@ import {isColor, isEvery, isOrdinal, isFirst, isTemporal, isTemporalString, isNu
 import {registry, color, position, radius, opacity, symbol, length} from "./scales/index.js";
 import {ScaleLinear, ScaleSqrt, ScalePow, ScaleLog, ScaleSymlog, ScaleQuantile, ScaleQuantize, ScaleThreshold, ScaleIdentity} from "./scales/quantitative.js";
 import {ScaleDiverging, ScaleDivergingSqrt, ScaleDivergingPow, ScaleDivergingLog, ScaleDivergingSymlog} from "./scales/diverging.js";
+import {isDivergingScheme} from "./scales/schemes.js";
 import {ScaleTime, ScaleUtc} from "./scales/temporal.js";
 import {ScaleOrdinal, ScalePoint, ScaleBand, ordinalImplicit} from "./scales/ordinal.js";
 import {isSymbol, maybeSymbol} from "./symbols.js";
@@ -212,7 +213,7 @@ function formatScaleType(type) {
   return typeof type === "symbol" ? type.description : type;
 }
 
-function inferScaleType(key, channels, {type, domain, range, scheme}) {
+function inferScaleType(key, channels, {type, domain, range, scheme, pivot}) {
   // The facet scales are always band scales; this cannot be changed.
   if (key === "fx" || key === "fy") return "band";
 
@@ -266,13 +267,15 @@ function inferScaleType(key, channels, {type, domain, range, scheme}) {
   if (domain !== undefined) {
     if (isOrdinal(domain)) return asOrdinalType(kind);
     if (isTemporal(domain)) return "utc";
+    if (kind === color && (pivot != null || isDivergingScheme(scheme))) return "diverging";
     return "linear";
   }
 
   // If any channel is ordinal or temporal, it takes priority.
   const values = channels.map(({value}) => value).filter(value => value !== undefined);
   if (values.some(isOrdinal)) return asOrdinalType(kind);
   if (values.some(isTemporal)) return "utc";
+  if (kind === color && (pivot != null || isDivergingScheme(scheme))) return "diverging";
   return "linear";
 }
 

src/scales/schemes.js

@@ -258,3 +258,21 @@ export function quantitativeScheme(scheme) {
   if (!quantitativeSchemes.has(s)) throw new Error(`unknown scheme: ${s}`);
   return quantitativeSchemes.get(s);
 }
+
+const divergingSchemes = new Set([
+  "brbg",
+  "prgn",
+  "piyg",
+  "puor",
+  "rdbu",
+  "rdgy",
+  "rdylbu",
+  "rdylgn",
+  "spectral",
+  "burd",
+  "buylrd"
+]);
+
+export function isDivergingScheme(scheme) {
+  return scheme != null && divergingSchemes.has(`${scheme}`.toLowerCase());
+}

test/plots/gistemp-anomaly-moving.js

@@ -10,7 +10,6 @@ export default async function() {
       grid: true
     },
     color: {
-      type: "diverging",
       scheme: "BuRd",
       symmetric: false
     },

test/plots/gistemp-anomaly-transform.js

@@ -12,7 +12,6 @@ export default async function() {
       grid: true
     },
     color: {
-      type: "diverging",
       scheme: "BuRd",
       domain: [-1, 1],
       transform

test/plots/gistemp-anomaly.js

@@ -10,8 +10,7 @@ export default async function() {
       grid: true
     },
     color: {
-      type: "diverging",
-      reverse: true
+      scheme: "BuRd"
     },
     marks: [
       Plot.ruleY([0]),

test/plots/hadcrut-warming-stripes.js

@@ -15,7 +15,6 @@ export default async function() {
       round: true
     },
     color: {
-      type: "diverging",
       scheme: "BuRd",
       symmetric: false
     },

test/plots/legend-color.js

@@ -273,7 +273,6 @@ export function colorLegendImplicitLabel() {
 export function colorLegendDiverging() {
   return Plot.legend({
     color: {
-      type: "diverging",
       domain: [-0.1, 0.1],
       scheme: "PiYG",
       label: "Daily change"
@@ -285,7 +284,6 @@ export function colorLegendDiverging() {
 export function colorLegendDivergingPivot() {
   return Plot.legend({
     color: {
-      type: "diverging",
       domain: [1, 4],
       pivot: 3,
       scheme: "PiYG"
@@ -296,7 +294,6 @@ export function colorLegendDivergingPivot() {
 export function colorLegendDivergingPivotAsymmetric() {
   return Plot.legend({
     color: {
-      type: "diverging",
       symmetric: false,
       domain: [1, 4],
       pivot: 3,

test/plots/metro-inequality-change.js

@@ -14,8 +14,7 @@ export default async function() {
       label: "↑ Inequality"
     },
     color: {
-      type: "diverging",
-      reverse: true,
+      scheme: "BuRd",
       symmetric: false
     },
     marks: [

test/plots/metro-unemployment-slope.js

@@ -8,7 +8,6 @@ export default async function() {
       grid: true
     },
     color: {
-      type: "diverging",
       scheme: "buylrd",
       domain: [-0.5, 0.5]
     },

test/plots/seattle-precipitation-rule.js

@@ -3,13 +3,5 @@ import * as d3 from "d3";
 
 export default async function() {
   const data = await d3.csv("data/seattle-weather.csv", d3.autoType);
-  return Plot.plot({
-    color: {
-      scheme: "RdBu",
-      reverse: true
-    },
-    marks: [
-      Plot.ruleX(data, {x: "date", strokeOpacity: "precipitation"})
-    ]
-  });
+  return Plot.ruleX(data, {x: "date", strokeOpacity: "precipitation"}).plot();
 }