onSampleRendered returns time elapsed (#79)

Lucas Crane · web-flow · commit 73d3727ccaee · 2020-03-27T10:15:29.000-07:00
diff --git a/src/renderer/RenderingPipeline.js b/src/renderer/RenderingPipeline.js
@@ -67,6 +67,7 @@ export function makeRenderingPipeline({
 
   let frameTime;
   let elapsedFrameTime;
+  let sampleTime;
 
   let sampleCount = 0;
   let numPreviewsRendered = 0;
@@ -311,6 +312,9 @@ export function makeRenderingPipeline({
       if (sampleCount === 0) { // previous rendered image was a preview image
         clearBuffer(hdrBuffer);
         reprojectPass.setPreviousCamera(lastCamera);
+      } else {
+        sampleRenderedCallback(sampleCount, frameTime - sampleTime || NaN);
+        sampleTime = frameTime;
       }
 
       updateSeed(screenWidth, screenHeight, true);
@@ -344,8 +348,6 @@ export function makeRenderingPipeline({
       } else {
         toneMapToScreen(hdrBuffer.color[0], fullscreenScale);
       }
-
-      sampleRenderedCallback(sampleCount);
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -67,6 +67,7 @@ export function makeRenderingPipeline({`
`67`	`67`
`68`	`68`	`let frameTime;`
`69`	`69`	`let elapsedFrameTime;`
	`70`	`+ let sampleTime;`
`70`	`71`
`71`	`72`	`let sampleCount = 0;`
`72`	`73`	`let numPreviewsRendered = 0;`
`@@ -311,6 +312,9 @@ export function makeRenderingPipeline({`
`311`	`312`	`if (sampleCount === 0) { // previous rendered image was a preview image`
`312`	`313`	`clearBuffer(hdrBuffer);`
`313`	`314`	`reprojectPass.setPreviousCamera(lastCamera);`
	`315`	`+ } else {`
	`316`	`+ sampleRenderedCallback(sampleCount, frameTime - sampleTime \|\| NaN);`
	`317`	`+ sampleTime = frameTime;`
`314`	`318`	`}`
`315`	`319`
`316`	`320`	`updateSeed(screenWidth, screenHeight, true);`
`@@ -344,8 +348,6 @@ export function makeRenderingPipeline({`
`344`	`348`	`} else {`
`345`	`349`	`toneMapToScreen(hdrBuffer.color[0], fullscreenScale);`
`346`	`350`	`}`
`347`		`-`
`348`		`- sampleRenderedCallback(sampleCount);`
`349`	`351`	`}`
`350`	`352`	`}`
`351`	`353`