Skip to content
This repository has been archived by the owner on Dec 10, 2024. It is now read-only.

Add interactive graph viewer #20

Merged
merged 2 commits into from
Aug 23, 2024
Merged

Conversation

mawildoer
Copy link
Contributor

Screenshot 2024-08-22 at 15 35 04

@mawildoer mawildoer force-pushed the mawildoer/interactive-viewer-2 branch from 75a94dd to 317376f Compare August 22, 2024 23:13
@mawildoer mawildoer changed the title Add traits for running scipy optimize Add interactive graph viewer Aug 22, 2024
Copy link
Contributor

@iopapamanoglou iopapamanoglou left a comment

Choose a reason for hiding this comment

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

Very nice addition thanks!

return {"data": {"source": source, "target": target, "type": type_name}}

all_gifs = list(
gif for node in get_all_nodes_graph(G) for gif in node.GIFs.get_all()
Copy link
Contributor

Choose a reason for hiding this comment

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

you can get the gifs with node.GIFs.self.edges instead of get_all to make it graph space

Copy link
Contributor

@iopapamanoglou iopapamanoglou Aug 23, 2024

Choose a reason for hiding this comment

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

but all_gifs = list(G)
so thats even easier

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah nice! Thanks mate! Much cleaner!

return x is not None

elements = [
*(filter(_not_none, (_node(gif) for gif in all_gifs))),
Copy link
Contributor

Choose a reason for hiding this comment

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

for gif in G

@mawildoer mawildoer merged commit 504ba1c into main Aug 23, 2024
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants