Shader Categories

[tebjan]

make them advanced and categories: Rendering\ImageShaders Rendering\DrawShaders Rendering\ComputeShaders

[tebjan]

In general:

• Suffix defines shader shows up in node browser
• FX has a meaning in vvvv land, expectations is that a handy wrapper node gets generated (e.g. draw call + render target/buffer)
• we generate the raw/pure shader node as well, for free

Current state:

_VLNode (no FX)

• drawing shader node Stride.Rendering.EffectLib
• no Shader suffix on the node
• no wrapper node generated

_TextureFX

• image shader in Stride.ImageShaders
• has Shader suffix
• TextureFX wrapper node in Stride.TextureFX

_ComputeFX

• compute shader in Stride.ComputeShaders
• has Shader suffix
• (includes dispatch (draw call) and has dispatcher input)
• no wrapper node generated

Issues:

• inconsistency between VLNode and FX
• user asks why only one of them is called VLNode, but the others also generate nodes
• raw shader nodes show up top-level

Cleanup:

• always have FX in the suffix
• always generate two nodes (when complete)
• pure/raw shader node in Shaders category with Shader suffix
• handy wrapper in FX category

Shader categories: Rendering.DrawShaders Rendering.ImageShaders Rendering.ComputeShaders

FX categories: Rendering.DrawFX (after node20) Textures.TextureFX Rendering.ComputeFX or Buffers.ComputeFX (after node20)

So in detail:

_DrawFX (change)

• drawing shader node Rendering.DrawShaders (change)
• has Shader suffix (change)
• wrapper node including draw call/MeshRenderer (after node20)

_TextureFX

• image shader in Rendering.ImageShaders (change)
• has Shader suffix
• wrapper node including render target

_ComputeFX

• compute shader in Rendering.ComputeShaders (change)
• has Shader suffix
• wrapper node including target buffers call (after node20)

[joreg]

bin gegen Rendering.Shaders.XX jede subcategory braucht einen klick mehr und machts unübersichtlicher

bin für: Rendering.XXShaders wie zuletzt besprochen (siehe initial post above)

[tebjan]

yes, also fine. updated.