search

cmlibs / zinc

Source code repository for OpenCMISS-Zinc
Mozilla Public License 2.0
15 stars 18 forks source link

Problems with partial graphics rebuild #88

Open rchristie opened 6 years ago

rchristie commented 6 years ago

When adding faces to existing surface graphics, some faces are not shown, or are visibly wrong with partial graphics rebuild (editing graphics vertex buffers). This can even cause crashes when rendering with invalid vertex buffers.

rchristie commented 6 years ago

The crash seemingly happens in OpenGL and can't be debugged. I haven't been able to reproduce it in a test: WebGL vertex data looks fine. I am attaching a Cmgui example which exhibits the issue. The crash is more-or-less guaranteed in release Cmgui when the comfile is all run together; when the define faces call is separately invoked, the expected surface graphics are typically missing, but the lines correctly appear. The crash is intermittent in debug Cmgui. part_surfaces.zip

rchristie commented 6 years ago

Investigation has shown there are some errors in notifying changes when faces are defined; parent elements should be marked as having their definition changes, new faces and lines must be marked as added.