Examples: More inspector usage.

[sunag]

Description

More inspector usage and RenderInspector revisions.

[github-actions]

📦 Bundle size

Full ESM build, minified and gzipped.

	Before	After	Diff
WebGL	338.24 79.14	338.24 79.14	+0 B +0 B
WebGPU	589.39 162.79	589.46 162.82	+70 B +26 B
WebGPU Nodes	587.99 162.55	588.06 162.58	+70 B +27 B

🌳 Bundle size after tree-shaking

Minimal build including a renderer, camera, empty scene, and dependencies.

	Before	After	Diff
WebGL	469.7 113.74	469.7 113.74	+0 B +0 B
WebGPU	658.79 178.24	658.79 178.24	+0 B +0 B
WebGPU Nodes	612.8 167.43	612.8 167.43	+0 B +0 B

[Makio64]

@sunag I was checking how to use the inspector, i noticed this pattern :

				renderer.inspector = new Inspector();
				//...
				const gui = renderer.inspector.createParameters( 'Settings' );

I think it would be cleaner if inspector variable is created at the global level with scene, camera etc..

let scene, camera, inspector // at top of the files.

				inspector = new Inspector();
				renderer.inspector = inspector; // not sure its necessary ?
				// .... 
				const gui = inspector.createParameters( 'Settings' );

[Mugen87]

inspector = new Inspector();
renderer.inspector = new Inspector();

TBH, it looks confusing to have two constructor calls.

As long as the inspector isn't referenced outside init(), I think the current pattern is appropriate.

[Makio64]

sorry for the typo @Mugen87 i updated the code, of course it shoudn't be 2 constructors.

I just feel this renderer.inspector.createParameters( 'Settings' ); should be improve, especially if all examples are migrate to it and there is multiple createParameters.

[sunag]

I like renderer.inspector, unlike scene, camera they are not assigned to the renderer as inspector is but passed as parameters.

The createParameters() function is not mandatory, some examples just want to inspect the performance and nodes and not use it, so what you often need to do is just assign renderer.inspector = new Inspector() with one line.

Nothing against separating this into a variable, but it doesn't seem that necessary.

[Makio64]

Thanks @sunag , i will play with it and come back ! 👍