-
Notifications
You must be signed in to change notification settings - Fork 30
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
export to graphml #32
Comments
@PaulSchweizer should I just add serialize_graphml and deserialize_graphml methods to the Graph class? I currently have this implemented in json so rewriting this to graphml will require little effort. |
@Anton-4 , good idea! Yes, adding these two methods to the graph would be really good. <graph id="{graph.name}" edgedefault="directed">
<node id="{node1.identifier}"/>
<node id="{node2.identifier>"/>
<edge id="{node1.outputs[output_name].name}-{node2.inputs[input_name].name}" source="{node1.identifier}" target="{node2.identifier}"/>
</graph> Not sure where to put the other attributes of the Node like "metadata", "class", "module" etc. but you'll find a way :D |
I started working on this and thought: do we really need full serialization and deserialization or just exporting of graph, nodes and edges without attributes like metadata, class, module... ? I mainly wanted to export to graphml to have a standardized format to feed into the visualizer. |
I see your point @Anton-4 but if we would fully serialize the graph like we currently do in json, we could use the graphml file to fully reconstruct the graph afterwards. Otherwise we would need to keep another (json) file for that purpose. |
I now found how I can cleanly add in nested xml. That issue was kind of holding me back. |
Amazing! Thanks @Anton-4 |
@Anton-4 I'm planning to put some time into flowpipe over the next weeks and am browsing through the backlog here, trying to get back into it and do some planning. Let me know if you're still working on this |
Perfect! Thanks for the links and the work you put into this already! I'm going to put some time into this after the compound plugs are done |
No description provided.
The text was updated successfully, but these errors were encountered: