search

natevm / BSDF-Visualizer

Project for scientific visualization
MIT License
18 stars 4 forks source link

Spawn brdf uniform sliders #29

Closed n8xm closed 6 years ago

n8xm commented 6 years ago

Resolve #13.

Changing color parameters is not implemented yet because only two of the existing analytical BRDFs from Disney support it. (I.e. it doesn't seem like a very useful feature, and a color picker is harder to implement than a slider / checkbox.) We will implement color picking if and when it becomes necessary.

natevm commented 6 years ago

Just checked this out. Checkboxes and sliders seem to come in alright with different BRDFs.

Numbers under the sliders look good.

We probably want a better solution for the checkboxes. They look a bit out of place at the moment. That might be something I put as an issue though.

n8xm commented 6 years ago

Since @n8vm mentioned it, I added polishing checkbox as issue #30