Add property to define size of a node in Sankey Chart

[peterviergutz]

What problem does this feature solve?

In the current implementation the node size in a Sankey chart is determined by the bigger number of incoming or outgoing edges. Our case involves a node size that is actually bigger than the connections between the nodes:

What does the proposed API look like?

An optional new property size on the nodes' properties could offer a clean option to define the node's size

{
  "nodes": [
    {
      "name": "a1",
      "size": 100
    }
...

To get the node's size in the chart could then be determined by whatever is bigger: the sum of incoming and outgoing edges or the node's defined size in

https://github.com/apache/incubator-echarts/blob/fcfa95012ca561becab607b065a62b5f87220801/src/chart/sankey/sankeyLayout.js#L86

function computeNodeValues(nodes) {
  zrUtil.each(nodes, function (node) {
    var value1 = sum(node.outEdges, getEdgeValue);
    var value2 = sum(node.inEdges, getEdgeValue);
    var value = Math.max(value1, value2, node.size); // <-- node.size here
    node.setLayout({
      value: value
    }, true);
  });
}

[echarts-bot]

Hi! We've received your issue and please be patient to get responded. 🎉
The average response time is expected to be within one day for weekdays.

In the meanwhile, please make sure that you have posted enough image to demo your request. You may also check out the API and chart option to get the answer.

If you don't get helped for a long time (over a week) or have an urgent question to ask, you may also send an email to dev@echarts.apache.org. Please attach the issue link if it's a technical questions.

If you are interested in the project, you may also subscribe our mail list.

Have a nice day! 🍵

[deqingli]

Maybe I can add new property value of each node, to keep consistent with graph series.

nodes: [{
    name: 'aaa',
    value: 100, // the same effect as size
    depth: 3
}]

[peterviergutz]

I agree value would have been the right name in the first place. Thanks for your consideration.

PS I guess just a typo, but 100 should be an integer.

[ChibahAbdelwahab]

Hi,
I'm actually in the same case, have this been added ?

[deqingli]

I agree value would have been the right name in the first place. Thanks for your consideration.

PS I guess just a typo, but 100 should be an integer.

Yeah, it is an integer, thanks.