From ff98dedbb289e99eb2f96389c17e3ac0dfca11d5 Mon Sep 17 00:00:00 2001 From: samcake Date: Fri, 11 Aug 2017 18:10:44 -0700 Subject: [PATCH 1/5] Debugging the graphics fields activation --- interface/resources/config/render.json | 16 +++++++++++++- interface/src/Application.cpp | 2 +- interface/src/ui/PreferencesDialog.cpp | 22 ++++++++++--------- .../spectator-camera/spectatorCamera.js | 2 +- 4 files changed, 29 insertions(+), 13 deletions(-) diff --git a/interface/resources/config/render.json b/interface/resources/config/render.json index b5b72d7d07..5266ee0666 100644 --- a/interface/resources/config/render.json +++ b/interface/resources/config/render.json @@ -1,4 +1,18 @@ { + "SecondaryCameraJob": { + "RenderShadowTask": { + "Enabled": { + "enabled": true + } + }, + "RenderDeferredTask": { + "AmbientOcclusion": { + "Enabled": { + "enabled": true + } + } + } + }, "RenderMainView": { "RenderShadowTask": { "Enabled": { @@ -13,4 +27,4 @@ } } } -} +} \ No newline at end of file diff --git a/interface/src/Application.cpp b/interface/src/Application.cpp index 3d7df0d579..5299ab5b3f 100644 --- a/interface/src/Application.cpp +++ b/interface/src/Application.cpp @@ -351,7 +351,7 @@ public: // Don't actually crash in debug builds, in case this apparent deadlock is simply from // the developer actively debugging code #ifdef NDEBUG - deadlockDetectionCrash(); + // deadlockDetectionCrash(); #endif } } diff --git a/interface/src/ui/PreferencesDialog.cpp b/interface/src/ui/PreferencesDialog.cpp index c9e59ada23..35184a391d 100644 --- a/interface/src/ui/PreferencesDialog.cpp +++ b/interface/src/ui/PreferencesDialog.cpp @@ -315,21 +315,23 @@ void setupPreferences() { static const QString RENDER("Graphics"); auto renderConfig = qApp->getRenderEngine()->getConfiguration(); if (renderConfig) { - auto ambientOcclusionConfig = renderConfig->getConfig(); - if (ambientOcclusionConfig) { - auto getter = [ambientOcclusionConfig]()->QString { return ambientOcclusionConfig->getPreset(); }; - auto setter = [ambientOcclusionConfig](QString preset) { ambientOcclusionConfig->setPreset(preset); }; + auto mainViewAmbientOcclusionConfig = renderConfig->getConfig("RenderMainView.AmbientOcclusion"); + auto camViewAmbientOcclusionConfig = renderConfig->getConfig("SecondaryCameraJob.AmbientOcclusion"); + if (mainViewAmbientOcclusionConfig && camViewAmbientOcclusionConfig) { + auto getter = [mainViewAmbientOcclusionConfig]()->QString { return mainViewAmbientOcclusionConfig->getPreset(); }; + auto setter = [mainViewAmbientOcclusionConfig, camViewAmbientOcclusionConfig](QString preset) { mainViewAmbientOcclusionConfig->setPreset(preset); camViewAmbientOcclusionConfig->setPreset(preset); }; auto preference = new ComboBoxPreference(RENDER, "Ambient occlusion", getter, setter); - preference->setItems(ambientOcclusionConfig->getPresetList()); + preference->setItems(mainViewAmbientOcclusionConfig->getPresetList()); preferences->addPreference(preference); } - auto shadowConfig = renderConfig->getConfig(); - if (shadowConfig) { - auto getter = [shadowConfig]()->QString { return shadowConfig->getPreset(); }; - auto setter = [shadowConfig](QString preset) { shadowConfig->setPreset(preset); }; + auto mainViewShadowConfig = renderConfig->getConfig("RenderMainView.RenderShadowTask"); + auto camViewShadowConfig = renderConfig->getConfig("SecondaryCameraJob.RenderShadowTask"); + if (mainViewShadowConfig && camViewShadowConfig) { + auto getter = [mainViewShadowConfig]()->QString { return mainViewShadowConfig->getPreset(); }; + auto setter = [mainViewShadowConfig, camViewShadowConfig](QString preset) { mainViewShadowConfig->setPreset(preset); camViewShadowConfig->setPreset(preset); }; auto preference = new ComboBoxPreference(RENDER, "Shadows", getter, setter); - preference->setItems(shadowConfig->getPresetList()); + preference->setItems(mainViewShadowConfig->getPresetList()); preferences->addPreference(preference); } } diff --git a/unpublishedScripts/marketplace/spectator-camera/spectatorCamera.js b/unpublishedScripts/marketplace/spectator-camera/spectatorCamera.js index f0b943ad92..fd08cd5655 100644 --- a/unpublishedScripts/marketplace/spectator-camera/spectatorCamera.js +++ b/unpublishedScripts/marketplace/spectator-camera/spectatorCamera.js @@ -153,7 +153,7 @@ // -showSpectatorInDesktop: Set to "true" to show the "SPECTATOR" app in desktop mode. var button = false; var buttonName = "SPECTATOR"; - var showSpectatorInDesktop = false; + var showSpectatorInDesktop = true; function addOrRemoveButton(isShuttingDown, isHMDMode) { if (!tablet) { print("Warning in addOrRemoveButton(): 'tablet' undefined!"); From 05b78b0df06b6962a2e32c09083923f74beee323 Mon Sep 17 00:00:00 2001 From: samcake Date: Mon, 14 Aug 2017 10:46:33 -0700 Subject: [PATCH 2/5] Just fixing the ao combo box field --- interface/resources/config/render.json | 14 -------------- interface/src/ui/PreferencesDialog.cpp | 10 ++++------ 2 files changed, 4 insertions(+), 20 deletions(-) diff --git a/interface/resources/config/render.json b/interface/resources/config/render.json index 5266ee0666..83477fef69 100644 --- a/interface/resources/config/render.json +++ b/interface/resources/config/render.json @@ -1,18 +1,4 @@ { - "SecondaryCameraJob": { - "RenderShadowTask": { - "Enabled": { - "enabled": true - } - }, - "RenderDeferredTask": { - "AmbientOcclusion": { - "Enabled": { - "enabled": true - } - } - } - }, "RenderMainView": { "RenderShadowTask": { "Enabled": { diff --git a/interface/src/ui/PreferencesDialog.cpp b/interface/src/ui/PreferencesDialog.cpp index 35184a391d..88b26a1b8c 100644 --- a/interface/src/ui/PreferencesDialog.cpp +++ b/interface/src/ui/PreferencesDialog.cpp @@ -316,20 +316,18 @@ void setupPreferences() { auto renderConfig = qApp->getRenderEngine()->getConfiguration(); if (renderConfig) { auto mainViewAmbientOcclusionConfig = renderConfig->getConfig("RenderMainView.AmbientOcclusion"); - auto camViewAmbientOcclusionConfig = renderConfig->getConfig("SecondaryCameraJob.AmbientOcclusion"); - if (mainViewAmbientOcclusionConfig && camViewAmbientOcclusionConfig) { + if (mainViewAmbientOcclusionConfig) { auto getter = [mainViewAmbientOcclusionConfig]()->QString { return mainViewAmbientOcclusionConfig->getPreset(); }; - auto setter = [mainViewAmbientOcclusionConfig, camViewAmbientOcclusionConfig](QString preset) { mainViewAmbientOcclusionConfig->setPreset(preset); camViewAmbientOcclusionConfig->setPreset(preset); }; + auto setter = [mainViewAmbientOcclusionConfig](QString preset) { mainViewAmbientOcclusionConfig->setPreset(preset); }; auto preference = new ComboBoxPreference(RENDER, "Ambient occlusion", getter, setter); preference->setItems(mainViewAmbientOcclusionConfig->getPresetList()); preferences->addPreference(preference); } auto mainViewShadowConfig = renderConfig->getConfig("RenderMainView.RenderShadowTask"); - auto camViewShadowConfig = renderConfig->getConfig("SecondaryCameraJob.RenderShadowTask"); - if (mainViewShadowConfig && camViewShadowConfig) { + if (mainViewShadowConfig) { auto getter = [mainViewShadowConfig]()->QString { return mainViewShadowConfig->getPreset(); }; - auto setter = [mainViewShadowConfig, camViewShadowConfig](QString preset) { mainViewShadowConfig->setPreset(preset); camViewShadowConfig->setPreset(preset); }; + auto setter = [mainViewShadowConfig](QString preset) { mainViewShadowConfig->setPreset(preset); }; auto preference = new ComboBoxPreference(RENDER, "Shadows", getter, setter); preference->setItems(mainViewShadowConfig->getPresetList()); preferences->addPreference(preference); From db1536c521a138799457b01bda1fac23bfff6065 Mon Sep 17 00:00:00 2001 From: samcake Date: Mon, 14 Aug 2017 10:47:58 -0700 Subject: [PATCH 3/5] Reactivate watch dog exit --- interface/src/Application.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/interface/src/Application.cpp b/interface/src/Application.cpp index 5299ab5b3f..3d7df0d579 100644 --- a/interface/src/Application.cpp +++ b/interface/src/Application.cpp @@ -351,7 +351,7 @@ public: // Don't actually crash in debug builds, in case this apparent deadlock is simply from // the developer actively debugging code #ifdef NDEBUG - // deadlockDetectionCrash(); + deadlockDetectionCrash(); #endif } } From 465e390f7b8eed76e4f3f78df3d8bc486133c4d5 Mon Sep 17 00:00:00 2001 From: samcake Date: Mon, 14 Aug 2017 10:49:08 -0700 Subject: [PATCH 4/5] Redisable spectator cam from desktop --- .../marketplace/spectator-camera/spectatorCamera.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/unpublishedScripts/marketplace/spectator-camera/spectatorCamera.js b/unpublishedScripts/marketplace/spectator-camera/spectatorCamera.js index fd08cd5655..f0b943ad92 100644 --- a/unpublishedScripts/marketplace/spectator-camera/spectatorCamera.js +++ b/unpublishedScripts/marketplace/spectator-camera/spectatorCamera.js @@ -153,7 +153,7 @@ // -showSpectatorInDesktop: Set to "true" to show the "SPECTATOR" app in desktop mode. var button = false; var buttonName = "SPECTATOR"; - var showSpectatorInDesktop = true; + var showSpectatorInDesktop = false; function addOrRemoveButton(isShuttingDown, isHMDMode) { if (!tablet) { print("Warning in addOrRemoveButton(): 'tablet' undefined!"); From f86f22309b6eede3425303c1d0a1b09a78337994 Mon Sep 17 00:00:00 2001 From: samcake Date: Mon, 14 Aug 2017 11:31:40 -0700 Subject: [PATCH 5/5] fix render.json --- interface/resources/config/render.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/interface/resources/config/render.json b/interface/resources/config/render.json index 83477fef69..b5b72d7d07 100644 --- a/interface/resources/config/render.json +++ b/interface/resources/config/render.json @@ -13,4 +13,4 @@ } } } -} \ No newline at end of file +}