Skip to content

Comments

[WIP] Expand shader documentation with feedback and sampling details#1124

Open
anikom15 wants to merge 2 commits intolibretro:masterfrom
anikom15:patch-1
Open

[WIP] Expand shader documentation with feedback and sampling details#1124
anikom15 wants to merge 2 commits intolibretro:masterfrom
anikom15:patch-1

Conversation

@anikom15
Copy link

This is a work-in-progress. Do not merge yet. I just want to get the conversation started and get feedback. Still a lot of work to do. I am updating the shader doc to be more developer-friendly. This is largely based off of my own experience writing my own shaders. There are probably some formatting issues and typos I need to address. It's a big document.

Added detailed explanations and examples for shader feedback, texture sampling, and parameter support. Enhanced clarity on shader structure and best practices for multi-pass shaders. There are a bunch of new examples and a section on Advanced Techniques. Let me know what you think. I am okay with MIT license. It's largely academic after all. Would appreciate having my name on it somewhere however (anikom15 is fine; doesn't need to be my real name).

Also, this document originally started as a speculative 'how should we implement this?' idea sheet and then also morphed into developer doc and sort of user doc. We should focus this to be a developer-focused doc of how the system works now. A lot of that speculative tone was removed, but if there's something we need to adjust further, let me know.

Added detailed explanations and examples for shader feedback, texture sampling, and parameter support. Enhanced clarity on shader structure and best practices for multi-pass shaders.
Added detailed explanations for required shader stages, universal and scoped declarations, and provided a comprehensive example of a .slangp preset file. Clarified the usage of #pragma directives and improved the structure of the documentation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant