Use UBOs in project module

[felixpalmer]

Background

Status:

• Requires luma 9.0.12
• Most shaders & layers working
• project64 module also ported
• All tests passing

Change List

• Update project module to use projectUniforms UBO
• Update references in all shaders, e.g. project_uDevicePixelRatio -> project.devicePixelRatio
• Remove project from DEFAULT_MODULES as it needs to be passed in the bindings
• project64 now passes fp64 matrix as a mat4 uniforms, rather than vec2[16]
• Test updates

[ibgreen]

Very nice, this is a big part of the lift required to make deck.gl WebGPU ready.

Hacky implementation as luma doesn't handle module dependencies

I am not sure exactly what that means, but is it something we can address in luma.gl to keep this complicated part of deck.gl a little cleaner?

[felixpalmer]

@Pessimistress @ibgreen landing this will be a breaking change for any custom shaders that reference project_uXXXX uniforms. Are we OK with this or do we want to consider automatically rewriting shaders by detecting "project_u" strings and replacing? We could warn that their shaders are using the old syntax and then force the upgrade in 9.1

[ibgreen]

Are we OK with this or do we want to consider automatically rewriting shaders by detecting "project_u" strings and replacing? We could warn that their shaders are using the old syntax and then force the upgrade in 9.1

I hadn't even thought about the possibility of making the uniform buffer change backwards compatible, but it would of course be nice. I recommend looking into the existing functionality in ShaderModules, perhaps it works as is, or can be made to work with small changes:

[coveralls]

coverage: 89.82% (+0.01%) from 89.807%
when pulling dac757d on felix/project-ubo
into 1a9dd6f on master.

[ibgreen]

Epic!

[donmccurdy]

Hi, just checking whether @felixpalmer's question in #8782 (comment) was resolved — are we comfortable with this PR going into a deckgl v9.0.x patch release, or should we hold it for v9.1? I'm working on publishing v9.0.15 now. Thanks!

EDIT: Discussed offline. Will leave this PR out of v9.0.x and hold it for v9.1.