egui_graphs icon indicating copy to clipboard operation
egui_graphs copied to clipboard

GraphMap support in transform

Open dmyyy opened this issue 2 years ago • 4 comments

I'm using GraphMap so this would be useful for me. Could make sense to add a Transformable trait, or even better impl Into<Graph<N, E, Ty>> for GraphMap<N, E, Ty>. Not sure I understand the use case for to_input_graph_custom.

dmyyy avatar Sep 03 '23 02:09 dmyyy

Good idea. I ll take a look into it on the upcoming week. If you wish you can file a PR as well.

You can check the usage of to_input_graph_custom in the examples. But I agree that variant with trait implementation would be more seamless and straightforward integration for users of the lib.

blitzarx1 avatar Sep 03 '23 08:09 blitzarx1

Sorry for long response. I ve recently added From support for new public API type Graph. It support petgraph::StableGraph conversion to Graph type. #77

In the following PRs I will also support GraphMap as we discussed here.

blitzarx1 avatar Sep 16 '23 15:09 blitzarx1

Long time since my last visit here. @dmyyy if you think you can take this one as well, please go for it. I don't feel like getting this done soon. I am focused on other issues now.

blitzarx1 avatar Nov 07 '23 19:11 blitzarx1

If I get something working I'll put out a pr for this one as well - not my focus currently but we'll see if I get to it

dmyyy avatar Nov 07 '23 19:11 dmyyy

Hi! I think this is a little outdated... Please check the latest version and feel free to create new issue if needed.

blitzarx1 avatar Aug 18 '25 08:08 blitzarx1