Skip to content

Conversation

@mvaligursky
Copy link
Contributor

  • replaced hardcoded number by attribute / uniform

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR exposes the color band width parameter for the radial reveal effect, allowing developers to customize the width of the color bands for dot and lift waves. Previously, these widths were hardcoded values (1.0 for dot wave, 1.5 for lift wave leading edge, 0.5 for lift wave trailing edge).

Changes:

  • Added uBandWidth uniform to both GLSL and WGSL shader implementations
  • Replaced all hardcoded band width values with expressions using the new uniform
  • Added bandWidth attribute to the GsplatRevealRadial class with a default value of 1.0 for backward compatibility

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mvaligursky mvaligursky merged commit 903304a into main Jan 28, 2026
13 checks passed
@mvaligursky mvaligursky deleted the mv-radial-reveal-params branch January 28, 2026 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: graphics Graphics related issue enhancement Request for a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants