anvaka / anvaka/VivaGraphJS

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

Open
#203 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
3.9k
Forks
418
PR merge metrics
No merged PRs in 30d

Description

First of all: thank you for all of ngraph!

I'm currently looking into rendering a network graph and the [youtube](https://github.com/anvaka/yasiv-youtube) and [amazon](http://www.yasiv.com/amazon#/Search?q=graph%20drawing&category=Books&lang=US) (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](https://github.com/anvaka/ngraph.remove-overlaps) (which sadly is CommonJS only) and applying it [VivaGraphJS](https://github.com/anvaka/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](https://github.com/anvaka/VivaGraphJS/blob/v0.8.3/package.json#L39) (`v0.0.21` vs `v0.1.2`). Is VivaGraph intentionally using outdated dependencies?

Contributor guide

No contributing guide indexed for this repository

Research direction

Inspect the v0.8.3 package.json dependency versions and compare them with the versions expected by ngraph.remove-overlaps. Reproduce removeOverlaps(layout, {active: true}) with VivaGraphJS and check the reported layout.on error. Done means establishing compatible dependency behavior or documenting why the older versions are intentional.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
computer-graphics
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.