Increase memory limit to handle high resolution displays

Scthe · Sep 7, 2024 · f4b32df · f4b32df
1 parent 045b393
commit f4b32df
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 5 deletions.
diff --git a/src/constants.ts b/src/constants.ts
@@ -93,7 +93,7 @@ export const CONFIG = {
     binaryFileReader: binaryFileReader_Web,
     createTextureFromFile: createTextureFromFile_Web,
   },
-  increaseStorageMemoryLimits: false,
+  increaseStorageMemoryLimits: true,
   displayMode: DISPLAY_MODE.FINAL,
 
   ///////////////
@@ -240,7 +240,7 @@ export const CONFIG = {
      * ANYTHING WRONG IF IT WAS A SIMPLE COLLISION (INCREASE SDF OFFSET,
      * OR HIDE THE BALL IN THAT CASE).
      */
-    invalidTilesPerSegmentThreshold: 64,
+    invalidTilesPerSegmentThreshold: 64 * 2,
 
     ////// SORT PASS
     sortBuckets: 64,
@@ -282,7 +282,7 @@ export const CONFIG = {
     // https://youtu.be/ool2E8SQPGU?si=yKgmYF6Wjbu6HXsF&t=815
     friction: 0.3,
     // 0.0 - do not use density gradient as external force. Hair can "squish" together
-    // >0.0 - move hair strands so from densely oocupied grid cells into ones that are more "free"
+    // >0.0 - move hair strands from densely occupied grid cells into ones that are more "free"
     volumePreservation: 0.00003,
     // If you set the radius to 0, the ball will disappear. There is a reason for this statement.
     // collisionSphere: [0.05, 1.48, 0.01, 0.06], // inside hair

diff --git a/src/utils/webgpu.ts b/src/utils/webgpu.ts
@@ -28,8 +28,19 @@ export async function createGpuDevice() {
     // Limits change: https://gpuweb.github.io/gpuweb/#gpusupportedlimits
     const requiredLimits: GPUSupportedLimits = {};
     if (CONFIG.increaseStorageMemoryLimits) {
-      requiredLimits.maxStorageBufferBindingSize = getBytes(1024, 'MB');
-      requiredLimits.maxBufferSize = getBytes(1024, 'MB');
+      // Warning: BigInt is used for memory limits (unsigned long long in WebGPU spec)
+      const lims = adapter.limits;
+
+      if (Number.isSafeInteger(lims.maxStorageBufferBindingSize)) {
+        requiredLimits.maxStorageBufferBindingSize = Number(lims.maxStorageBufferBindingSize); // prettier-ignore
+      }
+      if (Number.isSafeInteger(lims.maxBufferSize)) {
+        requiredLimits.maxBufferSize = Number(lims.maxBufferSize);
+      }
+
+      // defaults just in case
+      requiredLimits.maxStorageBufferBindingSize ||= getBytes(1024, 'MB');
+      requiredLimits.maxBufferSize ||= getBytes(1024, 'MB');
     }
     requiredLimits.maxComputeWorkgroupStorageSize = Math.max(
       getLocalMemoryRequirements(),