Currently with the graphviz output, all leaves will be inlined. This makes the graph a bit more compact, but end up with a representation that is less true to the source, by adding function calls in single nodes with a f(*args) calling convention.
Without the inlining it would resolve this complaint from a blog post:
Note: Yes, it’s annoying that the graphical output writes parentheses like E("C"), whilst the actual egglog language uses prefix-style (E "C"). That seems to be a quirk of the rendering; all of the actual code will always use the latter style!
Alternatively, we could add an option in the serialization for inling leaves but keeping the format as s-expressions. That could be useful to explore in the future and adding a way to expose in the graphviz rendering the ability to inline leaves (and how many times to inline the leaves). For the moment, having something consistent seems preferable over something succinct, to help with learning/teaching.
Currently with the graphviz output, all leaves will be inlined. This makes the graph a bit more compact, but end up with a representation that is less true to the source, by adding function calls in single nodes with a
f(*args)calling convention.Without the inlining it would resolve this complaint from a blog post:
Alternatively, we could add an option in the serialization for inling leaves but keeping the format as s-expressions. That could be useful to explore in the future and adding a way to expose in the graphviz rendering the ability to inline leaves (and how many times to inline the leaves). For the moment, having something consistent seems preferable over something succinct, to help with learning/teaching.