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

Integrate Dominik's Hair Compute Pass #15

Closed
1 of 6 tasks
CaffeineViking opened this issue Dec 4, 2018 · 1 comment
Closed
1 of 6 tasks

Integrate Dominik's Hair Compute Pass #15

CaffeineViking opened this issue Dec 4, 2018 · 1 comment
Assignees
Labels
Priority: Low Project: Renderer Issues relating to the core hair renderer itself. Type: Feature Something new and shiny is being requested.

Comments

@CaffeineViking
Copy link
Owner

CaffeineViking commented Dec 4, 2018

After solving #13 we should actually start trying to match the bindings of the constant buffer and creating the right storage buffers to match those in Dominik's implementation. We should take this in steps, and maybe compare the results of each pass side-by-side (i.e. VKHR and RealTimeHair) and see that they have similar results. I don't expect things to work out-of-the-box without some fixing, so let's do this iteratively.

I've currently inspected and verified local correctness of:

  • ReduceDepthBuffer
  • ClipCurves
  • PrefixSum_1
  • PrefixSum_2
  • Reorder
  • DrawStrands
@CaffeineViking CaffeineViking added Type: Feature Something new and shiny is being requested. Project: Renderer Issues relating to the core hair renderer itself. Priority: High labels Dec 4, 2018
@CaffeineViking CaffeineViking self-assigned this Dec 4, 2018
@CaffeineViking
Copy link
Owner Author

CaffeineViking commented Jan 27, 2019

This doesn't look like it'll happen, we'll instead roll our own PPLL-solution similar to TressFX. Maybe I can use some parts of the source code, but porting everything seems to be diverging from our current solution here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Low Project: Renderer Issues relating to the core hair renderer itself. Type: Feature Something new and shiny is being requested.
Projects
None yet
Development

No branches or pull requests

1 participant