Skip to content
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

Built-in graphviz alternative #51

Closed
sjoerdtalsma opened this issue Nov 21, 2017 · 4 comments · Fixed by #592
Closed

Built-in graphviz alternative #51

sjoerdtalsma opened this issue Nov 21, 2017 · 4 comments · Fixed by #592

Comments

@sjoerdtalsma
Copy link
Contributor

Since we already bundle PlantUML with the doclet, including a few extra libraries may not be so bad to reach an all-in-one solution without depending on graphviz.

A working solution is described here:

It may be possible to include this in the Java 9 (v2.0) doclet rewrite.

@sjoerdtalsma sjoerdtalsma self-assigned this Nov 21, 2017
@sjoerdtalsma sjoerdtalsma added this to the 2.0.0 milestone Nov 21, 2017
@sjoerdtalsma sjoerdtalsma changed the title Investigate built-in pure-java graphviz alternative Built-in pure-java graphviz alternative Nov 21, 2017
@sjoerdtalsma sjoerdtalsma removed this from the 2.0.0 milestone Feb 14, 2018
@sjoerdtalsma
Copy link
Contributor Author

We could perform a rudimentary test for Graphviz availability (e.g. check for the GRAPHVIZ_DOT environment variable) and if it is not available, fall back on jdot by adding the following pragma to the diagrams:

!pragma graphviz_dot jdot

This may be low-hanging fruit, as jdot is pre-bundled with plantuml

@sjoerdtalsma
Copy link
Contributor Author

sjoerdtalsma commented Nov 21, 2018

2018-11-21 Update: Positioning looks ok, but arrows are still broken:

image

@sjoerdtalsma sjoerdtalsma changed the title Built-in pure-java graphviz alternative Built-in graphviz alternative Mar 14, 2019
@huffstler
Copy link

PlantUML might have already gotten the work done for you on this issue 👀

https://plantuml.com/smetana02
plantuml/plantuml#1346

@sjoerdtalsma
Copy link
Contributor Author

sjoerdtalsma commented Mar 29, 2024

Thanks for the heads-up @huffstler , it looks like the diagrams created with UMLDoclet are nearly there.
Only the footer gets rendered wrong still, but possibly I can find a way to work around that.

Would be great if we could run without graphviz with the all-in-one jar. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants