Closed seankmartin closed 11 months ago
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
View this failed invocation of the CLA check for more information.
For the most up to date status, view the checks section at the bottom of the pull request.
This definitely seems like a good option, but what do you think about using the existing "Resolution" control (that currently does nothing) but modifying it to display the different resolutions in units of "voxel size / view frustum depth"? That way this "Resolution" control would behave similarly to the other Resolution controls and would no longer be a "display only" control.
I really like that idea. My hope was that the implementation here would be a stepping stone to something more robust, and that is a good suggestion.
We'd essentially be changing up the logic from the resolution being determined by a user setting of the near and far planes and samples per ray, to the number of samples per ray being determined by a user setting of the near and far planes and resolution level. Am I understanding that right?
Edit: Maybe we want to still leave the number of samples modifiable so that one can over/under sample the actual data resolution if they want to.
To be clear, I was imagining that the actual user setting would still be number of samples, as you have it now, it would just be controlled using the (modified) "Resolution (3d)" control rather than a separate control. In order for that to make sense, the "Resolution (3d)" control would need to display the number of samples in log scale, and plot each resolution according to the number of samples that would be optimal for it, so that it displays in the correct position relative to the vertical bar that indicates the current "number of samples" selection.
Sure, I understand better now - I still like the idea!
I'll get to making that change in the coming days
Please resolve merge conflicts and then should be ready.
@jbms merge conflicts fixed!
Goal
Change the number of depth samples from a fixed 64 to a user-controlled variable. Named "samples per ray".
Primary changes