Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lighting UBO #9074

Merged
merged 22 commits into from
Aug 19, 2024
Merged

Lighting UBO #9074

merged 22 commits into from
Aug 19, 2024

Conversation

felixpalmer
Copy link
Collaborator

@felixpalmer felixpalmer commented Aug 6, 2024

For #8997. Replaces the WebGL1 modules from luma with WebGL2 UBO enabled modules

Requires visgl/luma.gl#2159

Change List

  • Replace gouraudLighting with gouraudMaterial
  • Replace phongLighting with phongMaterial
  • Update shaderInputs for lighting, phongMaterial, & gouraudMaterial` when drawing

modules/core/src/lib/layer.ts Show resolved Hide resolved
@coveralls
Copy link

Coverage Status

coverage: 89.217% (+0.01%) from 89.207%
when pulling 2daddef on felix/lighting-ubo
into 5a90f5f on master.

@felixpalmer felixpalmer merged commit e101223 into master Aug 19, 2024
4 checks passed
@felixpalmer felixpalmer deleted the felix/lighting-ubo branch August 19, 2024 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants