search

ericmjl / nxviz

Visualization Package for NetworkX
https://ericmjl.github.io/nxviz
MIT License
449 stars 86 forks source link

API Refactor #658

Closed ericmjl closed 3 years ago

ericmjl commented 3 years ago

This is a major refactor of the nxviz API.

The goal is to make the package maintainable in the long-term by emphasizing composability in plotting.

ericmjl commented 3 years ago

The refactor is basically complete. I am going to merge first, but not cut a release just yet. @MridulS I'm going to test NAMS against nxviz master in a PR to see how it goes.

AlirezaTheH commented 2 years ago

Hi, I see this refactor drops many features of package. I wondering why? Are you gonna add those in future?

ericmjl commented 2 years ago

Hi @AlirezaH320, thanks for chiming in - can I ask what features you're referring to? If they fit within the new API paradigm, I'm open to working with you on a PR a to get it implemented.

AlirezaTheH commented 2 years ago

Mainly, node labels and it's extra. I'm also open to add them to new API if you want.