disguise-one / RenderStream-UE

RenderStream plugin for Unreal Engine
BSD 3-Clause "New" or "Revised" License
69 stars 23 forks source link

OpenXR plugin spams the rslog #24

Closed bbuckley94 closed 2 years ago

bbuckley94 commented 3 years ago

Summary: In Unreal Engine 4.27 (Preview 3), it appears as though the OpenXR plugin is enabled by default. In certain projects the OpenXR plugin causes the rslog to be spammed when starting a RenderStream workload. We should suppress this.

Depending on the installation either of the following will be spammed: LogHMD: Failed to enumerate extensions. Please check that you have a valid OpenXR runtime installed.

2021-08-04T14:02:15.413+08:00: LogHMD: Optional extension XR_VARJO_quad_views is not available
2021-08-04T14:02:15.413+08:00: LogHMD: Required extension XR_EXT_eye_gaze_interaction is not available
2021-08-04T14:02:15.413+08:00: LogHMD: Could not enable all required OpenXR extensions for OpenXREyeTracker on current system. This plugin will be loaded but ignored, but will be enabled on a target platform that supports the required extension.
2021-08-04T14:02:15.431+08:00: LogHMD: Failed to create an OpenXR instance, result is XR_ERROR_RUNTIME_FAILURE. Please check if you have an OpenXR runtime installed. The following extensions were enabled:
2021-08-04T14:02:15.431+08:00: LogHMD: - XR_KHR_D3D11_enable
2021-08-04T14:02:15.431+08:00: LogHMD: - XR_KHR_D3D12_enable
2021-08-04T14:02:15.432+08:00: LogHMD: - XR_KHR_opengl_enable
2021-08-04T14:02:15.432+08:00: LogHMD: - XR_KHR_vulkan_enable
2021-08-04T14:02:15.432+08:00: LogHMD: - XR_KHR_composition_layer_depth
2021-08-04T14:02:15.432+08:00: LogHMD: - XR_KHR_visibility_mask
2021-08-04T14:02:15.432+08:00: LogHMD: - XR_EXT_hand_tracking

Potentially just an issue with the UE 4.27 preview. Logging for tracking purposes only.

Bug reduction:

Steps to reproduce: See DSOF-17840

MunWolf commented 3 years ago

Maybe the OpenXR plugin should not be enabled by default. I know UE 4.27 added a lot of extra logging to different plugins.

MunWolf commented 2 years ago

I've mentioned this to Epic, they should fix their logging so we only get this message once, but we can't really do much on our end, OpenXR is enabled by default on the nDisplay sample and we don't want to prevent users from using it.