From a7e1807d2b83f6e17906ddc19328b7f5ca9b7b8d Mon Sep 17 00:00:00 2001 From: Steven Borkman Date: Wed, 5 Aug 2020 11:02:20 -0400 Subject: [PATCH 1/3] Fix for the blurred visualization bug --- .../Assets/Scripts/RandomizedPostProcessingFeature.cs | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/SynthDet/Assets/Scripts/RandomizedPostProcessingFeature.cs b/SynthDet/Assets/Scripts/RandomizedPostProcessingFeature.cs index c04f2070..8bd3d297 100644 --- a/SynthDet/Assets/Scripts/RandomizedPostProcessingFeature.cs +++ b/SynthDet/Assets/Scripts/RandomizedPostProcessingFeature.cs @@ -56,6 +56,14 @@ public class RandomizedPostProcessingFeature : ScriptableRendererFeature return; #endif + // When SynthDet is being ran in visualization mode, a second camera is created to display the scene. The first + // camera (which contains the perception camera) renders the scene to a render texture, and a second camera + // displays the render texture along with all visualization components. We do not one the custom passes applied + // to the visualized contents, only the contents that are used by the perception camera. In a scene without + // visualization only one camera (the perception camera) is available and so that case will bypass this check + if (renderingData.cameraData.camera.GetComponent() == null) + return; + // Initialize here because scene is not guaranteed to be initialized when Create is called if (m_InitParams == null) { From 046668b3e4dfb42e746234a4be0e15a763a776ce Mon Sep 17 00:00:00 2001 From: Steven Borkman Date: Thu, 6 Aug 2020 10:10:31 -0400 Subject: [PATCH 2/3] Linux test suite fix --- .yamato/ci-test.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.yamato/ci-test.yml b/.yamato/ci-test.yml index 1aa30c5b..933a124d 100644 --- a/.yamato/ci-test.yml +++ b/.yamato/ci-test.yml @@ -68,7 +68,7 @@ linux_{{suite.name}}_{{editor.version}}: commands: - git submodule update --init --recursive - sudo -H pip install --upgrade pip - - sudo -H pip install unity-downloader-cli --extra-index-url https://artifactory.internal.unity3d.com/api/pypi/common-python/simple + - sudo -H pip install unity-downloader-cli --index-url https://artifactory.prd.it.unity3d.com/artifactory/api/pypi/pypi/simple --upgrade --user - git clone git@github.cds.internal.unity3d.com:unity/utr.git utr - sudo unity-downloader-cli -u {{ editor.version }} -c editor -c StandaloneSupport-IL2CPP -c Linux --wait --published {% if suite.name == "standalone" %} @@ -81,4 +81,4 @@ linux_{{suite.name}}_{{editor.version}}: paths: - "test-results/**/*" {% endfor %} -{% endfor %} \ No newline at end of file +{% endfor %} From 5d7558b5627dd770d65185e3e0970dc380982e9c Mon Sep 17 00:00:00 2001 From: Steven Borkman Date: Thu, 6 Aug 2020 10:34:41 -0400 Subject: [PATCH 3/3] Fix for linux test suite --- .yamato/ci-test.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.yamato/ci-test.yml b/.yamato/ci-test.yml index 933a124d..6017b709 100644 --- a/.yamato/ci-test.yml +++ b/.yamato/ci-test.yml @@ -68,7 +68,7 @@ linux_{{suite.name}}_{{editor.version}}: commands: - git submodule update --init --recursive - sudo -H pip install --upgrade pip - - sudo -H pip install unity-downloader-cli --index-url https://artifactory.prd.it.unity3d.com/artifactory/api/pypi/pypi/simple --upgrade --user + - sudo -H pip install unity-downloader-cli --index-url https://artifactory.prd.it.unity3d.com/artifactory/api/pypi/pypi/simple --upgrade - git clone git@github.cds.internal.unity3d.com:unity/utr.git utr - sudo unity-downloader-cli -u {{ editor.version }} -c editor -c StandaloneSupport-IL2CPP -c Linux --wait --published {% if suite.name == "standalone" %}