Fix two bugs mentioned in T56865 (Spring weight paint bugs)
ClosedPublic
Actions

Authored by Germano Cavalcante (mano-wii) on Oct 24 2018, 3:35 PM.

Details

Reviewers

Commits

rBS603774c1eb8d: Fix T56865: Selection of bones not working properly if the option `In Front`…
rB603774c1eb8d: Fix T56865: Selection of bones not working properly if the option `In Front`…

Summary

The bugs are:

Fix bad depth test on weight paint while an armature is in pose mode.
Fix selection of bones not working properly if the option In Front is enabled.

To draw an object with OB_DRAWXRAY, the pose mode engine calls the GPU_framebuffer_clear_depth(dfbl->default_fb, 1.0f) which clears the depth of the default framebuffer that is used by all other engines.
So the solution is to move that engine to the end of the list.

Note: Since the workbench engine has its own framebuffer for XRAY, it may be a good idea to move this framebuffer to DefaultFramebufferList so that it can be used by other engines.

Diff Detail

Repository: rB Blender

Event Timeline

Germano Cavalcante (mano-wii) created this revision.Oct 24 2018, 3:35 PM

Fix selection of bones`In Front` not working properly.

I don't see any problem with the fact of reordering the engines.

But I do see a problem using the glDepthRange trick. I would prefer that you just use a clean depth buffer for that. But in my tests, it was not working for some reason.

So if you put glDepthRange in GPU_state.c and add a big XXX in the comment I will accept it.

source/blender/draw/modes/object_mode.c
2993	The rule of thumb is to avoid gl* calls directly in the engines. Instead use a wrapper inside the GPU module. This will make things easier if we one day decide to implement other backends.

This revision now requires changes to proceed.Oct 30 2018, 6:30 PM

Germano Cavalcante (mano-wii) mentioned this in rBbf5ba496989e: Partial fix to T56865: X-ray modes does not work for the weight paint overlay..Oct 30 2018, 7:21 PM