Provide support for other Graphviz layout commands

[danyill]

Currently kroki appears only to support the dot command of Graphviz

kroki/server/src/main/java/io/kroki/server/Server.java

Line 85 in 964b2ea

registry.register(new Graphviz(vertx, config, commander), "graphviz", "dot");

I would be keen to have support for additional layouts (as described in the documentation)

• circo
• dot
• fdp
• neato
• osage
• patchwork
• sfdp
• twopi

In each case these could be the name of the graph, with graphviz remaining dot for backwards compatibility.

WDYT?

[ggrossetie]

I would be keen to have support for additional layouts (as described in the documentation)

I believe that the layout can be defined as a graph attribute: https://graphviz.org/doc/info/attrs.html#d:layout

The default layout is dot but I think it's possible to use another layout. For instance:

digraph g1 {
  layout="circo";
  // ...  
}

For reference, the layout can also be defined as a command line argument: https://graphviz.org/doc/info/command.html#minusK but Kroki does not (yet) support command line arguments. See: #169

In any case, I don't think we should add each layout as diagram types. In my opinion, the diagram type should be graphviz and the layout should be defined as part of the diagram source or as an attribute (not supported yet).

[danyill]

Thanks for the prompt response. You are quite right, I hadn't noticed the `layout=` syntax. I think it would be a useful feature to allow specification of this within an attribute to support DRY behavior (e.g. to produce different layouts with the same source file) but it is by no means urgent. For the mean-time the `layout=` option is just fine for me.

…

On Sun, Aug 30, 2020 at 3:25 AM Guillaume Grossetie < ***@***.***> wrote: I would be keen to have support for additional layouts (as described in the documentation) I believe that the layout can be defined as a graph attribute: https://graphviz.org/doc/info/attrs.html#d:layout The default layout is dot but I think it's possible to use another layout. For instance: digraph g1 { layout="circo"; // ... } For reference, the layout can also be defined as a command line argument: https://graphviz.org/doc/info/command.html#minusK but Kroki does not (yet) support command line arguments. See: #169 <#169> In any case, I don't think we should add each layout as diagram types. In my opinion, the diagram type should be graphviz and the layout should be defined as part of the diagram source or as an attribute (not supported yet). — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#358 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFEXX26AJECJK5WAL7VXNDSDEMYNANCNFSM4QOW26NA> .

--

-- Private or confidential message? Public Key available here <https://pgp.mit.edu/pks/lookup?search=dan.mulholland%40gmail.com&op=index> :

[kukimik]

Isn't this solved by #827 (see #827 (comment))?

[ggrossetie]

That's correct, thanks.
It's now possible to specify diagram options: https://docs.kroki.io/kroki/setup/diagram-options/#_graphviz