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.

Sign up for GitHub

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

Jump to bottom

Polygon_mesh_processing: unknown group in documentation #7692

Closed
albert-github opened this issue Sep 6, 2023 · 3 comments · Fixed by #7812
Closed

Polygon_mesh_processing: unknown group in documentation #7692

albert-github opened this issue Sep 6, 2023 · 3 comments · Fixed by #7812
Labels
Doc bug Pkg::PMP
Milestone
6.0-beta

Comments

@albert-github
Copy link
Contributor

albert-github commented Sep 6, 2023 •
edited
Loading

Adding a warning to doxygen gives me the warning:

.../Polygon_mesh_processing/include/CGAL/polygon_mesh_processing.h:66: warning: Found non-existing group 'namedparameters' for the command '@ingroup', ignoring command

and I cannot find \defgroup or \addtogroup for namedparameters.

  • where does this \ingroup belong to or should the line be removed?

It now lands on (as it would without the \ingroup command): doc_output/Polygon_mesh_processing/polygon__mesh__processing_8h.html
@afabri
Copy link
Member

afabri commented Sep 25, 2023

@sloriot, do you agree that we can just remove the whole doumentation block here?

@albert-github
Copy link
Contributor Author

The code of doxygen/doxygen#10289 has been integrated in master on GitHub so the results show up in the overnight documentation build.

This was referenced Oct 11, 2023
Closed
Closed
Closed
Closed
Closed
Closed
@albert-github
Copy link
Contributor Author

Might be that here there should be something similar to the BGL Package where we see:

BGL/doc/BGL/NamedParameters.txt:\defgroup bgl_namedparameters Named Parameters

@afabri afabri mentioned this issue Oct 19, 2023
Merged
sloriot added a commit that referenced this issue Oct 25, 2023
@sloriot
PMP: Remove documentation of deprecated code (#7812)
256e77f 
## Summary of Changes

cleanup

## Release Management

* Affected package(s): PMP
* Issue(s) solved (if any): fix #7692
@lrineau lrineau added this to the 6.0-beta milestone Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Doc bug Pkg::PMP
Projects
None yet
Milestone
6.0-beta
Development

Successfully merging a pull request may close this issue.

PMP: Remove documentation of deprecated code afabri/cgal
4 participants
@afabri @MaelRL @albert-github @lrineau