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

Discussion on improved doxygen documentation for generated code #199

Open
tmadlener opened this issue Jul 20, 2021 · 1 comment
Open

Discussion on improved doxygen documentation for generated code #199

tmadlener opened this issue Jul 20, 2021 · 1 comment

Comments

@tmadlener
Copy link
Collaborator

This came up during todays meeting in the context of an improved documentation for EDM4Hep. The currently generated doxygen documentation lists all of the classes that are generated, which can be a bit overwhelming for newcomers or users who simply want to use EDM4Hep without having to worry about the "implementation details" of podio (e.g. the Obj and Data classes or all the different Iterators that are present). It would be nice to have some sort of grouping of the different classes into a "user interface" and "podio implementation classes".

One possibility would be to put all the podio layer classes into a separate (sub)namespace, which would make the grouping by namespaces trivial in doxygen (and also would be visible in the code). Another possibility could be to investigate how doxygen can be configured to group the different classes and potentially add the necessary things to the generated classes (e.g. labels).

A slightly different topic is the fact that currently only the user facing and the Data classes actually document their members, while the podio implementation classes in general do not. Also this could be improved.

@tmadlener tmadlener changed the title Discussion on improved doxygen documentation Discussion on improved doxygen documentation for generated code Nov 11, 2022
@tmadlener
Copy link
Collaborator Author

tmadlener commented May 7, 2024

Possible solutions:

  • Namespaces for grouping
  • Using doxygen groups
  • Filter out things using doxygen

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

No branches or pull requests

1 participant