Treemap layout (treemapBinary strategy) outputs NaN co-ordinates

[peterrcook]

I'm seeing x0 and x1 NaN values when data contains zeroes e.g.

var data = [
  {id: 'A', val1: 0, val2: 125},
  {id: 'B', val1: 0, val2: 150},
  {id: 'C', val1: 1, val2: 100}
];

If I sum by val2 or use the default tiling strategy, I don't see NaNs.

Example here:

https://bl.ocks.org/peterrcook/21c0582665261a5f9d270b01d1a12d76

Maybe related to #35

[mbostock]

As a notebook for easier inspection and editing:

https://observablehq.com/d/d69b436a2fcc4ed6

It’s almost certainly a divide by zero here:

d3-hierarchy/src/treemap/binary.js

Line 37 in 46f9e8b

var xk = (x0 * valueRight + x1 * valueLeft) / value;

d3-hierarchy/src/treemap/binary.js

Line 41 in 46f9e8b

var yk = (y0 * valueRight + y1 * valueLeft) / value;

[mbostock]

There’s a similar bug in d3.treemapResquarify, most likely a divide by zero here:

d3-hierarchy/src/treemap/resquarify.js

Lines 21 to 22 in d6adccb

	if (row.dice) treemapDice(row, x0, y0, x1, y0 += (y1 - y0) * row.value / value);
	else treemapSlice(row, x0, y0, x0 += (x1 - x0) * row.value / value, y1);

[Fil]

fixed by 39f420c