Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: create generalized networkx constructor #69

Merged
merged 14 commits into from
Sep 16, 2024
Merged

feat: create generalized networkx constructor #69

merged 14 commits into from
Sep 16, 2024

Conversation

jsstevenson
Copy link
Contributor

@jsstevenson jsstevenson commented Sep 5, 2024

  • house network/graph related functions in their own modules. This effectively duplicates some stuff that we've done previously for the dash app, but I think we can move/collapse that at a later date (i.e. I'm not making any breaking changes here yet, just adding in redundant functionality for now)

The motivation is to support generalization of some of the graph visualization tooling -- so that we aren't as tied to just interactions and can more easily support other kinds of queries (or combos of queries???)

@jsstevenson jsstevenson added the priority:low Low priority label Sep 5, 2024
@jsstevenson jsstevenson changed the title network feat: create generalized networkx constructor Sep 5, 2024
Copy link
Contributor

@mcannon068nw mcannon068nw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jsstevenson jsstevenson merged commit aa5a450 into main Sep 16, 2024
12 checks passed
@jsstevenson jsstevenson deleted the network branch September 16, 2024 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:low Low priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants