xBimTeam / XbimGeometry

XbimGeometry contains the CLR interop libraries and the c++ engine used to compute the 3D geometry of models.
https://xbimteam.github.io/
Other
260 stars 132 forks source link

Creating Geometry with attached file causes geometry engine to get stuck in an infinite loop #270

Closed zohelfer closed 4 years ago

zohelfer commented 4 years ago

Opening the attached file in the most recent version of XbimWindowsUI's Xplorer will hang the program on the "Creating Geometry" step. Looking at the created threads, it appears that the geometry engine is stuck on an initialize geometry step. The engine will keep attempting to spawn threads until the user's processor is maxed out. My best guess is there is a circular dependency that is causing the engine's code to get stuck in a infinite loop.

Assemblies and versions affected:

XbimGeometry v5+

Steps (or code) to reproduce the issue:

Open attached file in newest Xplorer release.

Minimal file to reproduce the issue:

CADSoft Example House.zip

Expected behavior:

File opens and 3D geometry is created

Actual behavior or exception details:

File gets stuck on the Creating Geometry Step

Additional Details

This file works in XBimGeometry v4 and Xplorer application that can be downloaded from the xBim website

bekraft commented 4 years ago

What version is your "most recent" Xplorer? Doing the same from the dev branches for XbimGeometry and XbimWindowsUI works fine for me. There's a dialog popup in "About". The tab "Assemblies" will show the assembly and file versions in use.

zohelfer commented 4 years ago

image

I pulled this on July 23rd from master. Is there a newer version on Develop?

Thank you so much, Zacc

bekraft commented 4 years ago

There are many changes done since last merge. I don't know which changes effect the model (and whether it's really a problem of geometry engine). Maybe @SteveLockley knows more details. My recent develop branch build uses the head of 83158514c0edbe650c2c9d166c194481c2fbee80. And it works. See screenshot.

Issue270-Screenshot-develop

zohelfer commented 4 years ago

Thank you so much for the help. Upgrading to the newest Develop fixed our issue as well.