Skip to content

Conversation

jwnimmer-tri
Copy link
Collaborator

@jwnimmer-tri jwnimmer-tri commented Oct 7, 2025

The requires clause was a problem for Doxygen:

drake/geometry/proximity/triangle_surface_mesh.h:299: error: argument 'p_MQ' of command @param is not found in the argument list of TriangleSurfaceMesh< T >::CalcBoundingBox() const
drake/geometry/proximity/triangle_surface_mesh.h:299: error: argument 't' of command @param is not found in the argument list of TriangleSurfaceMesh< T >::CalcBoundingBox() const
drake/geometry/proximity/volume_mesh.h:249: error: argument 'e' of command @param is not found in the argument list of VolumeMesh< T >::Equal(const VolumeMesh< T > &mesh, double vertex_tolerance=0) const
drake/geometry/proximity/volume_mesh.h:249: error: argument 'p_MQ' of command @param is not found in the argument list of VolumeMesh< T >::Equal(const VolumeMesh< T > &mesh, double vertex_tolerance=0) 

Amends #23527.


This change is Reviewable

The requires clause was a problem for Doxygen.
@jwnimmer-tri jwnimmer-tri added priority: emergency status: single reviewer ok https://drake.mit.edu/reviewable.html release notes: none This pull request should not be mentioned in the release notes labels Oct 7, 2025
@jwnimmer-tri
Copy link
Collaborator Author

@drake-jenkins-bot linux-noble-unprovisioned-gcc-bazel-experimental-documentation please

@SeanCurtis-TRI SeanCurtis-TRI self-assigned this Oct 7, 2025
Copy link
Contributor

@SeanCurtis-TRI SeanCurtis-TRI left a comment

Choose a reason for hiding this comment

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

:LGTM: X2

+a:@SeanCurtis-TRI

@SeanCurtis-TRI reviewed 4 of 4 files at r1, all commit messages.
Reviewable status: :shipit: complete! all discussions resolved, LGTM from assignee SeanCurtis-TRI(platform)

@SeanCurtis-TRI SeanCurtis-TRI merged commit b42bea6 into RobotLocomotion:master Oct 7, 2025
11 checks passed
@jwnimmer-tri jwnimmer-tri deleted the doc-hotfixes branch October 7, 2025 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: emergency release notes: none This pull request should not be mentioned in the release notes status: single reviewer ok https://drake.mit.edu/reviewable.html

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants