Image uniforms get reset after each draw using a shader

[nakednous]

Most appropriate sub-area of p5.js?

• Accessibility
• Color
• Core/Environment/Rendering
• Data
• DOM
• Events
• Image
• IO
• Math
• Typography
• Utilities
• WebGL
• Build process
• Unit testing
• Internationalization
• Friendly errors
• Other (specify if possible)

p5.js version

1.6.0 onwards

Web browser and version

No response

Operating system

No response

Steps to reproduce this

I've implemented shadow mapping in p5.js v1.5.0 using p5.Graphics and the p5.treegl library. While this works seamlessly in v1.5.0, I'm facing challenges with versions from p5.js v1.6.0 onwards. I am planning to transition to using p5.Framebuffer once this issue is resolved.

Steps to Reproduce:

1. Run the shadow mapping implementation in p5.js v1.5.0 here.
2. Attempt the same using p5.js v1.9.3 here.

I've verified that the p5.treegl commands function identically in both versions. Notably, p5.js v1.6.0 introduced significant updates to shader-related features. I'm unsure if this issue qualifies as a bug or if I should report it or ask for help at the p5.js forum.

@davepagurek, could you possibly take a look and provide some guidance?

[davepagurek]

Thanks for catching this! After #5923, textures are unbound when the shader is unbound to prevent some other bugs (keeping textures bound when not using them causes a lot of other downstream bugs). However, we unbind the shader immediately after each draw, so only the first shape has the depth texture bound, and then subsequent shapes have an empty texture. Right now, it means I guess you'll have to re-apply the uniform before each shape you draw.

To fix this in p5, maybe we need to keep track of what textures shader had bound the last time it was used so that we can re-bind those values when you use the shader again. That way textures will work like other uniforms.

[nakednous]

Thanks for the suggestion! I can confirm that reapplying the depth uniform before rendering each shape serves as a workaround.

[davepagurek]

If anyone is interested in taking this on, I think what we would need to do is adjust setUniform to store the last set texture on the shader:

p5.js/src/webgl/p5.Shader.js

Lines 957 to 960 in bcf9134

	case gl.SAMPLER_2D:
	gl.activeTexture(gl.TEXTURE0 + uniform.samplerIndex);
	uniform.texture =
	data instanceof p5.Texture ? data : this._renderer.getTexture(data);

...and then re-apply the last used texture value, if it exists, when you bind the shader again in _setFillUniforms and _setStrokeUniforms:

p5.js/src/webgl/p5.RendererGL.js

Lines 2071 to 2072 in bcf9134

	_setFillUniforms(fillShader) {
	fillShader.bindShader();

One complication: since this gets called after each draw finishes, this call below would end up storing an empty image when we really just want it to unbind the current image:

p5.js/src/webgl/p5.Shader.js

Lines 573 to 577 in bcf9134

	unbindTextures() {
	for (const uniform of this.samplers) {
	this.setUniform(uniform.name, this._renderer._getEmptyTexture());
	}
	}

So maybe unbindTextures should just do this bit instead of calling setUniform?

p5.js/src/webgl/p5.Shader.js

Lines 959 to 960 in bcf9134

	uniform.texture =
	data instanceof p5.Texture ? data : this._renderer.getTexture(data);

[JordanSucher]

I tried to recreate #5923 so I could understand the issues that arise if we don't unbind a texture, but I found that when I commented out the unbindTextures() call, it didn't seem to cause any problems - tried the sketch from that issue, fill renders as expected.

Is it possible we don't need to have that call anymore?

red.cube.mov

[davepagurek]

I made a new sketch on the most recent p5 here: https://editor.p5js.org/davepagurek/sketches/pwIAXyF0X

With the implementation of unbindTextures empty, the fills still stop working for me and I get the same error about framebuffer feedback in the console.

[JordanSucher]

very strange - that sketch works fine for me across chrome, safari, arc

[nakednous]

I tested this on Arch Linux.
It fills the texture in both Chromium and Falkon, but in Firefox it only does it the first frame (mousePress to reveal it).

[davepagurek]

It's a weird situation because it's left up to the driver developers to determine what's considered feedback or not, so on some platforms it works. Unfortunately we mostly have to program to the lowest common denominator, and for now that means cleaning up our textures when not using them.

[nakednous]

weirdly enough leaving unbindTextures empty, solves the shadow mapping issue here even in firefox.