Skip to content

C++ Class Diagrams#147

Draft
DiegoDeGusem wants to merge 15 commits intodevfrom
diagram
Draft

C++ Class Diagrams#147
DiegoDeGusem wants to merge 15 commits intodevfrom
diagram

Conversation

@DiegoDeGusem
Copy link
Copy Markdown
Contributor

@DiegoDeGusem DiegoDeGusem commented Apr 24, 2026

This PR would automatically generate class diagrams each time the website gets build and they will be added to the website. There are a total of 6 diagrams to avoid a cluttered image.

Class Diagrams

Parameters
image
Magnetics
image
Strayfield
image
Time Integration
image
Domain
image
Utilities
image

@DiegoDeGusem DiegoDeGusem self-assigned this Apr 24, 2026
@DiegoDeGusem DiegoDeGusem added the documentation Improvements or additions to documentation label Apr 24, 2026
Comment thread docs/conf.py Outdated
Comment thread docs/conf.py
@DiegoDeGusem DiegoDeGusem marked this pull request as draft April 28, 2026 09:24
@DiegoDeGusem DiegoDeGusem marked this pull request as ready for review April 28, 2026 10:07
Comment thread environment.yml Outdated
Comment thread docs/conf.py Outdated
@DiegoDeGusem DiegoDeGusem requested a review from ilateur April 29, 2026 12:27
@ilateur ilateur added this to the v1.2.1 milestone Apr 29, 2026
Copy link
Copy Markdown
Collaborator

@ilateur ilateur left a comment

Choose a reason for hiding this comment

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

I was unable to generate these images myself (on monster5). I tried installing clang-uml via conda but I got errors with make html (I did not try to solve them). I am not allowed to install clang-uml via the package manager there, but Lars tried and failed that install.

It would be best if everything was included inside the environment.yml as a self-contained package that works for everybody, but somehow that did not work on Windows?

The actual diagrams look nice, although I wasn't able to judge them on the website. Sadly I don't think we should merge this if only 1 person is able to successfully run it.

Comment thread docs/diagrams/plantuml.jar Outdated
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Damn, big file

Comment thread .gitignore
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just checking, these are generated files, but are themselves not needed by the website?

Comment thread README.md
```

The documentation can then be found at `docs/_build/html/index.html`.
The documentation can then be found at `docs/_build/html/index.html`. However, to generate the class diagrams you should be on a Linux device and first install [clang-uml](https://github.com/bkryza/clang-uml/tree/master) otherwise they will not be generated.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why not Windows?

@DiegoDeGusem
Copy link
Copy Markdown
Contributor Author

I will move this back to draft and see if I can fix the issues.

@DiegoDeGusem DiegoDeGusem marked this pull request as draft May 7, 2026 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants