rayzee 8.4.0 → 8.5.0

package/src/managers/DenoisingManager.js

@@ -149,6 +149,7 @@ export class DenoisingManager extends EventDispatcher {
 		this._holdWhileMoving = null;
 		this._onReset = null;
 		this._onPostProcessRefresh = null;
+		this._onDisplayRefresh = null;
 
 		// Resolution tracking — used for canvas restoration on reset
 		this._lastRenderWidth = 0;
@@ -835,14 +836,10 @@ export class DenoisingManager extends EventDispatcher {
 		if ( this.upscalerCanvas ) this.upscalerCanvas.style.display = 'none';
 		if ( this.mainCanvas ) this.mainCanvas.style.opacity = '1';
 
-		if ( wasShowing ) {
+		// The nominal size, not the path tracer's: that one is smaller while the camera moves.
+		if ( wasShowing && this._lastRenderWidth && this._lastRenderHeight ) {
 
-			const stage = this._stages?.pathTracer;
-			if ( stage?.width && stage?.height ) {
-
-				this.dispatchEvent( { type: 'resolution_changed', width: stage.width, height: stage.height } );
-
-			}
+			this.dispatchEvent( { type: 'resolution_changed', width: this._lastRenderWidth, height: this._lastRenderHeight } );
 
 		}
 
@@ -1057,6 +1054,10 @@ export class DenoisingManager extends EventDispatcher {
 
 			if ( ! isStillComplete() ) return;
 
+			// The loop stops one tick after a render completes, so nothing is left to draw the closing
+			// denoise. Ordered after `_denoiserEndHandler`, which publishes the picture.
+			if ( closing ) this._onDisplayRefresh?.();
+
 			// One owner of the overlay canvas, so the neural chain and the ONNX upscaler are
 			// exclusive. The neural chain also runs on its own when only its detail pass is on.
 			if ( this.neuralRendering || ( this.upscaler?.enabled && this.upscalerBackend === 'dlss' ) ) {
@@ -1220,6 +1221,10 @@ export class DenoisingManager extends EventDispatcher {
 		this._upscalerProgressHandler = null;
 		this._upscalerEndHandler = null;
 
+		this._onReset = null;
+		this._onPostProcessRefresh = null;
+		this._onDisplayRefresh = null;
+
 		if ( this.upscalerCanvas?.parentNode ) {
 
 			this.upscalerCanvas.parentNode.removeChild( this.upscalerCanvas );
@@ -1252,6 +1257,13 @@ export class DenoisingManager extends EventDispatcher {
 
 	}
 
+	/** @param {Function} fn - () => void, redraws the finished frame without re-running the completion chain */
+	setDisplayRefreshCallback( fn ) {
+
+		this._onDisplayRefresh = fn;
+
+	}
+
 	/** @param {import('../RenderSettings.js').RenderSettings} settings */
 	setSettings( settings ) {
 

package/src/managers/VideoRenderManager.js

@@ -7,7 +7,7 @@
  */
 
 import { EngineEvents } from '../EngineEvents.js';
-import { PRODUCTION_RENDER_CONFIG } from '../EngineDefaults.js';
+import { PRODUCTION_RENDER_CONFIG, modePresetSettings } from '../EngineDefaults.js';
 import { updateStats, getDisplaySamples } from '../Processor/utils.js';
 
 export class VideoRenderManager {
@@ -273,11 +273,11 @@ export class VideoRenderManager {
 	_saveState() {
 
 		const app = this._app;
+		const effective = app.settings.getEffective();
 
 		return {
-			maxSamples: app.settings.get( 'maxSamples' ),
-			maxBounces: app.settings.get( 'maxBounces' ),
-			transmissiveBounces: app.settings.get( 'transmissiveBounces' ),
+			// Everything configureForMode( 'production' ) overwrites, with who set it.
+			settings: Object.keys( modePresetSettings( PRODUCTION_RENDER_CONFIG ) ).map( key => [ key, effective[ key ] ] ),
 			renderMode: app.stages.pathTracer?.renderMode?.value,
 			controlsEnabled: app.cameraManager.controls?.enabled,
 			oidnEnabled: app.denoisingManager?.finalDenoise,
@@ -296,11 +296,11 @@ export class VideoRenderManager {
 
 		const app = this._app;
 
-		app.settings.setMany( {
-			maxSamples: state.maxSamples,
-			maxBounces: state.maxBounces,
-			transmissiveBounces: state.transmissiveBounces,
-		}, { silent: true } );
+		for ( const [ key, saved ] of state.settings ) {
+
+			if ( saved ) app.settings.set( key, saved.value, { silent: true, reset: false, source: saved.source } );
+
+		}
 
 		if ( app.stages.pathTracer && state.renderMode !== undefined ) {