Issue 82

package.json

@@ -61,7 +61,7 @@
     "karma": "^6.3.2",
     "karma-chrome-launcher": "^3.1.0",
     "karma-coverage": "^2.0.3",
-    "karma-firefox-launcher": "^2.1.0",
+    "karma-firefox-launcher": "^2.1.3",
     "karma-jasmine": "^5.0.1",
     "karma-jasmine-html-reporter": "^2.0.0",
     "karma-rollup-preprocessor": "7.0.7",

src/layout.js

@@ -39,7 +39,9 @@ export function calculateX(nodes, data) {
       }
     });
 
-  return x;
+  return [...nodes.values()]
+    .map((node) => node.x)
+    .reduce((n, acc) => Math.max(n, acc), 0);
 }
 
 /**
@@ -212,7 +214,7 @@ export function calculateYUsingPriority(nodeArray, maxX) {
   for (let x = 0; x <= maxX; x++) {
     let y = nextYStart;
     const nodes = nodeArray.filter(node => node.x === x).sort((a, b) => a.priority - b.priority);
-    nextYStart = nodes[0].to.filter(to => to.node.x > x + 1).reduce((acc, cur) => acc + cur.flow, 0) || 0;
+    nextYStart = nodes.length ? nodes[0].to.filter(to => to.node.x > x + 1).reduce((acc, cur) => acc + cur.flow, 0) || 0 : 0;
     for (const node of nodes) {
       node.y = y;
       y += Math.max(node.out, node.in);

test/fixtures/issues/issue81.js

@@ -0,0 +1,40 @@
+module.exports = {
+  config: {
+    type: 'sankey',
+    data: {
+      datasets: [
+        {
+          data: [
+            {from: 'Oil', to: 'Air1', flow: 15},
+            {from: 'Oil', to: 'Stuff1', flow: 20},
+            {from: 'Oil1', to: 'Air2', flow: 25},
+            {from: 'Oil1', to: 'Stuff2', flow: 25},
+          ],
+          priority: {
+            Oil: 1,
+            Oil1: 2,
+            Air: 3,
+            Air2: 1,
+            Stuff1: 2,
+            Stuff2: 1,
+          },
+          column: {
+            Oil: 0,
+            Oil1: 1,
+            Air1: 1,
+            Air2: 2,
+            Stuff1: 1,
+            Stuff2: 2,
+          },
+        },
+      ],
+    },
+  },
+  options: {
+    spriteText: true,
+    canvas: {
+      height: 512,
+      width: 512,
+    },
+  },
+};

test/fixtures/issues/issue82.js

@@ -0,0 +1,31 @@
+module.exports = {
+  config: {
+    type: 'sankey',
+    data: {
+      datasets: [
+        {
+          data: [
+            {from: 'Oil', to: 'Energy', flow: 15},
+            {from: 'Natural Gas', to: 'Energy', flow: 20},
+            {from: 'Coal', to: 'Energy', flow: 25},
+            {from: 'Electricity', to: 'Energy', flow: 25},
+          ],
+          column: {
+            Oil: 0,
+            'Natural Gas': 1,
+            Coal: 2,
+            Electricity: 3,
+            Energy: 4,
+          },
+        },
+      ],
+    },
+  },
+  options: {
+    spriteText: true,
+    canvas: {
+      height: 512,
+      width: 512,
+    },
+  },
+};