Skip to content

Volume Scattering Test Assets#269

Open
MiiBond wants to merge 7 commits into
mainfrom
mbond/volume_scatter_anisotropy
Open

Volume Scattering Test Assets#269
MiiBond wants to merge 7 commits into
mainfrom
mbond/volume_scatter_anisotropy

Conversation

@MiiBond
Copy link
Copy Markdown
Contributor

@MiiBond MiiBond commented Mar 10, 2026

Here are a couple of possible conformance assets to test KHR_materials_volume_scatter.

The first one test varying values for anisotropic scattering:
screenshot_raytracer

The second one tests varying scattering colours and attenuation distances while attenuationColor is held constant. As multiscatterColorFactor tends to black, scattering is replaced by attenuation.
screenshot_raytrace

@MiiBond MiiBond changed the title Volume Scatter Anisotropy asset Volume Scattering Test Assets Mar 10, 2026
@MiiBond
Copy link
Copy Markdown
Contributor Author

MiiBond commented Mar 10, 2026

Here are renders in Babylon.js:

screenshot_rasterizer screenshot_rasterizer

@MiiBond
Copy link
Copy Markdown
Contributor Author

MiiBond commented Mar 25, 2026

I've updated the second asset now to vary multiscatterColorFactor from yellow to black on the X-axis and vary attenuationDistance on the Y-axis. When scattering is black, there will no longer be any scattering. However, the attenuationColor is constant for every material and so the scattering will be replaced by absorption as the multiscatterColorFactor tends to black.

I also added text labels for the axes.

@emackey
Copy link
Copy Markdown
Member

emackey commented Apr 27, 2026

During the PBR meeting we talked about the need for updated screenshots here.

Comment thread Models/ScatterAnisotropy/metadata.json Outdated
"showcase",
"extension"
],
"screenshot": "screenshot/screenshot_raytrace.png",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks like the filename is missing an r on the end, screenshot_raytracer.png.

That's why the CI failed.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch. Fixed.

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.

2 participants