Data not present in pydot graph.

[lakhanjadhav1]

🐛 Bug I used pydotgraph_str = programl.to_dot(prog_graph) But I can see node labels only. No features information available in dot file.

To Reproduce

Steps to reproduce the behavior:

1. Generate programl graph from any code
2. Convert that graph to dot using programl.to_dot()
3. That generated dot string doesn't have features info

Expected behavior

Environment

• ProGraML version (if building from source, use git rev-parse HEAD):
• How you installed ProGraML (source, pip):
• OS:
• Python version:
• Build command you used (if building from source):
• GCC/clang version (if building from source):
• Bazel version (if building from source):
• Versions of any other relevant libraries:

Additional context

[ChrisCummins]

You're right, pg.to_dot(G) only adds the text of the node when exporting to dotfile.

Is there a specific feature you're looking to serialize?

You would need to modify this function and build from source to embed extra features:

ProGraML/programl/graph/format/graphviz_converter.cc

Lines 170 to 201 in cd7d293

	template <typename T>
	void SetVertexAttributes(const Node& node, T& attributes) {
	attributes["label"] = GetNodeLabel(node);
	attributes["style"] = "filled";
	switch (node.type()) {
	case Node::INSTRUCTION:
	attributes["shape"] = "box";
	attributes["fillcolor"] = "#3c78d8";
	attributes["fontcolor"] = "#ffffff";
	break;
	case Node::VARIABLE:
	attributes["shape"] = "ellipse";
	attributes["fillcolor"] = "#f4cccc";
	attributes["color"] = "#990000";
	attributes["fontcolor"] = "#990000";
	break;
	case Node::CONSTANT:
	attributes["shape"] = "octagon";
	attributes["fillcolor"] = "#e99c9c";
	attributes["color"] = "#990000";
	attributes["fontcolor"] = "#990000";
	break;
	case Node::TYPE:
	attributes["shape"] = "diamond";
	attributes["fillcolor"] = "#cccccc";
	attributes["color"] = "#cccccc";
	attributes["fontcolor"] = "#222222";
	break;
	default:
	LOG(FATAL) << "unreachable";
	}
	}

I could see that this would be generally useful. Patches welcome!

Cheers,
Chris