[] is an empty facet (#1216)

* [] is an empty facet

closes #1212
alternative to #1215

* consistent filtering

Co-authored-by: Mike Bostock <[email protected]>

Author: Fil

src/plot.js

@@ -303,8 +303,8 @@ export function plot(options = {}) {
           if (facets) {
             const fi = facetPosition.get(key);
             facet = facets[fi] ?? facets[0];
-            if (!facet) continue;
             facet = mark.filter(facet, channels, values);
+            if (!facet.length) continue;
             facet.fi = fi;
           }
           const node = mark.render(facet, scales, values, subdimensions, context);
@@ -316,8 +316,8 @@ export function plot(options = {}) {
       let facet = null;
       if (facets) {
         facet = facets[0];
-        if (!facet) continue;
         facet = mark.filter(facet, channels, values);
+        if (!facet.length) continue;
       }
       const node = mark.render(facet, scales, values, dimensions, context);
       if (node != null) svg.appendChild(node);
@@ -655,8 +655,8 @@ function maybeMarkFacet(mark, topFacetState, options) {
 // Facet filter, by mark; for now only the "eq" filter is provided.
 function filterFacets(facets, {channels: {fx, fy}, groups}) {
   return fx && fy
-    ? facets.map(({x, y}) => groups.get(x)?.get(y))
+    ? facets.map(({x, y}) => groups.get(x)?.get(y) ?? [])
     : fx
-    ? facets.map(({x}) => groups.get(x))
-    : facets.map(({y}) => groups.get(y));
+    ? facets.map(({x}) => groups.get(x) ?? [])
+    : facets.map(({y}) => groups.get(y) ?? []);
 }

test/output/athletesSample.svg

@@ -0,0 +1,13 @@
+import * as Plot from "@observablehq/plot";
+
+export default async function () {
+  const data = [
+    {PERIOD: 1, VALUE: 3, TYPE: "c"},
+    {PERIOD: 2, VALUE: 4, TYPE: "c"}
+  ];
+  return Plot.plot({
+    facet: {data, x: "TYPE"},
+    fx: {domain: ["a", "b"]},
+    marks: [Plot.barY(data, {x: "PERIOD", y: "VALUE"})]
+  });
+}

test/output/athletesSportWeight.svg

@@ -64,6 +64,7 @@ export {default as downloads} from "./downloads.js";
 export {default as downloadsOrdinal} from "./downloads-ordinal.js";
 export {default as driving} from "./driving.js";
 export {default as empty} from "./empty.js";
+export {default as emptyFacet} from "./empty-facet.js";
 export {default as emptyLegend} from "./empty-legend.js";
 export {default as emptyX} from "./empty-x.js";
 export {default as energyProduction} from "./energy-production.js";