VivaGraphJS icon indicating copy to clipboard operation
VivaGraphJS copied to clipboard

attempting to use ngraph.remove-overlaps fails because of outdated dependencies

Open rodneyrehm opened this issue 8 years ago • 1 comments
trafficstars

First of all: thank you for all of ngraph!

I'm currently looking into rendering a network graph and the youtube and amazon (source not found) examples have come the closest to what I want to have visually. The main problem with most of the other examples I've played with was overlapping nodes.

When loading ngraph.remove-overlaps (which sadly is CommonJS only) and applying it VivaGraphJS via removeOverlaps(layout, {active: true}) I get Uncaught TypeError: layout.on is not a function, presumably because the current VivaGraph 0.8.3 is using old versions (v0.0.21 vs v0.1.2). Is VivaGraph intentionally using outdated dependencies?

rodneyrehm avatar May 08 '17 13:05 rodneyrehm

Thanks for pointing this out. I updated the force layout to the latest version, so it should be compatible with ngraph.remove-overlaps.

I have to warn though, that overlap removal is a CPU intensive process. It would probably be better to not use { active: true }. Instead, call it just once, when your layout is almost stable.

anvaka avatar May 09 '17 05:05 anvaka