-
Notifications
You must be signed in to change notification settings - Fork 295
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
feat: change the number of depth samples in volume rendering to a controllable UI parameter #495
Conversation
This is not yet linked up to the backend in this commit
Wrapped property and tool
Linked samples per ray changing to the RPC so that the chunk manager can recompute priorities. Also trigger redraw on samples per ray change.
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 |
Sets min and max in log2 scale, and prevents mouse wheel update from leaving those bounds.
rendering depth samples tool
Had accidentally started the volume rendering scale at 4, not 2 Also remove accidental console.log
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