AmplifyCreations / AmplifyShaderEditor-Feedback

4 stars 0 forks source link

Update – Procedural Sample #90

Open Dawie3565 opened 2 years ago

Dawie3565 commented 2 years ago

Procedural Sample has 4 samples in one function the name is very generic and not many know even what it is ... plus we will make more Procedural types of samples in future

better for users if you break this up into 4 functions see below RAW

-- Sample Procedural Planar Array http://paste.amplify.pt/view/raw/b7bf43bb

-- Sample Procedural Planar http://paste.amplify.pt/view/raw/079b8ebf

-- Sample Procedural Stochastic Array http://paste.amplify.pt/view/raw/e5c3c894

-- Sample Procedural Stochastic http://paste.amplify.pt/view/raw/1d5becd3

diogovtx commented 2 years ago

You're confusing stochastic with triplanar. I also don't agree with splitting. Really sorry but I'll have to close this one.

Dawie3565 commented 2 years ago

Planar = has a single texture set ... stochastic = has a single texture set

triplanar = has two types, 2 texture sets or 3 texture sets Spherical projection = 2 texture set top/bottom Cylindrical projection = 3 texture set top/middle/bottom

Dawie3565 commented 1 year ago

marked as competed waiting review. reference

DESF Sample Stochastic Planer PreCompute DESF Sample Stochastic Planer

DESF Sample Stochastic Triplanar PreCompute DESF Sample Stochastic Triplanar