Added a shadertoy example #314
Conversation
|
Thanks for the example. I have done a quick code review and broadly it all looks good. I am heading out for the day soon so can't try it out right away. One thing I was curious about was the .shy shader extension, I haven't seen that done before. GLSL shaders would normally be .vert, .geom, .frag or .glsl etc. |
|
Thanks for the feedback! Regarding the "shy" extension (as in SHader toY), I made it up on my own to differentiate it from the "real" glsl shaders. Currently .shy files are turned into frag files within vsgshadertoy simply by wrapping them with some template code. I'd be happy if you could instruct me on how I can remove the MVP push constants, as well as other redundant code from the example. |
|
To get the ball rolling I have merged the example as is with a dov-shadertoy branch of vsgExamples: https://github.com/vsg-dev/vsgExamples/tree/dov-shadertoy I tried the shaders out an then all work :-) |
| toyShader = readFile(argv[1]); | ||
| string title; | ||
| // extract the filename from the path | ||
| auto pos = string(argv[1]).find_last_of("/"); |
There was a problem hiding this comment.
Would this work on Windows (which uses '')? Might be better to use std::filesystem's filename() function? (std::filesystem is C++17 so should be fine with VSG?)
There was a problem hiding this comment.
You're right. Won't work. I'll fix it.
There was a problem hiding this comment.
std::filesystem is officially part of C++17 so "should" be fine, except it's gcc and clang have only recently got there act together and properly support it out of the box, older versions don't properly support std::filesystem, it's why I had to resort to writing a vsg::Path class that is equivalent to std::filesystem::path.
The VSG has various path manipulator convenience functions:
https://github.com/vsg-dev/VulkanSceneGraph/blob/master/include/vsg/io/FileSystem.h
https://github.com/vsg-dev/VulkanSceneGraph/blob/master/include/vsg/io/Path.h#L218
Ironically these are more easily portable than the mess that was/is std::filesystem.
There was a problem hiding this comment.
Ok, I rewrote the code to use vsg::Path() instead of std::last_of("/")
|
It should be possible to pass the vsg::Path to the code that reads the shader file, rather than have it use std::string. vsg::Path handles the Windows wide string filename paths just file std::filesystem::path. |
- Use vsg::Path for paths instead of strings - Got rid of the redundant push constant - Removed `using std` as that is not the convention in vsgExamples - Changed the title so that it always display "vsgshadertoy"
- Removed the vertex and the index buffers - Removed their respective descriptors - Use single large triangle for covering quad like in https://www.saschawillems.de/blog/2016/08/13/vulkan-tutorial-on-rendering-a-fullscreen-quad-without-buffers/ - Change mouse behavior to react only if mouse is pressed
|
I just hardcoded the vertices coordinates in the vertex shader like in: This allowed me to remove the vertex and index buffers, and the binding descriptors for the vertices. It's less general as it doesn't draw an arbitrary mesh any longer, but it is an example of minimalism. |
This commit adds a shadertoy example. It runs a shadertoy shader from a file given on the command line. If no shader is given, then a default shader is run. The shadertoy directory contain another couple of example shaders. In addition a lot of the shaders from shadertoy can be run. (If a shader from shadertoy cannot be run, then it is bug that should be fixed).