Viewer didn't load the model

[youshengCode]

Description:
After select hello-wall.ifcx file in the viewer it's didn't load it into scene, tree view and scene are both empty.

File:
Using a fresh cloned repo with latest example file, tested with hello-wall.ifcx

Browser:
Tested with Edge, Chrome and Safari, normal mode and private mode, all behaved the same.

Screenshot:

Console error:

render.mjs:513  Uncaught (in promise) TypeError: activeLayer.imports is not iterable
    at IfcxLayerStackBuilder.SatisfyDependencies (render.mjs:513:36)
    at IfcxLayerStackBuilder.SatisfyDependencies (render.mjs:526:31)
    at async IfcxLayerStackBuilder.BuildLayerSet (render.mjs:543:18)
    at async IfcxLayerStackBuilder.Build (render.mjs:501:18)
    at async compose3 (render.mjs:599:20)
    at async composeAndRender (render.mjs:895:10)
    at async addModel (render.mjs:953:3)

[youshengCode]

Further investigation: local environment functions properly, only the online version is encountering this issue.

[aothms]

Seems you model does not have an imports node. You can apply the following patch (compile ts afterwards npm run build-viewer) to make it optional. Not sure if we want that, I think it makes sense to have imports, data, schemas all optional (so that you would have e.g bsi schema ifcx files that don't have data). What does the rest think?

diff --git a/src/ifcx-core/layers/layer-stack.ts b/src/ifcx-core/layers/layer-stack.ts
index 7a2b78c..b43d17a 100644
--- a/src/ifcx-core/layers/layer-stack.ts
+++ b/src/ifcx-core/layers/layer-stack.ts
@@ -88,7 +88,7 @@ export class IfcxLayerStackBuilder
     private async SatisfyDependencies(activeLayer: IfcxFile, placed: Map<string, boolean>, orderedLayers: IfcxFile[])
     {
         let pending: IfcxFile[] = [];
-        for (const impt of activeLayer.imports) {
+        for (const impt of activeLayer.imports || []) {
             if (!placed.has(impt.uri))
             {
                 let layer = await this.provider.GetLayerByURI(impt.uri);

[atomczak]

Decision to make:

Should imports, data and schemas be all optional in the schema?