Fixed a few more GL tests. (#1658)

Co-authored-by: Branimir Karadzic <branimirkaradzic@gmail.com>

Author: bkaradzic-microsoft

Apps/Playground/Scripts/config.json

@@ -30,11 +30,7 @@
         {
             "title": "NME Shadow Map",
             "playgroundId": "#M3QR7E#83",
-            "excludedGraphicsApis": [ "OpenGL" ],
-            "referenceImage": "nmeshadowmap.png",
-            "comments": {
-                "OpenGL": "xvfb : invalid enum with glRenderbufferStorage"
-            }
+            "referenceImage": "nmeshadowmap.png"
         },
         {
             "title": "Procedural texture with NME",
@@ -46,12 +42,8 @@
             "title": "GLTF Extension EXT_mesh_gpu_instancing",
             "playgroundId": "#QFIGLW#9",
             "renderCount": 10,
-            "errorRatio": 6,
-            "referenceImage": "gltfExtensionExtMeshGpuInstancingTest.png",
-            "excludedGraphicsApis": [ "OpenGL" ],
-            "comments": {
-                 "OpenGL": "Test works with OpenGL but it so slow that CI times out"
-            }
+            "errorRatio": 25,
+            "referenceImage": "gltfExtensionExtMeshGpuInstancingTest.png"
         },
         {
             "title": "GLTF ext MSFT_LOD",
@@ -72,11 +64,10 @@
             "title": "GLTF Extension KHR_materials_volume with attenuation",
             "playgroundId": "#YG3BBF#18",
             "referenceImage": "gltfExtensionKhrMaterialsVolumeAttenuation.png",
-            "excludedGraphicsApis": [ "OpenGL", "Metal" ],
-            "errorRatio": 1.2,
+            "excludedGraphicsApis": [ "Metal" ],
+            "errorRatio": 10,
             "renderCount": 10,
             "comments": {
-                "OpenGL": "xvfb : invalid enum with glRenderbufferStorage",
                 "Metal": "No object rendered"
             }
         },
@@ -152,10 +143,9 @@
             "title": "Glow layer and LODs",
             "playgroundId": "#UNS6ZV#2",
             "renderCount": 50,
-            "excludedGraphicsApis": [ "OpenGL", "D3D12", "Metal" ],
+            "excludedGraphicsApis": [ "D3D12", "Metal" ],
             "referenceImage": "glowlayerandlods.png",
             "comments": {
-                "OpenGL": "xvfb : invalid enum with glRenderbufferStorage",
                 "D3D12": "Mapping a texture fails with error HRESULT 0x887A0005",
                 "Metal": "Result doesn't match reference"
             }
@@ -255,10 +245,9 @@
         {
             "title": "Point light shadows",
             "playgroundId": "#U2F7P9#4",
-            "excludedGraphicsApis": [ "OpenGL", "Metal" ],
+            "excludedGraphicsApis": [ "Metal" ],
             "referenceImage": "point-light-shadows.png",
             "comments": {
-                "OpenGL": "xvfb : invalid enum with glRenderbufferStorage",
                 "Metal": "Rendered result doesn't match reference"
             }
         },
@@ -364,33 +353,30 @@
             "title": "Scissor test",
             "playgroundId": "#W7E7CF#34",
             "referenceImage": "scissor-test.png",
-            "excludedGraphicsApis": [ "D3D12", "OpenGL" ],
+            "excludedGraphicsApis": [ "D3D12" ],
             "comments": {
-                "D3D12": "reenable when automatic mip-maps issue is fixed in bgfx",
-                "OpenGL": "Incorrect rendering"
+                "D3D12": "reenable when automatic mip-maps issue is fixed in bgfx"
             }
         },
         {
             "title": "Scissor test with 0.9 hardware scaling",
             "playgroundId": "#W7E7CF#34",
             "replace": "//options//, hardwareScalingLevel = 0.9;",
             "referenceImage": "scissor-test-2.png",
-            "excludedGraphicsApis": [ "D3D12", "OpenGL" ],
+            "excludedGraphicsApis": [ "D3D12" ],
             "errorRatio": 50,
             "comments": {
-                "D3D12": "reenable when automatic mip-maps issue is fixed in bgfx",
-                "OpenGL": "Incorrect rendering"
+                "D3D12": "reenable when automatic mip-maps issue is fixed in bgfx"
             }
         },
         {
             "title": "Scissor test with 1.5 hardware scaling",
             "playgroundId": "#W7E7CF#34",
             "replace": "//options//, hardwareScalingLevel = 1.5;",
             "referenceImage": "scissor-test-3.png",
-            "excludedGraphicsApis": [ "D3D12", "OpenGL" ],
+            "excludedGraphicsApis": [ "D3D12" ],
             "comments": {
-                "D3D12": "reenable when automatic mip-maps issue is fixed in bgfx",
-                "OpenGL": "Incorrect rendering"
+                "D3D12": "reenable when automatic mip-maps issue is fixed in bgfx"
             }
         },
         {

CMakeLists.txt

@@ -35,7 +35,7 @@ FetchContent_Declare(base-n
     EXCLUDE_FROM_ALL)
 FetchContent_Declare(bgfx.cmake
     GIT_REPOSITORY https://github.com/BabylonJS/bgfx.cmake.git
-    GIT_TAG 7499c2fe6b7f403381f397c04325450312f74c2e
+    GIT_TAG 53d884a6e221f96fade72a15e0ad5f92ec7b66b5
     EXCLUDE_FROM_ALL)
 FetchContent_Declare(CMakeExtensions
     GIT_REPOSITORY https://github.com/BabylonJS/CMakeExtensions.git