Merge pull request #125 from JuliaRobotics/fix/3Q19/56

draw graph circles squares

Author: GearsAD

src/GraphsDFG/services/GraphsDFG.jl

@@ -7,7 +7,7 @@ function attributes(v::GraphsNode, g::T)::AttributeDict where T <:GenericInciden
     AttributeDict(
         "label" => v.dfgNode.label,
         "color" => v.dfgNode isa DFGVariable ? "red" : "blue",
-        "shape" => v.dfgNode isa DFGVariable ? "box" : "ellipse",
+        "shape" => v.dfgNode isa DFGVariable ? "ellipse" : "box",
         "fillcolor" => v.dfgNode isa DFGVariable ? "red" : "blue"
         )
 end

src/entities/DFGVariable.jl

@@ -15,6 +15,8 @@ mutable struct VariableNodeData
   inferdim::Float64
   ismargin::Bool
   dontmargin::Bool
+  # Tonio surprise TODO
+  # frontalonly::Bool
   # A valid, packable default constructor is needed.
   VariableNodeData() = new(zeros(1,1), zeros(1,1), Symbol[], Int[], 0, false, :NOTHING, Symbol[], "", false, false, false, false)
   VariableNodeData(x1::Array{Float64,2},

test/interfaceTests.jl

@@ -303,7 +303,7 @@ end
     addVariable!(dotdfg, v2)
     addFactor!(dotdfg, [v1, v2], f1)
 
-    @test toDot(dotdfg) == "graph graphname {\n2 [\"label\"=\"b\",\"shape\"=\"box\",\"fillcolor\"=\"red\",\"color\"=\"red\"]\n2 -- 3\n3 [\"label\"=\"f1\",\"shape\"=\"ellipse\",\"fillcolor\"=\"blue\",\"color\"=\"blue\"]\n1 [\"label\"=\"a\",\"shape\"=\"box\",\"fillcolor\"=\"red\",\"color\"=\"red\"]\n1 -- 3\n}\n"
+    @test toDot(dotdfg) == "graph graphname {\n2 [\"label\"=\"b\",\"shape\"=\"ellipse\",\"fillcolor\"=\"red\",\"color\"=\"red\"]\n2 -- 3\n3 [\"label\"=\"f1\",\"shape\"=\"box\",\"fillcolor\"=\"blue\",\"color\"=\"blue\"]\n1 [\"label\"=\"a\",\"shape\"=\"ellipse\",\"fillcolor\"=\"red\",\"color\"=\"red\"]\n1 -- 3\n}\n"
     @test toDotFile(dotdfg, "something.dot") == nothing
     Base.rm("something.dot")