@babylonjs/core 8.8.0 → 8.8.2

package/Engines/webgpuEngine.js

@@ -157,7 +157,9 @@ export class WebGPUEngine extends ThinWebGPUEngine {
             ? Promise.resolve(false)
             : navigator.gpu
                 .requestAdapter()
+                // eslint-disable-next-line github/no-then
                 .then((adapter) => !!adapter, () => false)
+                // eslint-disable-next-line github/no-then
                 .catch(() => false);
     }
     /**
@@ -234,9 +236,11 @@ export class WebGPUEngine extends ThinWebGPUEngine {
      * @param options Defines the options passed to the engine to create the GPU context dependencies
      * @returns a promise that resolves with the created engine
      */
+    // eslint-disable-next-line @typescript-eslint/promise-function-async, no-restricted-syntax
     static CreateAsync(canvas, options = {}) {
         const engine = new WebGPUEngine(canvas, options);
         return new Promise((resolve) => {
+            // eslint-disable-next-line @typescript-eslint/no-floating-promises, github/no-then
             engine.initAsync(options.glslangOptions, options.twgslOptions).then(() => resolve(engine));
         });
     }
@@ -366,12 +370,15 @@ export class WebGPUEngine extends ThinWebGPUEngine {
      * Load the glslang and tintWASM libraries and prepare them for use.
      * @returns a promise that resolves when the engine is ready to use the glslang and tintWASM
      */
+    // eslint-disable-next-line @typescript-eslint/promise-function-async, no-restricted-syntax
     prepareGlslangAndTintAsync() {
         if (!this._workingGlslangAndTintPromise) {
             this._workingGlslangAndTintPromise = new Promise((resolve) => {
+                // eslint-disable-next-line @typescript-eslint/no-floating-promises, github/no-then
                 this._initGlslangAsync(this._glslangOptions ?? this._options?.glslangOptions).then((glslang) => {
                     this._glslang = glslang;
                     this._tintWASM = new WebGPUTintWASM();
+                    // eslint-disable-next-line @typescript-eslint/no-floating-promises, github/no-then
                     this._tintWASM.initTwgsl(this._twgslOptions ?? this._options?.twgslOptions).then(() => {
                         this._glslangAndTintAreFullyLoaded = true;
                         resolve();
@@ -387,13 +394,15 @@ export class WebGPUEngine extends ThinWebGPUEngine {
      * @param twgslOptions Defines the Twgsl compiler options if necessary
      * @returns a promise notifying the readiness of the engine.
      */
+    // eslint-disable-next-line @typescript-eslint/promise-function-async, no-restricted-syntax
     initAsync(glslangOptions, twgslOptions) {
         this.uniqueId = WebGPUEngine._InstanceId++;
         this._glslangOptions = glslangOptions;
         this._twgslOptions = twgslOptions;
-        return navigator
+        return (navigator
             .gpu.requestAdapter(this._options)
-            .then((adapter) => {
+            // eslint-disable-next-line github/no-then
+            .then(async (adapter) => {
             if (!adapter) {
                 // eslint-disable-next-line no-throw-literal
                 throw "Could not retrieve a WebGPU adapter (adapter is null).";
@@ -429,9 +438,10 @@ export class WebGPUEngine extends ThinWebGPUEngine {
                     }
                 }
                 deviceDescriptor.label = `BabylonWebGPUDevice${this.uniqueId}`;
-                return this._adapter.requestDevice(deviceDescriptor);
+                return await this._adapter.requestDevice(deviceDescriptor);
             }
         })
+            // eslint-disable-next-line github/no-then
             .then((device) => {
             this._device = device;
             this._deviceEnabledExtensions = [];
@@ -449,6 +459,7 @@ export class WebGPUEngine extends ThinWebGPUEngine {
                 }
             });
             if (!this._doNotHandleContextLost) {
+                // eslint-disable-next-line @typescript-eslint/no-floating-promises, github/no-then
                 this._device.lost?.then((info) => {
                     if (this._isDisposed) {
                         return;
@@ -456,6 +467,7 @@ export class WebGPUEngine extends ThinWebGPUEngine {
                     this._contextWasLost = true;
                     Logger.Warn("WebGPU context lost. " + info);
                     this.onContextLostObservable.notifyObservers(this);
+                    // eslint-disable-next-line @typescript-eslint/no-misused-promises
                     this._restoreEngineAfterContextLost(async () => {
                         const snapshotRenderingMode = this.snapshotRenderingMode;
                         const snapshotRendering = this.snapshotRendering;
@@ -475,6 +487,7 @@ export class WebGPUEngine extends ThinWebGPUEngine {
                 });
             }
         })
+            // eslint-disable-next-line github/no-then
             .then(() => {
             this._initializeLimits();
             this._bufferManager = new WebGPUBufferManager(this, this._device);
@@ -518,11 +531,13 @@ export class WebGPUEngine extends ThinWebGPUEngine {
             this._initializeMainAttachments();
             this.resize();
         })
+            // eslint-disable-next-line github/no-then
             .catch((e) => {
             Logger.Error("A fatal error occurred during WebGPU creation/initialization.");
             throw e;
-        });
+        }));
     }
+    // eslint-disable-next-line @typescript-eslint/promise-function-async, no-restricted-syntax
     _initGlslangAsync(glslangOptions) {
         glslangOptions = glslangOptions || {};
         glslangOptions = {
@@ -536,6 +551,7 @@ export class WebGPUEngine extends ThinWebGPUEngine {
             return self.glslang(glslangOptions.wasmPath);
         }
         if (glslangOptions.jsPath && glslangOptions.wasmPath) {
+            // eslint-disable-next-line github/no-then
             return Tools.LoadBabylonScriptAsync(glslangOptions.jsPath).then(() => {
                 return self.glslang(Tools.GetBabylonScriptURL(glslangOptions.wasmPath));
             });
@@ -772,8 +788,8 @@ export class WebGPUEngine extends ThinWebGPUEngine {
      * @returns ImageBitmap
      */
     // eslint-disable-next-line @typescript-eslint/naming-convention
-    _createImageBitmapFromSource(imageSource, options) {
-        return CreateImageBitmapFromSource(this, imageSource, options);
+    async _createImageBitmapFromSource(imageSource, options) {
+        return await CreateImageBitmapFromSource(this, imageSource, options);
     }
     /**
      * Toggle full screen mode
@@ -1600,6 +1616,7 @@ export class WebGPUEngine extends ThinWebGPUEngine {
     /**
      * @internal
      */
+    // eslint-disable-next-line @typescript-eslint/no-misused-promises
     async _preparePipelineContextAsync(pipelineContext, vertexSourceCode, fragmentSourceCode, createAsRaw, rawVertexSourceCode, rawFragmentSourceCode, _rebuildRebind, defines, _transformFeedbackVaryings, _key, onReady) {
         const webGpuContext = pipelineContext;
         const shaderLanguage = webGpuContext.shaderProcessingContext.shaderLanguage;
@@ -2221,7 +2238,7 @@ export class WebGPUEngine extends ThinWebGPUEngine {
      * @param data defines the data to fill with the read pixels (if not provided, a new one will be created)
      * @returns a ArrayBufferView promise (Uint8Array) containing RGBA colors
      */
-    // eslint-disable-next-line @typescript-eslint/naming-convention
+    // eslint-disable-next-line @typescript-eslint/naming-convention, @typescript-eslint/promise-function-async
     readPixels(x, y, width, height, _hasAlpha = true, flushRenderer = true, data = null) {
         const renderPassWrapper = this._getCurrentRenderPassWrapper();
         const hardwareTexture = renderPassWrapper.colorAttachmentGPUTextures[0];
@@ -3064,9 +3081,10 @@ export class WebGPUEngine extends ThinWebGPUEngine {
         }
         this._bufferManager.setSubData(dataBuffer, byteOffset, view, 0, byteLength);
     }
-    _readFromGPUBuffer(gpuBuffer, size, buffer, noDelay) {
-        return new Promise((resolve, reject) => {
+    async _readFromGPUBuffer(gpuBuffer, size, buffer, noDelay) {
+        return await new Promise((resolve, reject) => {
             const readFromBuffer = () => {
+                // eslint-disable-next-line github/no-then
                 gpuBuffer.mapAsync(1 /* WebGPUConstants.MapMode.Read */, 0, size).then(() => {
                     const copyArrayBuffer = gpuBuffer.getMappedRange(0, size);
                     let data = buffer;
@@ -3087,6 +3105,7 @@ export class WebGPUEngine extends ThinWebGPUEngine {
                         resolve(new Uint8Array());
                     }
                     else {
+                        // eslint-disable-next-line @typescript-eslint/prefer-promise-reject-errors
                         reject(reason);
                     }
                 });
@@ -3113,7 +3132,7 @@ export class WebGPUEngine extends ThinWebGPUEngine {
      * @param noDelay If true, a call to flushFramebuffer will be issued so that the data can be read back immediately and not in engine.onEndFrameObservable. This can speed up data retrieval, at the cost of a small perf penalty (default: false).
      * @returns If not undefined, returns the (promise) buffer (as provided by the 4th parameter) filled with the data, else it returns a (promise) Uint8Array with the data read from the storage buffer
      */
-    // eslint-disable-next-line @typescript-eslint/naming-convention
+    // eslint-disable-next-line @typescript-eslint/naming-convention, @typescript-eslint/promise-function-async
     readFromStorageBuffer(storageBuffer, offset, size, buffer, noDelay) {
         size = size || storageBuffer.capacity;
         const gpuBuffer = this._bufferManager.createRawBuffer(size, WebGPUConstants.BufferUsage.MapRead | WebGPUConstants.BufferUsage.CopyDst, undefined, "TempReadFromStorageBuffer");
@@ -3129,7 +3148,7 @@ export class WebGPUEngine extends ThinWebGPUEngine {
      * @param noDelay If true, a call to flushFramebuffer will be issued so that the data can be read back immediately and not in engine.onEndFrameObservable. This can speed up data retrieval, at the cost of a small perf penalty (default: false).
      * @returns If not undefined, returns the (promise) buffer (as provided by the 4th parameter) filled with the data, else it returns a (promise) Uint8Array with the data read from the storage buffer
      */
-    // eslint-disable-next-line @typescript-eslint/naming-convention
+    // eslint-disable-next-line @typescript-eslint/naming-convention, @typescript-eslint/promise-function-async
     readFromMultipleStorageBuffers(storageBuffers, offset, size, buffer, noDelay) {
         size = size || storageBuffers[0].capacity;
         const gpuBuffer = this._bufferManager.createRawBuffer(size * storageBuffers.length, WebGPUConstants.BufferUsage.MapRead | WebGPUConstants.BufferUsage.CopyDst, undefined, "TempReadFromMultipleStorageBuffers");