d3-book icon indicating copy to clipboard operation
d3-book copied to clipboard

Chapter 12 - freakOut function in each()

Open ialsjbn opened this issue 2 years ago • 1 comments

The original code for freakout() has:

var colors = d3.schemeCategory20;
var colorIndex = Math.round(Math.random() *20);

For D3v5 and newer:

D3 no longer provides the d3.schemeCategory20* categorical color schemes. These twenty-color schemes were flawed because their grouped design could falsely imply relationships in the data: a shared hue can imply that the encoded data are part of a group (a super-category), while relative lightness can imply order

To get it to work, I changed it to:

var colors = d3.schemeCategory10;
var colorIndex = Math.round(Math.random() *10);

ialsjbn avatar Nov 10 '22 01:11 ialsjbn

Thanks @ialsjbn! I'm keeping this issue open, but filing it away in the event of a v5+ update for the book.

scotthmurray avatar Nov 10 '22 05:11 scotthmurray