Skip to content
This repository has been archived by the owner on Nov 7, 2018. It is now read-only.

Sankey Plugin, complex path to draw links #36

Closed
wants to merge 1 commit into from

Conversation

ghedamat
Copy link

As seen in
http://bl.ocks.org/d/4599519/

using a thick line to draw links between blocks can lead to artifacts ( just drag the blocks around)

this patch uses a set of 4 lines to draw the connection between blocks, see
http://bl.ocks.org/d/4599792/

see also
http://bl.ocks.org/d/4590576/
for the usual example after this patch

As seen in
http://bl.ocks.org/d/4599519/

using a thick line to draw links between blocks can lead to artifacts

this patch uses a set of 4 lines to draw the connection between blocks, see
http://bl.ocks.org/d/4599792/
@mbostock
Copy link
Member

Unfortunately, the problem with this fix is that the links are no longer of constant width—which is generally a requirement with Sankey diagrams, as the width of the link is the primary visual encoding.

Screen Shot 2013-01-22 at 4 04 22 PM

@mbostock mbostock closed this Jan 23, 2013
@mbostock
Copy link
Member

Also. Thank you for the suggestion and the helpful demonstrative examples!

@ghedamat
Copy link
Author

I understand, my knowledge of the sankey diagrams is not deep unfortunately :(

This fix suits my actual needs but I guess we should probably look for another solution for those graphical quirks,
if you have any better solution/idea let me know, I'll be happy to give it a try!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants