attempting to use ngraph.remove-overlaps fails because of outdated dependencies
- 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