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

Add graph output #27

Merged
merged 2 commits into from
Jan 28, 2020
Merged

Add graph output #27

merged 2 commits into from
Jan 28, 2020

Conversation

loebl
Copy link
Contributor

@loebl loebl commented Jan 10, 2020

I am currently testing doxyrest for a project which makes use of embedded graphs (mainly dot and plantuml). I noticed the graph code is currently passed through as paragraph and figured out it isn't much needed to correctly wrap it in sphinx directives rendering them.

Added frame code and a short explanation for the manual.

Missing is output for diafiles (\diafile) as there doesn't seem a sphinx module available. As well as the file commands (\dotfile, \mscfile). Enabling support for the latter two is possibly not complicated, but I haven't looked into it yet. If I get the chance I might provide another patch.

Greetings
loebl

Supports 3 out of 4 graph types supported by doxygen. For those 3
graph types there are sphinx extensions available to render them:

* dot -> sphinx.ext.graphviz
* plantuml -> sphinxcontrib.plantuml
* msc -> sphinxcontrib.mscgen

If the required extension isn't available, the graph code will not be
given in the html output which is also a plus IMO, as it normally
clutters the descritpion.

It seems there is no extension available for dia. Also missing currently
is support for graph file references, as well as passthrough of
captions.
@codecov-io
Copy link

Codecov Report

Merging #27 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #27   +/-   ##
=======================================
  Coverage   87.32%   87.32%           
=======================================
  Files          14       14           
  Lines        2043     2043           
=======================================
  Hits         1784     1784           
  Misses        259      259

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 47c9c80...c1f44fd. Read the comment docs.

@vovkos
Copy link
Owner

vovkos commented Jan 16, 2020

Thanks for your contribution! Could you please share some project which makes use of the newly added functionality? I wanted to play with it a little before merging it in.

@loebl
Copy link
Contributor Author

loebl commented Jan 18, 2020

I think I should be able to provide an example in the second half of next week. I can't provide the project I am using doxyrest for, as that's internal to our company, but deriving some examples how we use it should be possible.

@loebl
Copy link
Contributor Author

loebl commented Jan 27, 2020

Sorry for the delay.

Attached is an example containing one of each graph type (dot, msc, dia). It should build without errors (doxygen; doxyrest; sphinx). Graphs are rather simple and silly.

I currently can't think of any real project using graphs in their doxygen docs. (But there probably are some out there)

graphs.zip

@vovkos vovkos merged commit 18b65aa into vovkos:master Jan 28, 2020
@vovkos
Copy link
Owner

vovkos commented Jan 28, 2020

Thanks, it actually looks pretty cool!

I never used that before, but I might just start doing that :) Totally can see where it can be useful.

@loebl loebl mentioned this pull request Oct 19, 2021
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 this pull request may close these issues.

3 participants