perf: skip temporal accumulation when disabled

.gitignore

@@ -1,3 +1,3 @@
 node_modules/
 dist/
-.DS_Store
+.DS_Store

src/N8AONode.ts

@@ -702,7 +702,9 @@ export class N8AONode extends TempNode {
     } else {
       this.frame = 0;
       this.needsFrame = false;
-      this.clearAccumulationTargets(renderer);
+      if (this.configuration.accumulate) {
+        this.clearAccumulationTargets(renderer);
+      }
     }
 
     this.lastViewMatrix.copy(this.camera.matrixWorldInverse);
@@ -757,20 +759,24 @@ export class N8AONode extends TempNode {
         writeTarget = nextWriteTarget;
       }
 
-      this.accumulationCurrentTextureNode.value = readTarget.texture;
-      this.accumulationPreviousTextureNode.value =
-        this.accumulationTargetA.texture;
-      renderer.setRenderTarget(this.accumulationTargetB);
-      this.quadMesh.material = this.accumulationMaterial;
-      this.quadMesh.name = "N8AO.Accumulation";
-      this.quadMesh.render(renderer as any);
+      if (this.configuration.accumulate) {
+        this.accumulationCurrentTextureNode.value = readTarget.texture;
+        this.accumulationPreviousTextureNode.value =
+          this.accumulationTargetA.texture;
+        renderer.setRenderTarget(this.accumulationTargetB);
+        this.quadMesh.material = this.accumulationMaterial;
+        this.quadMesh.name = "N8AO.Accumulation";
+        this.quadMesh.render(renderer as any);
 
-      const previousAccumulationTarget = this.accumulationTargetA;
-      this.accumulationTargetA = this.accumulationTargetB;
-      this.accumulationTargetB = previousAccumulationTarget;
-      this.accumulationPreviousTextureNode.value =
-        this.accumulationTargetA.texture;
-      this.compositeAoTextureNode.value = this.accumulationTargetA.texture;
+        const previousAccumulationTarget = this.accumulationTargetA;
+        this.accumulationTargetA = this.accumulationTargetB;
+        this.accumulationTargetB = previousAccumulationTarget;
+        this.accumulationPreviousTextureNode.value =
+          this.accumulationTargetA.texture;
+        this.compositeAoTextureNode.value = this.accumulationTargetA.texture;
+      } else {
+        this.compositeAoTextureNode.value = readTarget.texture;
+      }
     }
 
     renderer.setRenderTarget(this.outputTarget);

src/math.test.ts

@@ -7,6 +7,20 @@ import {
   resolveDisplayMode,
 } from "./math.js";
 
+const expectCloseTo = (
+  actual: number[][],
+  expected: number[][],
+  precision = 10,
+) => {
+  expect(actual).toHaveLength(expected.length);
+  actual.forEach((row, i) => {
+    expect(row).toHaveLength(expected[i].length);
+    row.forEach((val, j) => {
+      expect(val).toBeCloseTo(expected[i][j], precision);
+    });
+  });
+};
+
 describe("n8ao webgpu math", () => {
   it("matches upstream n8ao configuration defaults", () => {
     expect(createDefaultN8AOConfiguration()).toMatchObject({
@@ -36,8 +50,7 @@ describe("n8ao webgpu math", () => {
 
   it("generates the same hemisphere sample distribution as upstream n8ao", () => {
     const samples = generateHemisphereSamples(4);
-
-    expect(samples.map((sample) => sample.toArray())).toEqual([
+    expectCloseTo(samples.map((s) => s.toArray()), [
       [0.3535533905932738, 0, 0.9354143466934853],
       [-0.4515442808474507, 0.4136517405184686, 0.7905694150420949],
       [0.06911574220702117, -0.7875423888141976, 0.6123724356957944],
@@ -47,8 +60,7 @@ describe("n8ao webgpu math", () => {
 
   it("generates the same denoise poisson pattern as upstream n8ao", () => {
     const samples = generateDenoiseSamples(4, 11);
-
-    expect(samples.map((sample) => sample.toArray())).toEqual([
+    expectCloseTo(samples.map((s) => s.toArray()), [
       [0.35355339059327373, 0],
       [-1.4567267349998804e-15, -0.5946035575013605],
       [-0.8059274488676564, 3.948903589373758e-15],