tomshanley / d3-sankey-circular

A fork of the d3-sankey library to allow circular links.
MIT License
89 stars 41 forks source link

Node sort #37

Closed peteruithoven closed 1 year ago

peteruithoven commented 5 years ago

Similar to d3-sankey's nodeSort. https://github.com/d3/d3-sankey#sankey_nodeSort

tadeaustria commented 1 year ago

Hi Guys, With this change I get an error

Uncaught (in promise) ReferenceError: ascendingBreadth is not defined
    resolveNodesOverlap http://127.0.0.1:5500/third_party/d3-sankey-circular.js:1499
    resolveNodesOverlap http://127.0.0.1:5500/third_party/d3-sankey-circular.js:1492
    sankeyCircular http://127.0.0.1:5500/third_party/d3-sankey-circular.js:169
    ...

and it is true, that due to the pull in of ascendingBreadth into sankeyCircular() it is not visible anymore for the use in line 1499.