Merge pull request #1858 from Unity-Technologies/HDRP/staging

Merge Hdrp/staging

com.unity.render-pipelines.high-definition/CHANGELOG.md

 
 ## [3.3.0-preview]
 
+### Added
+- Added an error message to say to use Metal or Vulkan when trying to use OpenGL API
+- Added a new Fabric shader model that supports Silk and Cotton/Wool
+
 ### Fixed
 - Fix an issue where the screen where darken when rendering camera preview
 - Fix display correct target platform when showing message to inform user that a platform is not supported
 
 ### Changed
 - Changed default reflection probe to be 256x256x6 and array size to be 64
+- Removed dependence on the NdotL for thickness evaluation for translucency (based on artist's input)
-
-### Added
-- Added a new Fabric shader model that supports Silk and Cotton/Wool
 
 ## [3.2.0-preview]
 

com.unity.render-pipelines.high-definition/HDRP/Lighting/LightEvaluation.hlsl

 
             // Our subsurface scattering models use the semi-infinite planar slab assumption.
             // Therefore, we need to find the thickness along the normal.
-            float thicknessInUnits = (distFrontFaceToLight - distBackFaceToLight) * -NdotL;
+            // Warning: based on the artist's input, dependence on the NdotL has been disabled.
+            float thicknessInUnits = (distFrontFaceToLight - distBackFaceToLight) /* * -NdotL */;
             float thicknessInMeters = thicknessInUnits * _WorldScales[bsdfData.diffusionProfile].x;
             float thicknessInMillimeters = thicknessInMeters * MILLIMETERS_PER_METER;
 

com.unity.render-pipelines.high-definition/HDRP/RenderPipeline/HDRenderPipeline.cs

             {
                 CoreUtils.DisplayUnsupportedAPIMessage();
 
+                // Display more information to the users when it should have use Metal instead of OpenGL
+                if (SystemInfo.graphicsDeviceType.ToString().StartsWith("OpenGL"))
+                {
+                    if (SystemInfo.operatingSystem.StartsWith("Mac"))
+                        CoreUtils.DisplayUnsupportedMessage("Use Metal API instead.");
+                    else if (SystemInfo.operatingSystem.StartsWith("Windows"))
+                        CoreUtils.DisplayUnsupportedMessage("Use Vulkan API instead.");
+                }
+
                 return false;
             }