search

AmplifyCreations / AmplifyShaderEditor-Feedback

4 stars 0 forks source link

Update Dither Node DitheringNode.cs #266

Closed Dawie3565 closed 3 weeks ago

Dawie3565 commented 9 months ago

step operation in dither is returning artifacts due to interpolator imprecision. one out of 16 fragments is 1, 16/16, which means it's doing input ≥ 1

suggestion change step into multiply

dataCollector.AddLocalVariable( UniqueId, "dither" + OutputId+" = step( dither"+ OutputId + ", "+ driver + " );" );

change to

dataCollector.AddLocalVariable( UniqueId, "dither" + OutputId+" = * ( dither"+ OutputId + ", "+ driver + " );" );

to replicate -- add dither node -- add vertex color and plug alpha into dither node -- connect dither into main block alpha -- connect float 0-1 into main block alpha cutoff check results with a small value for alpha cutoff, expected results with vertex color =1 should be solid however results some fragments will disappear.

Dawie3565 commented 9 months ago

Note contact user Nesterboy (discord) when action completed

Dawie3565 commented 8 months ago

Note testing this in URP returns error for syntax error: unexpected token '*'

Dawie3565 commented 7 months ago

closed in 1.9.3.3

Dawie3565 commented 7 months ago

open bumped to 1.9.3.4 balance from 1.9.3.3