Skip to content
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

Refactor isosurface viewer #356

Open
astrofrog opened this issue May 15, 2020 · 0 comments
Open

Refactor isosurface viewer #356

astrofrog opened this issue May 15, 2020 · 0 comments

Comments

@astrofrog
Copy link
Member

The isosurface viewer is now very out of date. The current isosurface directory should be removed, and instead we should add an isosurface rendering option to the current volume viewer. I think the best way forward would be to add isosurface vs volume rendering as a volume layer option, and then if that mode is enabled, allow the user to show (for now) one contour level, which they can specify as a float. Then, the shaders should be updated so that for each ray propagation, it checks if the current texture needs to be rendered as an isosurface or a volume, and if an isosurface, we should use a similar approach to the current isosurface shader, where we first use bisection to find the actual level of the isocontour and then compute the color taking into account the lighting. All this should be reasonably straightforward, but the main trick will be to figure out how to combine the isosurface and volume rendering results. If this is too tricky to solve, we could also make volume vs isosurface a viewer-level option for now since that should be easier to do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant