Skip to content

Fix Documentation - #270

Merged
merakulix merged 4 commits into
mainfrom
267-tixi-documentation-missing-docstring-generated-documentation
Aug 3, 2026
Merged

Fix Documentation#270
merakulix merged 4 commits into
mainfrom
267-tixi-documentation-missing-docstring-generated-documentation

Conversation

@joergbrech

@joergbrech joergbrech commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Fixes #267.

The issue is that newer doxygen versions are more strict when parsing the beginning and end marks for doxygen groups. Previously, the documentation was most likely built with doxygen 1.8.x and now we are using 1.13.2.

The/*@{*/ had to be replaced with /**@{*/.

In addition to this, I fixed a wrong argument documentation and updated the doxyfile using doxygen -u to get rid of some warnings. And I updated the pixi lock file.

Note that the large changes in Doxyfile.in and pixi.lock are auto-generated by doxygen and pixi respectively.

In doxygen 1.13.2 there is still an issue that breaks page scrolling, doxygen/doxygen#11379. I manually fixed it by modifying resize.js in the gh-pages branch. The issue is resolved in doxygen 1.14.0, which is not yet on conda-forge.

@joergbrech joergbrech linked an issue Jun 9, 2026 that may be closed by this pull request
@joergbrech
joergbrech requested a review from merakulix June 9, 2026 10:55

@merakulix merakulix left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything seems to be there, but when I build the docs locally, scrolling is not possible.

@joergbrech

Copy link
Copy Markdown
Contributor Author

Everything seems to be there, but when I build the docs locally, scrolling is not possible.

Did you see my comment?

In doxygen 1.13.2 there is still an issue that breaks page scrolling, doxygen/doxygen#11379. I manually fixed it by modifying resize.js in the gh-pages branch. The issue is resolved in doxygen 1.14.0, which is not yet on conda-forge.

@merakulix

Copy link
Copy Markdown
Contributor

Sorry, sure, cosidering these changes it looks good!

@merakulix
merakulix merged commit 69c44a8 into main Aug 3, 2026
13 checks passed
@merakulix
merakulix deleted the 267-tixi-documentation-missing-docstring-generated-documentation branch August 3, 2026 11:01
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.

tixi documentation missing docstring generated documentation

2 participants