[Bf-viewport] OCIO update

[Clément FOUCAULT]

Hi Sergey,

OCIO is interleaved with blender's glaDrawPixelsTexScaled_clipping which
emit vertices. But ocio uses deprecated gl_texCoord and we can't bind VBOs
(now required) to this shader (as far as I know).

Steps to update are simple and I can do it already:
- Create a vertex shader for OCIO.
- Remove builtin gl_texCoord.
- Change code that uses glsl OCIO to not change the shader when using OCIO.

> Is that because GLSL 1.3 can't be used form inside 3.2 core?
I mostly believe that's the case.


2017-02-23 9:40 GMT+01:00 Sergey Sharybin <sergey.vfx at gmail.com>:

> Hi,
>
> We would surely need to support that, but for the time being we can
> disable that code fairly easily by returning false from OCIOImpl::
> supportGLSLDraw.
>
> I'm also not really sure what's the exact issue with using current
> shaders, those are in their own pipeline which is not interleaved with
> "main" viewport drawing pipeline. Is that because GLSL 1.3 can't be used
> form inside 3.2 core?
>
> On Thu, Feb 23, 2017 at 5:27 AM, Mike Erwin <significant.bit at gmail.com>
> wrote:
>
>> Clément brought this up in IRC — OCIO uses older GLSL and needs to be
>> updated. Is this something we need now, or can it be a todo task?
>>
>> Taking a quick look now... seems OCIO can generate GLSL 1.0 or 1.3, and
>> we're moving toward GLSL 3.3 for everything.
>>
>> Mike Erwin
>> musician, naturalist, pixel pusher, hacker extraordinaire
>>
>> _______________________________________________
>> Bf-viewport mailing list
>> Bf-viewport at blender.org
>> https://lists.blender.org/mailman/listinfo/bf-viewport
>>
>>
>
>
> --
> With best regards, Sergey Sharybin
>
> _______________________________________________
> Bf-viewport mailing list
> Bf-viewport at blender.org
> https://lists.blender.org/mailman/listinfo/bf-viewport
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.blender.org/pipermail/bf-viewport/attachments/20170223/eec5cd65/attachment.htm