@jdultra/threedtiles 13.3.2 → 13.3.3

package/dist/threedtiles.es.js

@@ -7340,19 +7340,21 @@ void getVertexData(out vec3 position, out mat3 covariance) {
         int( (y + 0.5) ),              // y pixel
         int( sliceIndex + 0.5 ) );     // z slice */
 
-    float index = float(order)+0.1; // add small offset to avoid floating point errors with modulo
-    float pixelsPerSlice = textureSize * textureSize;
-    float sliceIndex = floor(index / pixelsPerSlice);
-    float slicePixelIndex = mod(index,pixelsPerSlice);
+    
+    highp uint uOrder = order; // Use a local uint copy
 
-    float x = mod(slicePixelIndex,textureSize);
-    float y = floor(slicePixelIndex / textureSize);
+    // It's good practice to ensure textureSize is treated as uint for these calcs
+    uint uTextureSize = uint(textureSize); // textureSize uniform is float
+    uint uPixelsPerSlice = uTextureSize * uTextureSize;
 
-    //vec3 uvw = vec3((x + 0.5) / textureSize, (y + 0.5) / textureSize, (sliceIndex + 0.5) / numSlices);
-    ivec3 coord = ivec3(
-        int( (x + 0.5) ),              // x pixel
-        int( (y + 0.5) ),              // y pixel
-        int( sliceIndex + 0.5 ) );     // z slice
+    uint sliceIndexVal = uOrder / uPixelsPerSlice;
+    uint slicePixelIndex = uOrder % uPixelsPerSlice; // umod(uOrder, uPixelsPerSlice) also works
+
+    uint xVal = slicePixelIndex % uTextureSize; // umod(slicePixelIndex, uTextureSize)
+    uint yVal = slicePixelIndex / uTextureSize;
+
+    // texelFetch takes ivec3 for coordinates, no +0.5 needed as these are direct integer indices
+    ivec3 coord = ivec3(xVal, yVal, sliceIndexVal);
 
     // Position
     highp uvec4 positionColor = texelFetch(positionColorTexture, coord,0);