GXAPI mismatch of validation in DrawSurface3DEx

[ianneilmacleod]

The validity are inconsistent so that one cannot pass colors in a VV.

geoengine.map\mview\gxx_mvw.c, line 3260 compares to lNT, number or triangles:

ASSERT_GXX(lNT == Len_VV(hColorGVV->hVV), modn, "Color VV length");

is inconsistent with geoengine.map\gensurf.c line 195 which compares to lNVert, number of verticies:

ASSERT(Len_VV(hVVc) == lNVert);

[ianneilmacleod]

And this toy script does not draw all the faces, only the first two. The viewer recognizes the faces because the extents are correct, but only the first two faces are drawn. Note that render_backfaces is True so I don't believe this is the result of a winding the normals the wrong way.

:

verts = np.array([[0, 0, 0],
                  [5, 0, 0],
                  [5, 5, 0],
                  [0, 3, 5],
                  [2.5, 2, 10],
                  [-3, 6, 8],
                  [-4, 0, 12]], dtype=np.float64)
faces = np.array([[0, 1, 2],
                  [0, 2, 3],
                  [3, 2, 4],
                  [1, 2, 4],
                  [3, 4, 5],
                  [6, 4, 5]], dtype=np.int32)

with gxv.View_3d.new() as v3d:
    v3d_file = v3d.file_name
    with gxg.Draw_3d(v3d, 'Surface') as g:
        g._surface(faces, verts)
gxviewer.view_document(v3d_file, wait_for_close=True)

[ianneilmacleod]

Now redundant, part of GXAPI

[ianneilmacleod]

Reopening, assigning to "future". See the TODO's in group.py

This feature is not currently used by gxpy, so the impact only affects calling the gxapi directlyu