Changed calculation for range clipping (#325)

* changed calculation for depht

Signed-off-by: Tomas Lorente <[email protected]>

* Ian comments

Signed-off-by: Tomas Lorente <[email protected]>

* test fix

Signed-off-by: Tomas Lorente <[email protected]>

* fix test

Signed-off-by: Tomas Lorente <[email protected]>

* updated tests and migration

Signed-off-by: Tomas Lorente <[email protected]>

* tweak to migration guide

Signed-off-by: Ian Chen <[email protected]>

Co-authored-by: Ian Chen <[email protected]>

Author: Lobotuerk

Migration.md

@@ -37,6 +37,9 @@ release will remove the deprecated code.
     `Node::UserData` now returns no data for keys that don't exist (prior to Rendering 6.x, if
     `Visual::UserData` was called with a key that doesn't exist, an `int` was returned by default).
 
+1. **depth_camera_fs.glsl** and **depth_camera_final_fs.glsl**
+    + Far clipping changed from clipping by depth to clipping by range, i.e. distance to point, so that the data generated will never exceed the specified max range of the camera.
+
 ## Ignition Rendering 4.0 to 4.1
 
 ## ABI break

ogre2/src/media/materials/programs/depth_camera_final_fs.glsl

@@ -50,7 +50,7 @@ void main()
   // to be outside of min/max range
 
   // clamp xyz
-  if (point.x > far - tolerance)
+  if (!isinf(point.x) && length(point) > far - tolerance)
   {
     if (isinf(max))
     {

ogre2/src/media/materials/programs/depth_camera_fs.glsl

@@ -139,7 +139,7 @@ void main()
   }
 
   // clamp xyz and set rgb to background color
-  if (point.x > far - tolerance)
+  if (length(point) > far - tolerance)
   {
     if (isinf(max))
     {