why separate from dx ?

[akva2]

any reason for this to be separate from the windows version? is it the list of shaders, are the shaders different or what's the deal?

[popcornmix]

Yes, list of shaders are different. Only a small percentage of shaders designed for desktop use are usable on arm platforms.
Also arm platforms generally can't maintain a decent framerate at full (e.g. 1080p) resolution, so we render to a smaller texture and scale that up.
GL and GLES are already sufficiently different that we need a number of ifdefs - I wonder if adding dx to the mix would have much common code at all. It probably could be refactored to separate the rendering code from the control code, with separate back-ends for dx/gl/gles, but that is not a trivial change.
And we still have the issue that kodi can't distinguish a GL from a GLES implementation of an add-on so they can't live in the same repo.

[akva2]

reason i ask is that i did https://github.com/notspiff/screensaver.shadertoy/tree/add_gl_gles which is a copy-pasty draft for gl(es) before i realized this one exists.

it's not so much about code sharing as it is about having a single upstream in my mind. thus,
the repo issue is orthogonal as such.

i guess i will add gl-only support to the dx one, and then we'll have "desktop" and "embedded" versions.

[popcornmix]

Your code does seems to be based on this repo's code (measure_performance and determine_bits_precision were originally written by me). Perhaps you picked it from a repo based on this one? I'll try and test it when I get a chance.

Supporting shadertoy's recent multipass rendering would be a nice thing to support if you are working on this - it makes shaders much more powerful (without needing much more powerful hardware).

[notspiff]

(sorry for being schizo, so many devices);

sure, it's your code from visualization.shadertoy that i copy-pasted and modified. where we do have one common upstream (dx code in separate file, gl/gles a nasty ifdef spaghetti).

i wasn't trying to push my version, just to give some fuel to my fire; the multiple upstreams cause confusion (and in this case partially duplicated work). the version here is better i'm sure.