bjin / mpv-prescalers

prescalers for mpv, as user shaders
GNU Lesser General Public License v3.0
355 stars 34 forks source link

Update README.md #17

Closed JulianLai closed 7 years ago

bjin commented 7 years ago

Thanks for the PR.

The reason why I removed these description is that they are inaccurate and could be misleading. Instead of specified OpenGL version, they actually maps to ARB_texture_gather and ARB_compute_shader extension. There are also cases where ARB_texture_gather is supported but just don't actually work. Compute shader also requires other extensions to work, and might require some being enabled in mpv explicitly. Those performance declaration is also not necessarily always true on all hardware.

So, no, I'm fine with current description, the less verbose one. I actually wish user could try those shaders to see if they actually work, and how they perform.