Fix T39242 no support for backface culling in wireframe and sculpting except for dyntopo.
AbandonedPublic
Actions

Authored by Antonis Ryakiotakis (psy-fi) on Mar 18 2014, 3:10 PM.

Details

Reviewers

Maniphest Tasks

T39242: Backface culling in sculpt mode with dynamic topology

Summary

Support for dyntopo was there because we used polygons with polygon mode
line, so culling could work. In other cases we just did line drawing.

Fix is twofold:

Always use pbvh faces in sculpting
Support mixed triangle/quad rendering. This was done because sculpting

rendering code used triangles only in meshes for polygons.
This is not such a big change but it may introduce performance
regressions becuase quads can lose performance on modern hardware so
this commit may well be reverted in the future.

Diff Detail

Branch: occlude_fix

Event Timeline

Hi Morten, there was a bug report that could only be fixed by enabling mixed quad/triangle rendering in sculpting. This is not such a big change setup/wise but I seem to remember there were serious reasons to not prefer quads? Could you please share your opinion? If this is the case then the bug report will be rejected of course.

Thanks!

Antonis Ryakiotakis (psy-fi) abandoned this revision.Mar 19 2014, 11:23 PM