powervr-graphics / Native_SDK

C++ cross-platform 3D graphics SDK. Includes demos & helper code (resource loading etc.) to speed up development of Vulkan, OpenGL ES 2.0 & 3.x applications
https://docs.imgtec.com/sdk-documentation/html/introduction.html
MIT License
702 stars 197 forks source link

[BUG] incorrect glDisable(GL_FRAMEBUFFER_SRGB) for MRT #41

Open zhaijialong opened 5 years ago

zhaijialong commented 5 years ago

Description

Hi,

I find the emulator will add a glDisable(GL_FRAMEBUFFER_SRGB) call in the underlying OpenGL implementation for every draw, if my current FBO contains SRGB color attachement in GL_COLOR_ATTACHMENT1 or 2, 3, etc.

If the GL_COLOR_ATTACHMENT0 is SRGB format, the emulator works correctly, it calls glEnable(GL_FRAMEBUFFER_SRGB).

Maybe the underlying implementation only check color attachement 0.

Repro steps

Environment

Additional Info

graptis commented 5 years ago

Hi @zhaijialong, Thanks for the report. I have forwarded this to the engineers responsible for PVRVFrame, but since this is a matter of Tools and not the actual codebase of the SDK, support will arguably be better in https://forums.imgtec.com/ which are for this purpose only. I would suggest opening an issue there (as well?). (We can still keep this issue open, but probably feedback will be sparse).

zhaijialong commented 5 years ago

@graptis Thanks for your reply. I signed in and openned a topic in https://forums.imgtec.com/, and then it says my account is "temporarily placed on hold" and the topic can't be seen.

graptis commented 5 years ago

Hi @zhaijialong , sorry about that something strange happened when your account was opened, IT tells me it should be OK now.

zhaijialong commented 5 years ago

https://forums.imgtec.com/t/issue-bug-incorrect-gldisable-gl-framebuffer-srgb-for-mrt/2951/3