kwiver icon indicating copy to clipboard operation
kwiver copied to clipboard

Vital mesh vertex_array has inconsistent dimension template

Open willdunklin opened this issue 3 years ago • 4 comments

mesh.h's mesh_vertex_array<d> claims to be an array of vertices of dimension d, however the typedef for the internal vertex representation vert_t is hardcoded to have a dimension of 3.

It seems to me that the internal representation should either reflect the description provided in the doc comments, or the template should be removed entirely. I can only find one use case where mesh_vertex_array<d> is used with a d != 3 here, and the fact that d = 2 has no effect on the usage.

willdunklin avatar Jun 27 '22 15:06 willdunklin