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

Documentation warnings about large graphs #5184

Closed
albert-github opened this issue Jan 26, 2025 · 3 comments · Fixed by #5192
Closed

Documentation warnings about large graphs #5184

albert-github opened this issue Jan 26, 2025 · 3 comments · Fixed by #5192
Labels
type:Documentation Documentation improvement or change

Comments

@albert-github
Copy link
Contributor

Description

When creating the documentation (locally, as I don't see them on cdash, https://open.cdash.org/viewBuildError.php?type=1&buildid=10169225, might be that I'm missing there something) we get the warnings:

warning: Included by graph for 'itkImageToImageFilter.h' not generated, too many nodes (153), threshold is 150. Consider increasing DOT_GRAPH_MAX_NODES.
warning: Inheritance graph for 'itk::ImageToImageFilter' not generated, too many nodes (209), threshold is 150. Consider increasing DOT_GRAPH_MAX_NODES.
warning: Inheritance graph for 'itk::Object' not generated, too many nodes (243), threshold is 150. Consider increasing DOT_GRAPH_MAX_NODES.

Expected information

No warning

Actual information

See Description

Versions

Current master: acaddf4

Additional Information

There are the solutions

For a full list of the available commands and their documentation see https://www.doxygen.nl/manual/commands.html
For a full list of commands and the doxygen version in which they are introduced see https://doxygen.github.io/doxygen-docs/da/de1/pg_cmds.html

@albert-github albert-github added the type:Documentation Documentation improvement or change label Jan 26, 2025
@dzenanz
Copy link
Member

dzenanz commented Jan 27, 2025

Increasing DOT_GRAPH_MAX_NODES seems like the right solution. There are many issues you have identified over the last few days, and in most of them you suggested a solution. Thank you!

Is there any chance you could produce a patch (or patch set) which applies the suggestions? If you are still having trouble creating a PR, zipping up the patch (set) and sharing it in a comment would work.

@albert-github
Copy link
Contributor Author

Indeed I've been a bit busy and I still have problems with creating a PR, for a number of issues it would be possible for me to generate a patch (Unix diff format that can be applied with the patch command) and provide it as a comment in a zip file.

There are a number of issues though where there is first some advise necessary (like you have given here in the comment #5184 (comment)).

I'll see what I can do.

@albert-github
Copy link
Contributor Author

Proposed patch: diff.patch

dzenanz added a commit to dzenanz/ITK that referenced this issue Jan 28, 2025
This should prevent warnings about large graphs. Closes InsightSoftwareConsortium#5184.
@dzenanz dzenanz closed this as completed in cc0de40 Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:Documentation Documentation improvement or change
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants