material.id is missing and leads to cache issues about pex-renderer HOT 4 CLOSED

Well that's why program.id is used. As program is cached by flags and those should contain all the required features. Do you have a test case?

from pex-renderer.

Test case:

• add log where pipeline creation happens in getGeometryPipeline:
  

  pex-renderer/systems/renderer/standard.js

  Lines 426 to 428 in 41759af

  	if (!pipeline || material.needsPipelineUpdate) {
  	material.needsPipelineUpdate = false;
  	pipeline = ctx.pipeline({

• go to gltf example TextureSettingsTest
• see it failed doubleSided test because it doesn't go through pipeline creation and reuse another material (I assume)
• add id: Material_${MaterialID++} to material component returned value, see test passing

from pex-renderer.

I think that id was omitted on purpose to avoid material explosion. For now i would double check if we cache all the flags eg culling and long term find better approach than dirty flag.

from pex-renderer.

Should be fixed in v4 so closing.

from pex-renderer.