Append aggregates with their parts, sub-aggregates and openings (#5909)

bim.append_library_element_by_query only ever brought in the top-level
element of an assembly. Two layers needed fixing:

ifcopenshell.api.project.append_asset never whitelisted IsDecomposedBy
for IfcElement, so IfcRelAggregates was not walked, and even then the
is_another_asset guard filtered aggregated children because they share
the IfcProduct target class with the top asset. IfcRelAggregates is now
treated as a dependent relationship, the same way IfcRelVoidsElement and
IfcRelProjectsElement already are (precedent: 55e97fd170). Scoped to
IfcElement so spatial decomposition is untouched.

The Bonsai AppendLibraryElement operator then only created a viewport
object for the single top element, leaving appended parts invisible in
the 3D view. It now creates objects, materials, styles and types for
every part and sub-assembly via get_decomposition, filtering openings
the same way normal import does.

This change was written with AI assistance.

.github/workflows/build_osx.yml

@@ -21,12 +21,12 @@ jobs:
 
     steps:
       - name: Checkout Repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           submodules: recursive
 
       - name: Checkout Build Repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           repository: IfcOpenShell/build-outputs
           path: ./build

.github/workflows/build_pyodide.yml

@@ -9,13 +9,13 @@ jobs:
 
     steps:
       - name: Checkout Repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           submodules: recursive
           path: IfcOpenShell
 
       - name: Checkout Build Repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           repository: IfcOpenShell/build-outputs
           path: ifcopenshell_build

.github/workflows/build_rocky.yml

@@ -35,12 +35,12 @@ jobs:
           aws --version
 
       - name: Checkout Repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           submodules: recursive
 
       - name: Checkout Build Repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           repository: IfcOpenShell/build-outputs
           path: ./build

.github/workflows/build_rocky_arm.yml

@@ -35,12 +35,12 @@ jobs:
           aws --version
 
       - name: Checkout Repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           submodules: recursive
 
       - name: Checkout Build Repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           repository: IfcOpenShell/build-outputs
           path: ./build

.github/workflows/build_win.yml

@@ -27,12 +27,12 @@ jobs:
 
     steps:
       - name: Checkout Repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           submodules: recursive
 
       - name: Checkout Build Repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           repository: IfcOpenShell/build-outputs
           path: ${{ matrix.deps_dir }}

.github/workflows/ci-bcf-pypi.yml

@@ -19,8 +19,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7 # https://github.com/actions/checkout
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6 # https://github.com/actions/checkout
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           python-version: '3.11' # Version range or exact version of a Python version to use, using SemVer's version range syntax
           architecture: 'x64' # optional x64 or x86. Defaults to x64 if not specified

.github/workflows/ci-bonsai-choco.yml

@@ -34,7 +34,7 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
+      - uses: actions/checkout@v6
         with:
           fetch-tags: true
           fetch-depth: 0

.github/workflows/ci-bonsai-daily.yml

@@ -65,8 +65,8 @@ jobs:
             config:
               short_name: macos
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           architecture: 'x64' # optional x64 or x86. Defaults to x64 if not specified
           python-version: '3.11'
@@ -98,7 +98,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout bonsai_unstable_repo repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           repository: IfcOpenShell/bonsai_unstable_repo
           token: ${{ secrets.IFCOPENBOT_TOKEN }}

.github/workflows/ci-bonsai.yml

@@ -48,8 +48,8 @@ jobs:
             config:
               short_name: macos
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           architecture: 'x64' # optional x64 or x86. Defaults to x64 if not specified
           python-version: '3.11'

.github/workflows/ci-bsdd-pypi.yaml

@@ -18,8 +18,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           python-version: '3.11' # Version range or exact version of a Python version to use, using SemVer's version range syntax
       - name: Compile

.github/workflows/ci-ifc4d-pypi.yaml

@@ -18,8 +18,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           python-version: '3.11' # Version range or exact version of a Python version to use, using SemVer's version range syntax
       - name: Compile

.github/workflows/ci-ifc5d-pypi.yaml

@@ -18,8 +18,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           python-version: '3.11' # Version range or exact version of a Python version to use, using SemVer's version range syntax
       - name: Compile

.github/workflows/ci-ifccityjson-pypi.yaml

@@ -18,8 +18,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           python-version: '3.11' # Version range or exact version of a Python version to use, using SemVer's version range syntax
       - name: Compile

.github/workflows/ci-ifcclash-pypi.yaml

@@ -18,8 +18,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           python-version: '3.11' # Version range or exact version of a Python version to use, using SemVer's version range syntax
       - name: Compile

.github/workflows/ci-ifcconvert.yml

@@ -37,8 +37,8 @@ jobs:
             short_name: macosm164
           }
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           python-version: '3.11' # Version range or exact version of a Python version to use, using SemVer's version range syntax
           architecture: 'x64' # optional x64 or x86. Defaults to x64 if not specified

.github/workflows/ci-ifccsv-pypi.yaml

@@ -18,8 +18,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           python-version: '3.11' # Version range or exact version of a Python version to use, using SemVer's version range syntax
       - name: Compile

.github/workflows/ci-ifcdiff-pypi.yaml

@@ -18,8 +18,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           python-version: '3.11' # Version range or exact version of a Python version to use, using SemVer's version range syntax
       - name: Compile

.github/workflows/ci-ifcedit-pypi.yaml

@@ -18,8 +18,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6
         with:
           python-version: '3.11'
       - name: Compile

.github/workflows/ci-ifcfm-pypi.yaml

@@ -18,8 +18,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           python-version: '3.11' # Version range or exact version of a Python version to use, using SemVer's version range syntax
       - name: Compile

.github/workflows/ci-ifcmcp-pypi.yaml

@@ -18,8 +18,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6
         with:
           python-version: '3.11'
       - name: Compile

.github/workflows/ci-ifcopenshell-conda-daily.yml

@@ -21,7 +21,7 @@ jobs:
       date: ${{ steps.date.outputs.date }}
       verdate: ${{ steps.verdate.outputs.verdate }}
     steps:
-      - uses: actions/checkout@v7
+      - uses: actions/checkout@v6
       - name: Set env
         run: echo ok go
       
@@ -75,7 +75,7 @@ jobs:
             echo "ARTIFACTS_DIR=/home/runner/work/artifacts" >> $GITHUB_ENV
           fi
       
-      - uses: actions/checkout@v7
+      - uses: actions/checkout@v6
         with:
           submodules: recursive
 

.github/workflows/ci-ifcopenshell-docker.yml

@@ -21,7 +21,7 @@ jobs:
     runs-on: ubuntu-22.04
     needs: activate
     steps:
-    - uses: actions/checkout@v7
+    - uses: actions/checkout@v6
       with:
         submodules: recursive
 
@@ -86,7 +86,7 @@ jobs:
     name: Docker Build, Tag, Push
 
     steps:
-    - uses: actions/checkout@v7
+    - uses: actions/checkout@v6
       with:
         lfs: true
     

.github/workflows/ci-ifcopenshell-python-pypi.yml

@@ -47,10 +47,10 @@ jobs:
             short_name: macosm164
           }
     steps:
-      - uses: actions/checkout@v7
+      - uses: actions/checkout@v6
         with:
           submodules: recursive
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           python-version: '3.11' # Version range or exact version of a Python version to use, using SemVer's version range syntax
           architecture: 'x64' # optional x64 or x86. Defaults to x64 if not specified

.github/workflows/ci-ifcopenshell-python.yml

@@ -38,10 +38,10 @@ jobs:
             short_name: macosm164
           }
     steps:
-      - uses: actions/checkout@v7
+      - uses: actions/checkout@v6
         with:
           submodules: recursive
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           python-version: '3.11' # Version range or exact version of a Python version to use, using SemVer's version range syntax
           architecture: 'x64' # optional x64 or x86. Defaults to x64 if not specified

.github/workflows/ci-ifcpatch-pypi.yaml

@@ -18,8 +18,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           python-version: '3.11' # Version range or exact version of a Python version to use, using SemVer's version range syntax
       - name: Compile

.github/workflows/ci-ifcquery-pypi.yaml

@@ -18,8 +18,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6
         with:
           python-version: '3.11'
       - name: Compile

.github/workflows/ci-ifcsverchok-daily.yml

@@ -25,8 +25,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           architecture: 'x64' # optional x64 or x86. Defaults to x64 if not specified
           python-version: '3.11'

.github/workflows/ci-ifcsverchok.yml

@@ -19,8 +19,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           architecture: 'x64' # optional x64 or x86. Defaults to x64 if not specified
           python-version: '3.11'

.github/workflows/ci-ifctester-org.yml

@@ -10,9 +10,9 @@ jobs:
   publish_website:
     runs-on: ubuntu-22.04
     steps:
-      - uses: actions/checkout@v7
+      - uses: actions/checkout@v6
       - name: Checkout ifctester_org_static_html
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           repository: IfcOpenShell/ifctester_org_static_html
           token: ${{ secrets.IFCOPENBOT_TOKEN }}

.github/workflows/ci-ifctester-pypi.yml

@@ -18,8 +18,8 @@ jobs:
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
+      - uses: actions/checkout@v6
+      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
         with:
           python-version: '3.11' # Version range or exact version of a Python version to use, using SemVer's version range syntax
       - name: Compile

.github/workflows/ci-ifcwrap-standalone.yml

@@ -34,7 +34,7 @@ jobs:
     runs-on: ubuntu-22.04
 
     steps:
-      - uses: actions/checkout@v7
+      - uses: actions/checkout@v6
         with:
           submodules: recursive
 
@@ -85,7 +85,7 @@ jobs:
           cmake --build build-ifcopenshell --target install -j "$(nproc)"
 
       - name: Set up Python 3.11
-        uses: actions/setup-python@v7
+        uses: actions/setup-python@v6
         with:
           python-version: 3.11
 
@@ -120,7 +120,7 @@ jobs:
           PY
 
       - name: Set up Python 3.12
-        uses: actions/setup-python@v7
+        uses: actions/setup-python@v6
         with:
           python-version: 3.12
 

.github/workflows/ci-lint.yaml

@@ -12,15 +12,15 @@ jobs:
       MIN_BLENDER_PY_VERSION: "3.11"
     steps:
       - name: Action - checkout repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
 
       - name: Action - install python
-        uses: actions/setup-python@v7
+        uses: actions/setup-python@v6
         with:
           python-version: ${{ env.MIN_IOS_PY_VERSION }}
 
       - name: Action - install python
-        uses: actions/setup-python@v7
+        uses: actions/setup-python@v6
         with:
           python-version: ${{ env.MIN_BLENDER_PY_VERSION }}
 

.github/workflows/ci-pyodide-wasm-release.yml

@@ -8,7 +8,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout IfcOpenShell
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
 
       - name: Install uv
         uses: astral-sh/setup-uv@v7
@@ -25,7 +25,7 @@ jobs:
           echo "name=$(basename $WHEEL)" >> $GITHUB_OUTPUT
 
       - name: Checkout wasm-wheels
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           repository: IfcOpenShell/wasm-wheels
           path: wasm-wheels

.github/workflows/ci.yml

@@ -43,12 +43,12 @@ jobs:
       CLICOLOR_FORCE: "1"
       CMAKE_COLOR_DIAGNOSTICS: "ON"
     steps:
-      - uses: actions/checkout@v7
+      - uses: actions/checkout@v6
         with:
             submodules: recursive
 
       - name: Set up Python
-        uses: actions/setup-python@v7
+        uses: actions/setup-python@v6
         with:
           python-version: 3.11
 
@@ -257,15 +257,13 @@ jobs:
           cd ../ifcdiff && make test || ERROR=1
           cd ../ifcpatch && make test || ERROR=1
           pip install -e ../ifc5d --no-deps
-          pip install odfpy openpyxl
+          pip install odfpy xlsxwriter
           cd ../ifc5d && make test || ERROR=1
           pip install -e ../ifcquery --no-deps
           cd ../ifcquery && make test || ERROR=1
           pip install -e ../ifcedit --no-deps
           cd ../ifcedit && make test || ERROR=1
-          # Pinned <2: mcp 2.0.0 renamed mcp.server.fastmcp.FastMCP to
-          # mcp.server.mcpserver.MCPServer, which ifcmcp doesn't support yet.
-          pip install "mcp>=1.0,<2"
+          pip install mcp
           pip install -e ../ifcmcp --no-deps
           cd ../ifcmcp && make test || ERROR=1
           pip install -e ../ifctester --no-deps

.github/workflows/docs-deployment-unstable.yml

@@ -11,10 +11,10 @@ jobs:
 
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
 
       - name: Set up Python
-        uses: actions/setup-python@v7
+        uses: actions/setup-python@v6
         with:
           python-version: '3.x'
 

.github/workflows/publish-aichat-app.yaml

@@ -27,12 +27,12 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout (recursive)
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           submodules: recursive
           fetch-depth: 0
       - name: Checkout intermediate Pages repo
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           repository: IfcOpenShell/aichat_ifcopenshell_org_static_html
           ref: gh-pages
@@ -42,7 +42,7 @@ jobs:
         run: |
           rsync -av --delete --exclude='.git/' src/ifcchat/ output/
       - name: Setup Python
-        uses: actions/setup-python@v7
+        uses: actions/setup-python@v6
         with:
           python-version: "3.x"
       - name: Download wheels

.github/workflows/publish-bonsai-releases.yml

@@ -7,7 +7,7 @@ jobs:
   publish:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v7
+      - uses: actions/checkout@v6
 
       - uses: astral-sh/setup-uv@v7
 

.github/workflows/publish-pyodide-demo-app.yml

@@ -27,12 +27,12 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout (recursive)
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           submodules: recursive
           fetch-depth: 0
       - name: Checkout intermediate Pages repo
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           repository: IfcOpenShell/wasm_ifcopenshell_org_static_html
           ref: gh-pages

.github/workflows/release.yml

@@ -17,7 +17,7 @@ jobs:
     runs-on: ubuntu-22.04
     steps:
       - name: Checkout
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
         with:
           submodules: recursive
       - name: Install C++ dependencies

choco/bonsai/blenderbim.nuspec

@@ -3,7 +3,7 @@
 <package xmlns="http://schemas.microsoft.com/packaging/2015/06/nuspec.xsd">
   <metadata>
     <id>blenderbim-nightly</id>
-    <version>blenderbim_build_version</version>
+    <version>blenderbim_build_version-alpha</version>
     <packageSourceUrl>https://github.com/IfcOpenShell/IfcOpenShell</packageSourceUrl>
     <owners>fbpyr</owners>
     <!-- == SOFTWARE SPECIFIC SECTION == -->

choco/bonsai/choco_release.py

@@ -3,12 +3,11 @@
 apt update && apt install git wget curl ptpython mono-devel micro
 mkdir -p /home/runner/work/IfcOpenShell && cd /home/runner/work/IfcOpenShell
 git clone https://github.com/IfcOpenShell/IfcOpenShell
-cd /home/runner/work/IfcOpenShell/IfcOpenShell/choco/bonsai/
+cd /home/runner/work/IfcOpenShell/IfcOpenShell/choco/blenderbim/
 micro choco_release.py # paste this script, comment out push command
 export CHOCO_TOKEN="secret_choco_release_token"
 python3 choco_release.py
 """
-
 import datetime
 import hashlib
 import os
@@ -29,7 +28,7 @@ def get_repo_tag_names() -> list[str]:
 
 
 def request_repo_info(url: str):
-    req = request.Request(url)
+    req  = request.Request(url)
     resp = request.urlopen(req)
     if not resp.status == 200:
         print(f"[ERROR] could not contact server: {url}")
@@ -86,15 +85,13 @@ def run(command: str) -> None:
 
 start = datetime.datetime.now()
 
-URL_CHOCO_PACKAGE = "https://community.chocolatey.org/packages/blender"
-URL_BLENDER_CMAKE = (
-    "https://raw.githubusercontent.com/blender/blender/{}/build_files/cmake/Modules/FindPythonLibsUnix.cmake"
-)
-RE_BLENDER_VERSION_MIN_MAJ = r"Latest Version.+<span>Blender (\d+\.\d+)\..+</span>"
-RE_BLENDER_VERSION_MIN_MAJ_PAT = r"Latest Version.+<span>Blender (\d+\.\d+\.\d+)</span>"
+URL_CHOCO_PACKAGE  = "https://community.chocolatey.org/packages/blender"
+URL_BLENDER_CMAKE  = "https://raw.githubusercontent.com/blender/blender/{}/build_files/cmake/Modules/FindPythonLibsUnix.cmake"
+RE_BLENDER_VERSION_MIN_MAJ        = r"Latest Version.+<span>Blender (\d+\.\d+)\..+</span>"
+RE_BLENDER_VERSION_MIN_MAJ_PAT    = r"Latest Version.+<span>Blender (\d+\.\d+\.\d+)</span>"
 RE_BLENDER_PYTHON_VERSION_MAJ_MIN = r"\(_PYTHON_VERSION_SUPPORTED (\d+\.\d+)\)"
 
-BLENDERBIM_DIR = pathlib.Path("/home/runner/work/IfcOpenShell/IfcOpenShell/choco/bonsai/")
+BLENDERBIM_DIR = pathlib.Path("/home/runner/work/IfcOpenShell/IfcOpenShell/choco/blenderbim/")
 
 print("_____ check choco release needed?")
 
@@ -151,7 +148,7 @@ print(f"{blender_python_version_maj_min=}")
 python_version = f"py{found[0].replace('.', '')}"
 print(f"{python_version=}")
 
-blenderbim_build_version = target_release_tag.replace("bonsai-", "")
+blenderbim_build_version = target_release_tag.replace("blenderbim-", "")
 
 # url_blenderbim_py3x_win_zip
 release_zip_file_name, url_blenderbim_py3x_win_zip = get_release_zip(target_release_tag)
@@ -169,15 +166,15 @@ topics = {
         "path": HERE_DIR / "blenderbim.nuspec",
         "key_values": {
             "latest_blender_version_maj_min_pat": latest_blender_release_maj_min_pat,
-            "blenderbim_build_version": blenderbim_build_version,
+            "blenderbim_build_version"          : blenderbim_build_version,
         },
     },
     "install": {
         "path": HERE_DIR / "tools" / "chocolateyinstall.ps1",
         "key_values": {
-            "url_blenderbim_py3x_win_zip": url_blenderbim_py3x_win_zip,
+            "url_blenderbim_py3x_win_zip"      : url_blenderbim_py3x_win_zip,
             "sha256sum_blenderbim_py3x_win_zip": sha256sum_blenderbim_py3x_win_zip,
-            "latest_blender_version_maj_min": blender_version_min_maj,
+            "latest_blender_version_maj_min"   : blender_version_min_maj,
         },
     },
     "uninstall": {

nix/build-all.py

@@ -121,7 +121,7 @@ import tarfile
 import threading
 from datetime import datetime
 
-ssl._create_default_https_context = ssl._create_unverified_context  # ty:ignore[invalid-assignment]
+ssl._create_default_https_context = ssl._create_unverified_context
 
 import time
 from collections.abc import Generator, Sequence
@@ -697,10 +697,7 @@ def build_dependency(
             compr = "xz"
         else:
             raise RuntimeError("fix source for new download type")
-        # ty: false positive bug upstream.
-        download_tarfile = tarfile.open(
-            name=download_tarfile_path, mode=f"r:{compr}"
-        )  # ty:ignore[no-matching-overload]
+        download_tarfile = tarfile.open(name=download_tarfile_path, mode=f"r:{compr}")
         # tarfile seriously doesn't have a function to retrieve the root directory more easily
         extract_dir_name = os.path.commonprefix([x for x in download_tarfile.getnames() if x != "."])
         # run([tar, "--exclude=\"*/*\"", "-tf", download_name], cwd=build_dir).strip() no longer works

pyproject.toml

@@ -156,8 +156,7 @@ exclude = [
 [tool.poe.tasks]
 
 dev-setup.sequence = [
-    # 3.13 is chosen because it's the version used in the latest Bonsai.
-    {cmd = "uv sync --python 3.13"},
+    {cmd = "uv sync"},
     {cmd = "uv pip install -e ./src/bsdd/"},
     {cmd = "uv pip install -e ./src/ifcopenshell-python/[advanced,dev]"},
     {cmd = "uv pip install -e ./src/ifcedit/"},

requirements-tools.txt

@@ -1,5 +1,5 @@
 black==26.3.1
-ruff==0.16.0
+ruff==0.15.22
 poethepoet
-ty==0.0.63
-gersemi==0.28.0
+ty==0.0.61
+gersemi==0.26.1

src/bonsai/bonsai/bim/data/webui/sioserver.py

@@ -10,7 +10,6 @@ if bonsai_lib_path:
 import argparse
 import base64
 import json
-import urllib.parse
 import xml.etree.ElementTree as ET
 
 import pystache
@@ -19,53 +18,8 @@ from aiohttp import web
 
 sio_port = 8080  # default port
 
-
-def get_asset_version() -> str:
-    """A cache-busting token appended to locally served static asset URLs.
-
-    Browsers otherwise keep serving a stale cached copy of static/js and
-    static/css after Bonsai ships a code change, until the user does a hard
-    refresh. Using the Bonsai version (which includes the build's commit
-    hash) means the token changes on every shipped update.
-    """
-    if bonsai_version:
-        return urllib.parse.quote(bonsai_version, safe="")
-    # Fallback for standalone runs without BONSAI_VERSION set (e.g. running
-    # sioserver.py directly outside of Blender): derive a token from the
-    # newest mtime among the static assets, so it still changes whenever the
-    # shipped files change.
-    static_dir = os.path.join(os.path.dirname(os.path.abspath(__file__)), "static")
-    latest_mtime = 0
-    for root, _dirs, files in os.walk(static_dir):
-        for name in files:
-            latest_mtime = max(latest_mtime, int(os.path.getmtime(os.path.join(root, name))))
-    return f"dev-{latest_mtime}"
-
-
-asset_version = get_asset_version()
-
-
-@web.middleware
-async def no_cache_static_middleware(request: web.Request, handler):
-    """Force revalidation of locally served static assets.
-
-    Query-string version stamping (see `asset_version`) busts the cache for
-    the HTML-referenced entry points, but JS files that statically import
-    other local modules (e.g. cost.js/gantt.js importing utilities/costui.js)
-    reference those modules by an un-stamped relative path. Marking all
-    /static/ and /jsgantt/ responses as no-cache makes browsers always
-    revalidate with the server (a cheap conditional GET / 304 when nothing
-    changed), so nested imports also pick up shipped changes without
-    requiring a hard refresh.
-    """
-    response = await handler(request)
-    if request.path.startswith("/static/") or request.path.startswith("/jsgantt/"):
-        response.headers["Cache-Control"] = "no-cache, must-revalidate"
-    return response
-
-
 sio = socketio.AsyncServer(cors_allowed_origins="*", async_mode="aiohttp", max_http_buffer_size=10000000)
-app = web.Application(middlewares=[no_cache_static_middleware])
+app = web.Application()
 sio.attach(app)
 
 
@@ -245,28 +199,28 @@ class BlenderNamespace(socketio.AsyncNamespace):
 async def schedules(request):
     with open("templates/index.html", "r") as f:
         template = f.read()
-    html_content = pystache.render(template, {"port": sio_port, "version": bonsai_version, "v": asset_version})
+    html_content = pystache.render(template, {"port": sio_port, "version": bonsai_version})
     return web.Response(text=html_content, content_type="text/html")
 
 
 async def costing(request):
     with open("templates/costing.html", "r") as f:
         template = f.read()
-    html_content = pystache.render(template, {"port": sio_port, "version": bonsai_version, "v": asset_version})
+    html_content = pystache.render(template, {"port": sio_port, "version": bonsai_version})
     return web.Response(text=html_content, content_type="text/html")
 
 
 async def sequencing(request):
     with open("templates/gantt.html", "r") as f:
         template = f.read()
-    html_content = pystache.render(template, {"port": sio_port, "version": bonsai_version, "v": asset_version})
+    html_content = pystache.render(template, {"port": sio_port, "version": bonsai_version})
     return web.Response(text=html_content, content_type="text/html")
 
 
 async def documentation(request):
     with open("templates/drawings.html", "r") as f:
         template = f.read()
-    html_content = pystache.render(template, {"port": sio_port, "version": bonsai_version, "v": asset_version})
+    html_content = pystache.render(template, {"port": sio_port, "version": bonsai_version})
     return web.Response(text=html_content, content_type="text/html")
 
 
@@ -275,7 +229,7 @@ async def documentation(request):
 async def demo(request):
     with open("templates/demo.html", "r") as f:
         template = f.read()
-    html_content = pystache.render(template, {"port": sio_port, "version": bonsai_version, "v": asset_version})
+    html_content = pystache.render(template, {"port": sio_port, "version": bonsai_version})
     return web.Response(text=html_content, content_type="text/html")
 
 

src/bonsai/bonsai/bim/data/webui/static/js/utilities/costui.js

@@ -250,14 +250,6 @@ export class CostUI {
       },
     });
 
-    CostUI.addRibbonButton({
-      text: "Download CSV",
-      icon: "fa-solid fa-file-csv",
-      callback: () => {
-        CostUI.downloadCsv();
-      },
-    });
-
     CostUI.addRibbonButton({
       text: "Hide Schedules",
       icon: "fa-regular fa-eye-slash",
@@ -531,81 +523,6 @@ export class CostUI {
     }
   }
 
-  static downloadCsv() {
-    const tables = document.querySelectorAll("table[id^='cost-items-']");
-    if (tables.length === 0) {
-      alert("No cost schedule loaded to export!");
-      return;
-    }
-    tables.forEach((table) => {
-      const scheduleId = table.id.split("-").pop();
-      const csv = CostUI.tableToCsv(table);
-      if (csv === null) {
-        return;
-      }
-      const nameEl = document.querySelector(
-        "#cost-schedule-container-" + scheduleId + " .form-header span"
-      );
-      const scheduleName = nameEl
-        ? nameEl.textContent
-        : "cost_schedule_" + scheduleId;
-      CostUI.triggerCsvDownload(csv, scheduleName + ".csv");
-    });
-  }
-
-  static tableToCsv(table) {
-    const escapeCsvCell = (value) => {
-      const text = (value === null || value === undefined ? "" : value)
-        .toString()
-        .trim();
-      if (/[",\n]/.test(text)) {
-        return '"' + text.replace(/"/g, '""') + '"';
-      }
-      return text;
-    };
-
-    const cellText = (cell) => {
-      const input = cell.querySelector("input");
-      return input ? input.value : cell.innerText;
-    };
-
-    // The Actions column only holds buttons (edit/delete/etc), not data.
-    const isDataColumn = (column) => column && column !== "Actions";
-
-    const headerCells = Array.from(table.querySelectorAll("thead th")).filter(
-      (th) => isDataColumn(th.getAttribute("data-column"))
-    );
-    if (headerCells.length === 0) {
-      return null;
-    }
-
-    const rows = [headerCells.map((th) => escapeCsvCell(th.textContent)).join(",")];
-
-    table.querySelectorAll("tbody tr").forEach((row) => {
-      const cells = Array.from(row.children).filter((cell) =>
-        isDataColumn(cell.getAttribute("data-column"))
-      );
-      if (cells.length === 0) {
-        return; // e.g. the "No cost items found" placeholder row.
-      }
-      rows.push(cells.map((cell) => escapeCsvCell(cellText(cell))).join(","));
-    });
-
-    return rows.join("\n");
-  }
-
-  static triggerCsvDownload(csvContent, filename) {
-    const blob = new Blob([csvContent], { type: "text/csv;charset=utf-8;" });
-    const url = URL.createObjectURL(blob);
-    const link = document.createElement("a");
-    link.href = url;
-    link.download = filename;
-    document.body.appendChild(link);
-    link.click();
-    document.body.removeChild(link);
-    URL.revokeObjectURL(url);
-  }
-
   static createCostTable({ costSchedule, currency, callbacks }) {
     const preferences = CostUI.getColumnPreferences();
     const isScheduleOfRates = costSchedule.PredefinedType === "SCHEDULEOFRATES";

src/bonsai/bonsai/bim/data/webui/templates/costing.html

@@ -7,7 +7,7 @@
     <link
       rel="stylesheet"
       type="text/css"
-      href="/static/css/cost.css?v={{v}}"
+      href="/static/css/cost.css"
       id="index-stylesheet"
     />
     <link
@@ -21,7 +21,7 @@
     />
     <script
       type="text/javascript"
-      src="/static/js/jquery.min.js?v={{v}}"
+      src="/static/js/jquery.min.js"
     ></script>
     <script
       type="text/javascript"
@@ -34,7 +34,7 @@
     <script>
       var SOCKET_PORT = {{port}};
     </script>
-    <script type="module" defer src="./static/js/cost.js?v={{v}}"></script>
+    <script type="module" defer src="./static/js/cost.js"></script>
   </head>
   <body>
     <nav>

src/bonsai/bonsai/bim/data/webui/templates/demo.html

@@ -12,14 +12,14 @@
     />
     <!-- here we request the CSS file from the server, -->
     <!-- using registered static path in the server -->
-    <link rel="stylesheet" href="/static/css/demo.css?v={{v}}" id="demo-stylesheet" />
+    <link rel="stylesheet" href="/static/css/demo.css" id="demo-stylesheet" />
     <link
       rel="stylesheet"
       href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/6.6.0/css/all.min.css"
     />
     <script
       type="text/javascript"
-      src="/static/js/jquery.min.js?v={{v}}"
+      src="/static/js/jquery.min.js"
     ></script>
     <script
       type="text/javascript"
@@ -33,7 +33,7 @@
     </script>
     <!-- here we request the JS file from the server-->
     <!-- using registered static path in the server -->
-    <script defer src="./static/js/demo.js?v={{v}}"></script>
+    <script defer src="./static/js/demo.js"></script>
   </head>
   <body>
     <!-- the navigation bar at the top of the page. -->

src/bonsai/bonsai/bim/data/webui/templates/drawings.html

@@ -15,7 +15,7 @@
     />
     <link
       rel="stylesheet"
-      href="/static/css/drawings.css?v={{v}}"
+      href="/static/css/drawings.css"
       id="drawings-stylesheet"
     />
     <link
@@ -24,7 +24,7 @@
     />
     <script
       type="text/javascript"
-      src="/static/js/jquery.min.js?v={{v}}"
+      src="/static/js/jquery.min.js"
     ></script>
     <script
       type="text/javascript"
@@ -45,7 +45,7 @@
     <script>
       var SOCKET_PORT = {{port}};
     </script>
-    <script defer src="./static/js/drawings.js?v={{v}}"></script>
+    <script defer src="./static/js/drawings.js"></script>
   </head>
   <body>
     <nav>

src/bonsai/bonsai/bim/data/webui/templates/gantt.html

@@ -9,25 +9,25 @@
       type="image/x-icon"
       href="https://bonsaibim.org/assets/images/favicon-blender.png"
     />
-    <link rel="stylesheet" type="text/css" href="/jsgantt/jsgantt.css?v={{v}}" />
-    <link rel="stylesheet" href="/static/css/gantt.css?v={{v}}" id="gantt-stylesheet" />
+    <link rel="stylesheet" type="text/css" href="/jsgantt/jsgantt.css" />
+    <link rel="stylesheet" href="/static/css/gantt.css" id="gantt-stylesheet" />
     <link
       rel="stylesheet"
       href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/6.6.0/css/all.min.css"
     />
     <script
       type="text/javascript"
-      src="/static/js/jquery.min.js?v={{v}}"
+      src="/static/js/jquery.min.js"
     ></script>
     <script
       type="text/javascript"
       src="https://cdn.socket.io/4.0.0/socket.io.min.js"
     ></script>
-    <script type="text/javascript" src="./jsgantt/jsgantt.js?v={{v}}"></script>
+    <script type="text/javascript" src="./jsgantt/jsgantt.js"></script>
     <script>
       var SOCKET_PORT = {{port}};
     </script>
-    <script type="module" defer src="./static/js/gantt.js?v={{v}}"></script>
+    <script type="module" defer src="./static/js/gantt.js"></script>
   </head>
   <body>
     <nav class="no-print">

src/bonsai/bonsai/bim/data/webui/templates/index.html

@@ -9,7 +9,7 @@
       type="image/x-icon"
       href="https://bonsaibim.org/assets/images/favicon-blender.png"
     />
-    <link rel="stylesheet" href="/static/css/index.css?v={{v}}" id="index-stylesheet" />
+    <link rel="stylesheet" href="/static/css/index.css" id="index-stylesheet" />
     <link
       rel="stylesheet"
       id="tabulator-stylesheet"
@@ -21,7 +21,7 @@
     />
     <script
       type="text/javascript"
-      src="/static/js/jquery.min.js?v={{v}}"
+      src="/static/js/jquery.min.js"
     ></script>
     <script
       type="text/javascript"
@@ -34,7 +34,7 @@
     <script>
       var SOCKET_PORT = {{port}};
     </script>
-    <script defer src="./static/js/index.js?v={{v}}"></script>
+    <script defer src="./static/js/index.js"></script>
   </head>
   <body>
     <nav>

src/bonsai/bonsai/bim/handler.py

@@ -418,10 +418,9 @@ def get_user(ifc: ifcopenshell.file) -> Union[ifcopenshell.entity_instance, None
 
 
 def viewport_shading_changed_callback(area: bpy.types.Area) -> None:
-    shading_type = area.spaces.active.shading.type
-    tool.Style.restore_material_style_types(shading_type)
-    if shading_type == "SOLID":
-        area.spaces.active.shading.color_type = "MATERIAL"
+    shading = area.spaces.active.shading.type
+    if shading == "RENDERED":
+        tool.Style.get_style_props().active_style_type = "External"
 
 
 def subscribe_to_viewport_shading_changes():

src/bonsai/bonsai/bim/helper.py

@@ -236,8 +236,11 @@ def import_attribute(
     elif data_type == "integer":
         new.int_value = 0 if new.is_null else int(data[attribute.name()])
     elif data_type == "float":
-        measure_class = attribute.type_of_attribute().declared_type().name()
-        new.special_type = tool.Pset.get_special_type_for_measure_class(measure_class)
+        attribute_type = attribute.type_of_attribute()
+        if attribute_type._is("IfcLengthMeasure"):
+            new.special_type = "LENGTH"
+        elif attribute_type._is("IfcForceMeasure"):
+            new.special_type = "FORCE"
         new.float_value = 0.0 if new.is_null else float(data[attribute.name()])
     elif data_type == "enum":
         attribute_type = attribute.type_of_attribute()

src/bonsai/bonsai/bim/import_ifc.py

@@ -133,6 +133,10 @@ class MaterialCreator:
                 if shape_has_openings and coords.is_a("IfcIndexedTextureMap"):
                     continue
                 tool.Loader.load_indexed_map(coords, self.mesh)
+            elif tool.Style.get_texture_style(material):
+                # No explicit coordinate mapping (e.g. IFC2X3 has no IsMappedBy,
+                # and IFC4 COORD uses generated UVs). Bake XY→UV as fallback.
+                tool.Loader.load_generated_uv_map(self.mesh)
 
     def assign_material_slots_to_faces(self) -> None:
         if not self.mesh["ios_materials"]:

src/bonsai/bonsai/bim/module/material/__init__.py

@@ -56,7 +56,6 @@ classes = (
     operator.RemoveMaterial,
     operator.RemoveMaterialSet,
     operator.RemoveProfile,
-    operator.RenameMaterial,
     operator.ReorderMaterialSetItem,
     operator.SelectByMaterial,
     operator.SelectMaterialInMaterialsUI,

src/bonsai/bonsai/bim/module/material/operator.py

@@ -102,26 +102,6 @@ class EditMaterial(bpy.types.Operator, tool.Ifc.Operator):
         core.edit_material(tool.Ifc, tool.Material, material=tool.Ifc.get().by_id(self.material))
 
 
-class RenameMaterial(bpy.types.Operator, tool.Ifc.Operator):
-    bl_idname = "bim.rename_material"
-    bl_label = "Rename Material"
-    bl_description = "Rename an IfcMaterial"
-    bl_options = {"REGISTER", "UNDO"}
-    material: bpy.props.IntProperty()
-    name: bpy.props.StringProperty(name="Name")
-
-    def invoke(self, context, event):
-        material = tool.Ifc.get().by_id(self.material)
-        self.name = material.Name or ""
-        return context.window_manager.invoke_props_dialog(self)
-
-    def draw(self, context):
-        self.layout.prop(self, "name")
-
-    def _execute(self, context):
-        core.rename_material(tool.Ifc, tool.Material, material=tool.Ifc.get().by_id(self.material), name=self.name)
-
-
 class DisableEditingMaterial(bpy.types.Operator):
     bl_idname = "bim.disable_editing_material"
     bl_label = "Disable Editing Material"

src/bonsai/bonsai/bim/module/project/data.py

@@ -162,7 +162,7 @@ class ProjectLibraryData:
         library_file = IfcStore.library_file
         if library_file is None or library_file.schema == "IFC2X3":
             return results
-        root = library_file.by_type("IfcProject")[0]
+        root = tool.Project.get_root_context(library_file)
         results.append((str(root.id()), f"{root.is_a()} {root.Name or 'Unnamed'}", root.Description or ""))
         for library_id, data in cls.data["project_libraries"].items():
             results.append((str(library_id), data["Name"] or "Unnamed", data["Description"] or ""))

src/bonsai/bonsai/bim/module/project/operator.py

@@ -281,7 +281,7 @@ class RefreshLibrary(bpy.types.Operator):
         elements = {e for e in elements if not tool.Project.is_element_assigned_to_project_library(e, rels)}
         self.props.add_library_project_library("Unassigned", len(elements), 0, False)
 
-        root_context = library_file.by_type("IfcProject")[0]
+        root_context = tool.Project.get_root_context(library_file)
         hierarchy = tool.Project.get_project_hierarchy(library_file)
         tool.Project.load_project_libraries_to_ui(root_context, hierarchy)
         return {"FINISHED"}
@@ -637,10 +637,12 @@ class AppendLibraryElement(bpy.types.Operator, tool.Ifc.Operator):
         elif element.is_a("IfcProduct"):
             # NOTE: Non-types are not exposed in UI directly
             # but the code is still used when appending products by query.
-            self.import_product_from_ifc(element, context)
-            element_type = ifcopenshell.util.element.get_type(element)
-            if element_type is not None and tool.Ifc.get_object(element_type) is None:
-                self.import_type_from_ifc(element_type, context)
+            elements = self.get_appended_elements(element)
+            self.import_product_from_ifc(elements, context)
+            for appended_element in elements:
+                element_type = ifcopenshell.util.element.get_type(appended_element)
+                if element_type is not None and tool.Ifc.get_object(element_type) is None:
+                    self.import_type_from_ifc(element_type, context)
         elif element.is_a("IfcMaterial"):
             self.import_material_from_ifc(element, context)
         elif element.is_a("IfcSurfaceStyle"):
@@ -676,17 +678,32 @@ class AppendLibraryElement(bpy.types.Operator, tool.Ifc.Operator):
         ifc_importer.file = self.file
         ifc_importer.create_style(style)
 
-    def import_product_from_ifc(self, element: ifcopenshell.entity_instance, context: bpy.types.Context) -> None:
+    def get_appended_elements(self, element: ifcopenshell.entity_instance) -> set[ifcopenshell.entity_instance]:
+        """``element`` plus any parts brought in with it (e.g. the parts of an appended assembly)."""
+        elements = {element}
+        for part in ifcopenshell.util.element.get_decomposition(element):
+            if not part.is_a("IfcFeatureElement") or part.is_a("IfcSurfaceFeature"):
+                elements.add(part)
+        return elements
+
+    def import_product_from_ifc(
+        self,
+        elements: Union[ifcopenshell.entity_instance, set[ifcopenshell.entity_instance]],
+        context: bpy.types.Context,
+    ) -> None:
         self.file = tool.Ifc.get()
+        if isinstance(elements, ifcopenshell.entity_instance):
+            elements = {elements}
         logger = logging.getLogger("ImportIFC")
         ifc_import_settings = import_ifc.IfcImportSettings.factory(context, IfcStore.path, logger)
         ifc_importer = import_ifc.IfcImporter(ifc_import_settings)
         ifc_importer.file = self.file
         ifc_importer.process_context_filter()
         ifc_importer.material_creator.load_existing_materials()
-        self.import_materials(element, ifc_importer)
-        self.import_styles(element, ifc_importer)
-        ifc_importer.create_generic_elements({element})
+        for element in elements:
+            self.import_materials(element, ifc_importer)
+            self.import_styles(element, ifc_importer)
+        ifc_importer.create_generic_elements(elements)
         ifc_importer.place_objects_in_collections()
 
     def import_type_from_ifc(self, element: ifcopenshell.entity_instance, context: bpy.types.Context) -> None:
@@ -761,22 +778,21 @@ class EditProjectLibrary(bpy.types.Operator):
         attributes = bonsai.bim.helper.export_attributes(props.project_library_attributes)
         ifcopenshell.api.attribute.edit_attributes(library_file, project_library, attributes)
 
-        # Update parent library. Tear down the old IfcRelDeclares/IfcRelNests before
-        # creating the new one; a library must have exactly one of the two, never both.
+        # Update parent library.
         previous_parent_library = tool.Project.get_parent_library(project_library)
         new_parent_library = library_file.by_id(int(props.parent_library))
         if previous_parent_library != new_parent_library:
-            if previous_parent_library is not None:
-                if previous_parent_library.is_a("IfcProject"):
-                    ifcopenshell.api.project.unassign_declaration(
-                        library_file, [project_library], previous_parent_library
-                    )
-                else:
-                    ifcopenshell.api.nest.unassign_object(library_file, [project_library])
-            if new_parent_library.is_a("IfcProject"):
-                ifcopenshell.api.project.assign_declaration(library_file, [project_library], new_parent_library)
-            else:
+            if previous_parent_library is None:
+                # Edited library was a root in a library-only file; nest it under the new parent.
                 ifcopenshell.api.nest.assign_object(library_file, [project_library], new_parent_library)
+            elif previous_parent_library.is_a("IfcProject"):
+                # Then new one is IfcProjectLibrary.
+                ifcopenshell.api.nest.assign_object(library_file, [project_library], new_parent_library)
+            else:  # Previous is IfcProjectLibrary.
+                ifcopenshell.api.nest.unassign_object(library_file, [project_library])
+                # If new one is IfcProject, then it's already assigned by default.
+                if new_parent_library.is_a("IfcProjectLibrary"):
+                    ifcopenshell.api.nest.assign_object(library_file, [project_library], new_parent_library)
 
         props.is_editing_project_library = False
         bpy.ops.bim.refresh_library()
@@ -810,9 +826,12 @@ class AddProjectLibrary(bpy.types.Operator):
         props = tool.Project.get_project_props()
         library_file = IfcStore.library_file
         assert library_file
-        root_context = library_file.by_type("IfcProject")[0]
+        root_context = tool.Project.get_root_context(library_file)
         project_library = ifcopenshell.api.root.create_entity(library_file, "IfcProjectLibrary")
-        ifcopenshell.api.project.assign_declaration(library_file, [project_library], root_context)
+        if root_context.is_a("IfcProject"):
+            ifcopenshell.api.project.assign_declaration(library_file, [project_library], root_context)
+        else:
+            ifcopenshell.api.nest.assign_object(library_file, [project_library], root_context)
         ProjectLibraryData.load()  # Update enum.
         props.selected_project_library = str(project_library.id())
         props.is_editing_project_library = True
@@ -1299,10 +1318,6 @@ class LoadProjectElements(bpy.types.Operator):
         tool.Project.set_default_modeling_dimensions()
         tool.Root.reload_grid_decorator()
         bonsai.bim.handler.refresh_ui_data()
-        for screen in bpy.data.screens:
-            for area in screen.areas:
-                if area.type == "VIEW_3D":
-                    bonsai.bim.handler.viewport_shading_changed_callback(area)
         return {"FINISHED"}
 
     def get_decomposition_elements(self) -> set[ifcopenshell.entity_instance]:
@@ -1580,13 +1595,10 @@ class LoadLink(bpy.types.Operator, tool.Ifc.Operator):
         json_filepath = self.filepath_.with_suffix(".ifc.cache.json")
 
         def should_clear_cache() -> bool:
-            # Nothing to clear if the cache file was never created (e.g. a
-            # fresh link). Check this first so os.remove below is never
-            # called on a non-existent path, regardless of use_cache.
-            if not blend_filepath.exists():
-                return False
             if not self.use_cache:
                 return True
+            if not blend_filepath.exists():
+                return False
             data = json.loads(json_filepath.read_text())
             # Empty 'query' - model loaded without custom query.
             # Missing 'query' - model was loaded before custom queries were introduced in Bonsai.
@@ -2777,10 +2789,12 @@ class AppendInspectedLinkedElement(AppendLibraryElement):
             library=linked_ifc_file,
             element=element_to_append,
         )
-        self.import_product_from_ifc(element, context)
-        element_type = ifcopenshell.util.element.get_type(element)
-        if element_type and tool.Ifc.get_object(element_type) is None:
-            self.import_type_from_ifc(element_type, context)
+        elements = self.get_appended_elements(element)
+        self.import_product_from_ifc(elements, context)
+        for appended_element in elements:
+            element_type = ifcopenshell.util.element.get_type(appended_element)
+            if element_type and tool.Ifc.get_object(element_type) is None:
+                self.import_type_from_ifc(element_type, context)
 
         return {"FINISHED"}
 

src/bonsai/bonsai/bim/module/pset/data.py

@@ -54,7 +54,7 @@ class Data:
         ifc_file = tool.Ifc.get()
         results = []
         psetqtos = ifcopenshell.util.element.get_psets(
-            element, psets_only=psets_only, qtos_only=qtos_only, should_inherit=False, verbose=True
+            element, psets_only=psets_only, qtos_only=qtos_only, should_inherit=False
         )
         for name, data in sorted(psetqtos.items()):
             pset = ifc_file.by_id(data["id"])
@@ -69,28 +69,13 @@ class Data:
                     "id": data["id"],
                     "Name": name,
                     "is_expanded": is_expanded.get(data["id"], True),
-                    "Properties": [
-                        cls.property_display_data(ifc_file, k, v) for k, v in sorted(data.items()) if k != "id"
-                    ],
+                    "Properties": [{"Name": k, "NominalValue": v} for k, v in sorted(data.items()) if k != "id"],
                     "shared_pset_uses": len(pset_uses),
                     "has_template": has_template,
                 }
             )
         return sorted(results, key=lambda v: v["Name"])
 
-    @classmethod
-    def property_display_data(cls, ifc_file: ifcopenshell.file, name: str, verbose_value: Any) -> dict[str, Any]:
-        # Predefined property sets (e.g. IfcDoorPanelProperties) expose plain
-        # attribute values even in verbose mode, since they're typed IFC
-        # attributes rather than IfcProperty entities with their own id/Unit.
-        if not isinstance(verbose_value, dict):
-            return {"Name": name, "NominalValue": verbose_value, "UnitSymbol": ""}
-
-        unit_symbol = ""
-        if (prop_id := verbose_value.get("id")) and (prop_entity := ifc_file.by_id(prop_id)):
-            unit_symbol = tool.Pset.get_unit_symbol_for_prop(prop_entity, ifc_file)
-        return {"Name": name, "NominalValue": verbose_value["value"], "UnitSymbol": unit_symbol}
-
     @classmethod
     def format_pset_enum(cls, psets):
         enum_items = []

src/bonsai/bonsai/bim/module/pset/operator.py

@@ -120,20 +120,13 @@ class EditPset(bpy.types.Operator, tool.Ifc.Operator):
             properties = json.loads(self.properties)
         else:
             for prop in props.properties:
-                metadata = prop.metadata
                 if prop.value_type == "IfcPropertySingleValue":
-                    value = metadata.get_value()
+                    properties[prop.metadata.name] = prop.metadata.get_value()
                 elif prop.value_type == "IfcPropertyEnumeratedValue":
-                    value_name = metadata.get_value_name()
-                    value = [e[value_name] for e in prop.enumerated_value.enumerated_values if e.is_selected]
-                else:
-                    continue
-                # None (a purge/skip-creation signal, handled by edit_pset/edit_qto before any
-                # unit wrapping is unpacked) must stay bare -- only wrap real values.
-                if value is not None and tool.Pset.is_measurable_special_type(metadata.special_type):
-                    unit = self.file.by_id(metadata.unit_id) if metadata.unit_id else None
-                    value = {"NominalValue": value, "Unit": unit}
-                properties[metadata.name] = value
+                    value_name = prop.metadata.get_value_name()
+                    properties[prop.metadata.name] = [
+                        e[value_name] for e in prop.enumerated_value.enumerated_values if e.is_selected
+                    ]
 
         if pset.is_a() in ("IfcPropertySet", "IfcMaterialProperties", "IfcProfileProperties"):
             ifcopenshell.api.pset.edit_pset(
@@ -147,18 +140,10 @@ class EditPset(bpy.types.Operator, tool.Ifc.Operator):
             for key, value in properties.items():
                 if value is None:
                     continue
-                is_wrapped = isinstance(value, dict) and "Unit" in value
-                raw = value["NominalValue"] if is_wrapped else value
-                if raw is None:
-                    continue
-                if isinstance(raw, float):
-                    raw = round(raw, 4)
-                elif not isinstance(raw, int):
-                    raw = 0
-                if is_wrapped:
-                    value["NominalValue"] = raw
-                else:
-                    properties[key] = raw
+                if isinstance(value, float):
+                    properties[key] = round(value, 4)
+                elif not isinstance(value, int):
+                    properties[key] = 0
             ifcopenshell.api.pset.edit_qto(
                 self.file,
                 qto=pset,

src/bonsai/bonsai/bim/module/pset/ui.py

@@ -67,10 +67,6 @@ def draw_single_property(prop: IfcProperty, layout: bpy.types.UILayout, copy_ope
     if prop.metadata.special_type == "URI":
         op = layout.operator("bim.select_uri_attribute", text="", icon="FILE_FOLDER")
         op.attribute_data_path = tool.Blender.get_full_data_path(prop.metadata)
-    if tool.Pset.is_measurable_special_type(prop.metadata.special_type):
-        unit_row = layout.row(align=True)
-        unit_row.scale_x = 0.5
-        prop_with_search(unit_row, prop.metadata, "unit_id_enum", text="")
     if prop.metadata.is_optional:
         layout.prop(prop.metadata, "is_null", icon="RADIOBUT_OFF" if prop.metadata.is_null else "RADIOBUT_ON", text="")
     if copy_operator:
@@ -207,10 +203,9 @@ def draw_psetqto_ui(
                 row = box.row(align=True)
                 row.scale_y = 0.8
                 row.label(text=prop["Name"])
-                display_value = get_display_value(nominal_value)
-                if unit_symbol := prop["UnitSymbol"]:
-                    display_value = f"{display_value} {unit_symbol}"
-                op = row.operator("bim.select_similar", text=display_value, icon="NONE", emboss=False)
+                op = row.operator(
+                    "bim.select_similar", text=get_display_value(nominal_value), icon="NONE", emboss=False
+                )
                 op.key = '"' + pset["Name"].replace('"', '\\"') + '"."' + prop["Name"].replace('"', '\\"') + '"'
                 # calculate sum of all selected objects
                 if active_operator:

src/bonsai/bonsai/bim/module/qto/calculator.py

@@ -104,7 +104,7 @@ def get_footing_length(o: bpy.types.Object) -> float:
         return get_length(o)
     if predefined_type == "FOOTING_BEAM" or predefined_type == "STRIP_FOOTING":
         return get_z(o)
-    elif predefined_type == "PAD_FOOTING" or predefined_type == "PILE_CAP":
+    elif predefined_type == "PAD_FOOTING":
         return max(get_x(o), get_y(o))
     else:
         return get_length(o)
@@ -197,26 +197,12 @@ def get_footing_height(o: bpy.types.Object) -> float:
         return get_height(o)
     if predefined_type == "FOOTING_BEAM" or predefined_type == "STRIP_FOOTING":
         return get_y(o)
-    elif predefined_type == "PAD_FOOTING" or predefined_type == "PILE_CAP":
+    elif predefined_type == "PAD_FOOTING":
         return get_z(o)
     else:
         return get_height(o)
 
 
-def get_footing_width(o: bpy.types.Object) -> float:
-    element = tool.Ifc.get_entity(o)
-    assert element
-    predefined_type = ifcopenshell.util.element.get_predefined_type(element)
-    if not predefined_type:
-        return get_width(o)
-    if predefined_type == "FOOTING_BEAM" or predefined_type == "STRIP_FOOTING":
-        return get_x(o)
-    elif predefined_type == "PAD_FOOTING" or predefined_type == "PILE_CAP":
-        return get_width(o)
-    else:
-        return get_width(o)
-
-
 def get_height(o: bpy.types.Object) -> float:
     """_summary_: Returns the height of the object bounding box
 
@@ -239,7 +225,7 @@ def get_opening_depth(obj: bpy.types.Object) -> float:
     if is_opening_horizontal(obj):
         return get_height(obj)
     else:
-        return get_y(obj)
+        return get_width(obj)
 
 
 def get_opening_mapping_area(obj: bpy.types.Object) -> float:

src/bonsai/bonsai/bim/module/qto/operator.py

@@ -151,7 +151,7 @@ class CalculateSingleQuantity(bpy.types.Operator, tool.Ifc.Operator):
         ifc_file = tool.Ifc.get()
         with Profiler("Quantify function time:"):
             results = ifc5d.qto.quantify(ifc_file, elements, rules)
-        ifc5d.qto.edit_qtos(ifc_file, results, target_units=tool.Qto.get_target_units(), rules=rules)
+        ifc5d.qto.edit_qtos(ifc_file, results)
 
         not_quantified_elements = elements - set(results.keys())
         not_quantified_message = tool.Qto.get_not_quantified_elements_message(not_quantified_elements)
@@ -194,7 +194,7 @@ class PerformQuantityTakeOff(bpy.types.Operator, tool.Ifc.Operator):
             ifc_file = tool.Ifc.get()
             with Profiler("Quantify function time:"):
                 results = ifc5d.qto.quantify(ifc_file, elements, rules)
-            ifc5d.qto.edit_qtos(ifc_file, results, target_units=tool.Qto.get_target_units(), rules=rules)
+            ifc5d.qto.edit_qtos(ifc_file, results)
             not_quantified_elements = elements - set(results.keys())
             return not_quantified_elements
 

src/bonsai/bonsai/bim/module/qto/prop.py

@@ -27,28 +27,10 @@ from bpy.props import (
 )
 from bpy.types import PropertyGroup
 
-import bonsai.bim.prop
 import bonsai.tool as tool
 
 CALCULATOR_FUNCTION_ENUM_ITEMS: list[Union[tuple[str, str, str], None]] = []
 
-# Measure class (matching ifc5d.qto's Function.measure / SI2ProjectUnitConverter.project_units'
-# keys) -> (BIMQtoProperties field name, tool.Pset special_type, UI label).
-MEASURE_TO_TARGET_UNIT_FIELD: dict[str, tuple[str, str, str]] = {
-    "IfcLengthMeasure": ("target_unit_length", "LENGTH", "Length"),
-    "IfcAreaMeasure": ("target_unit_area", "AREA", "Area"),
-    "IfcVolumeMeasure": ("target_unit_volume", "VOLUME", "Volume"),
-    "IfcMassMeasure": ("target_unit_mass", "MASS", "Mass"),
-    "IfcTimeMeasure": ("target_unit_time", "TIME", "Time"),
-}
-
-
-def _target_unit_items(special_type: str):
-    def getter(self: "BIMQtoProperties", context: bpy.types.Context) -> tool.Blender.BLENDER_ENUM_ITEMS:
-        return bonsai.bim.prop.get_unit_enum_items_for_special_type(special_type, tool.Ifc.get())
-
-    return getter
-
 
 def get_qto_rule(self: "BIMQtoProperties", context: bpy.types.Context) -> list[tuple[str, str, str]]:
     results: list[tuple[str, str, str]] = []
@@ -110,11 +92,6 @@ class BIMQtoProperties(PropertyGroup):
         ),
         default=False,
     )
-    target_unit_length: EnumProperty(items=_target_unit_items("LENGTH"), name="Length Unit")
-    target_unit_area: EnumProperty(items=_target_unit_items("AREA"), name="Area Unit")
-    target_unit_volume: EnumProperty(items=_target_unit_items("VOLUME"), name="Volume Unit")
-    target_unit_mass: EnumProperty(items=_target_unit_items("MASS"), name="Mass Unit")
-    target_unit_time: EnumProperty(items=_target_unit_items("TIME"), name="Time Unit")
 
     if TYPE_CHECKING:
         qto_rule: str
@@ -124,8 +101,3 @@ class BIMQtoProperties(PropertyGroup):
         qto_name: str
         prop_name: str
         fallback: bool
-        target_unit_length: str
-        target_unit_area: str
-        target_unit_volume: str
-        target_unit_mass: str
-        target_unit_time: str

src/bonsai/bonsai/bim/module/qto/ui.py

@@ -17,12 +17,9 @@
 # along with Bonsai.  If not, see <http://www.gnu.org/licenses/>.
 
 import bpy
-import ifc5d.qto
 
 import bonsai.tool as tool
-from bonsai.bim.helper import prop_with_search
 from bonsai.bim.module.qto.data import QtoData
-from bonsai.bim.module.qto.prop import MEASURE_TO_TARGET_UNIT_FIELD
 
 
 class BIM_PT_qto(bpy.types.Panel):
@@ -47,14 +44,6 @@ class BIM_PT_qto(bpy.types.Panel):
         row = layout.row()
         row.prop(props, "qto_rule", text="")
         row.prop(props, "fallback", text="", icon="RADIOBUT_ON" if props.fallback else "RADIOBUT_OFF")
-
-        box = layout.box()
-        box.label(text="Target Units (optional, otherwise project default)")
-        for field_name, _special_type, label in MEASURE_TO_TARGET_UNIT_FIELD.values():
-            row = box.row(align=True)
-            row.label(text=label)
-            prop_with_search(row, props, field_name, text="")
-
         row = layout.row()
         row.operator("bim.perform_quantity_take_off")
 
@@ -77,15 +66,6 @@ class BIM_PT_qto_manual(bpy.types.Panel):
         row = layout.row()
         row.prop(props, "calculator_function", text="Function")
 
-        calculator = ifc5d.qto.calculators.get(props.calculator)
-        function = calculator.functions.get(props.calculator_function) if calculator else None
-        target_unit_field = MEASURE_TO_TARGET_UNIT_FIELD.get(function.measure) if function else None
-        if target_unit_field:
-            field_name, _special_type, label = target_unit_field
-            row = layout.row(align=True)
-            row.label(text=f"{label} Unit")
-            prop_with_search(row, props, field_name, text="")
-
         row = layout.row(align=True)
         row.prop(props, "qto_name", text="")
         row.prop(props, "prop_name", text="")

src/bonsai/bonsai/bim/module/search/data.py

@@ -132,12 +132,5 @@ class SelectSimilarData:
             if pset.endswith("Common"):
                 keys.extend([f'/.*Common/."{name}"' for name in properties.keys() if name != "id"])
             else:
-                pset_part = f'"{pset}"' if " " in pset else pset
-                keys.extend(
-                    [
-                        f'{pset_part}."{name}"' if " " in name else f"{pset_part}.{name}"
-                        for name in properties.keys()
-                        if name != "id"
-                    ]
-                )
+                keys.extend([f"{pset}.{name}" for name in properties.keys() if name != "id"])
         return [(k, k, "") for k in keys]

src/bonsai/bonsai/bim/module/spatial/workspace.py

@@ -45,7 +45,6 @@ class SpatialTool(WorkSpaceTool):
         ("bim.spatial_hotkey", {"type": "T", "value": "PRESS", "shift": True}, {"properties": [("hotkey", "S_T")]}),
         ("bim.spatial_hotkey", {"type": "G", "value": "PRESS", "shift": True}, {"properties": [("hotkey", "S_G")]}),
         ("bim.spatial_hotkey", {"type": "H", "value": "PRESS", "shift": True}, {"properties": [("hotkey", "S_H")]}),
-        ("bim.spatial_hotkey", {"type": "Q", "value": "PRESS", "shift": True}, {"properties": [("hotkey", "S_Q")]}),
     )
 
     def draw_settings(context, layout, ws_tool):
@@ -185,9 +184,3 @@ class Hotkey(bpy.types.Operator, tool.Ifc.Operator):
 
     def hotkey_S_G(self):
         bpy.ops.bim.generate_space()
-
-    def hotkey_S_Q(self):
-        # Mirrors BimTool.hotkey_S_Q so quantities can be (re)calculated without switching tools.
-        if not bpy.context.selected_objects:
-            return
-        bpy.ops.bim.perform_quantity_take_off()

src/bonsai/bonsai/bim/module/style/__init__.py

@@ -45,9 +45,7 @@ classes = (
     operator.SelectByStyle,
     operator.SelectStyleInStylesUI,
     operator.SetAssetMaterialToExternalStyle,
-    operator.SuggestShadeFromExternalStyle,
     operator.UnlinkStyle,
-    operator.TogglePreferIfcShading,
     operator.UpdateCurrentStyle,
     operator.UpdateStyleColours,
     operator.UpdateStyleTextures,

src/bonsai/bonsai/bim/module/style/operator.py

@@ -16,7 +16,6 @@
 # You should have received a copy of the GNU General Public License
 # along with Bonsai.  If not, see <http://www.gnu.org/licenses/>.
 
-import colorsys
 import os
 from pathlib import Path
 from typing import Any, Union
@@ -239,15 +238,13 @@ class UpdateCurrentStyle(bpy.types.Operator):
             if not isinstance(obj.data, (bpy.types.Mesh, bpy.types.Curve)):
                 continue
             for mat in obj.data.materials:
-                if not mat:
-                    continue
-                msprops_ = tool.Style.get_material_style_props(mat)
-                if msprops_.ifc_definition_id == 0:
-                    continue
-                if mat in updated_materials:
-                    continue
-                msprops_.active_style_type = current_style_type
-                updated_materials.add(mat)
+                if (
+                    mat
+                    and mat not in updated_materials
+                    and (msprops_ := tool.Style.get_material_style_props(mat)).ifc_definition_id != 0
+                ):
+                    msprops_.active_style_type = current_style_type
+                    updated_materials.add(mat)
         return {"FINISHED"}
 
 
@@ -460,14 +457,10 @@ class ActivateExternalStyle(bpy.types.Operator):
             self.report({"ERROR"}, f"Error loading external style for \"{material.name}\" - {db['msg']}")
             return {"CANCELLED"}
 
-        ext_mat = db["data_block"]
-        self.copy_material_attributes(ext_mat, material)
+        self.copy_material_attributes(db["data_block"], material)
         if tool.Style.get_use_nodes(material):
-            if material.get("bim_dual_branch"):
-                tool.Style.update_external_branch(material, ext_mat)
-            else:
-                tool.Style.setup_dual_branch(material, ext_mat)
-        bpy.data.materials.remove(ext_mat)
+            tool.Blender.copy_node_graph(material, db["data_block"])
+        bpy.data.materials.remove(db["data_block"])
         return {"FINISHED"}
 
     def copy_material_attributes(self, source, target):
@@ -510,218 +503,6 @@ class ActivateExternalStyle(bpy.types.Operator):
             set_prop(prop_name)
 
 
-class TogglePreferIfcShading(bpy.types.Operator):
-    bl_idname = "bim.toggle_prefer_ifc_shading"
-    bl_label = "Toggle Flat/Pretty"
-    bl_description = (
-        "Toggle between Flat (IFC-native shading) and Pretty (external .blend style) for ALL styles.\n\n"
-        "SHIFT+CLICK to apply to this style only"
-    )
-    bl_options = {"REGISTER", "UNDO"}
-    material_name: bpy.props.StringProperty(name="Material Name", default="", options={"SKIP_SAVE"})
-    single_only: bpy.props.BoolProperty(name="Single Only", default=False, options={"SKIP_SAVE"})
-
-    def invoke(self, context, event):
-        if event.shift:
-            self.single_only = True
-        return self.execute(context)
-
-    def execute(self, context):
-        wm = context.window_manager
-        space = tool.Blender.get_view3d_space()
-        is_solid = space and space.shading.type == "SOLID"
-
-        if is_solid:
-            if space.shading.color_type == "TEXTURE":
-                space.shading.color_type = "MATERIAL"
-            else:
-                meshes_needing_uv = []
-                for obj in bpy.context.scene.objects:
-                    if not isinstance(obj.data, bpy.types.Mesh):
-                        continue
-                    for slot in obj.material_slots:
-                        mat = slot.material
-                        if not mat or not tool.Blender.get_ifc_definition_id(mat):
-                            continue
-                        style_elements = tool.Style.get_style_elements(mat)
-                        if style_elements.get("IfcSurfaceStyleWithTextures") and not obj.data.uv_layers:
-                            meshes_needing_uv.append(obj.data)
-                            break
-                wm.progress_begin(0, max(len(meshes_needing_uv), 1))
-                try:
-                    for i, mesh in enumerate(meshes_needing_uv):
-                        tool.Loader.load_generated_uv_map(mesh)
-                        wm.progress_update(i)
-                finally:
-                    wm.progress_end()
-                space.shading.color_type = "TEXTURE"
-            return {"FINISHED"}
-
-        if self.single_only:
-            mat = bpy.data.materials.get(self.material_name)
-            if not mat:
-                return {"CANCELLED"}
-            msprops = tool.Style.get_material_style_props(mat)
-            msprops.prefer_ifc_shading = not msprops.prefer_ifc_shading
-        else:
-            # Default: apply to all IFC materials
-            source_mat = bpy.data.materials.get(self.material_name)
-            new_value = not tool.Style.get_material_style_props(source_mat).prefer_ifc_shading if source_mat else True
-            ifc_mats = [m for m in bpy.data.materials if tool.Blender.get_ifc_definition_id(m)]
-            for mat in ifc_mats:
-                tool.Style.get_material_style_props(mat).prefer_ifc_shading = new_value
-        return {"FINISHED"}
-
-
-class SuggestShadeFromExternalStyle(bpy.types.Operator, tool.Ifc.Operator):
-    bl_idname = "bim.suggest_shade_from_external_style"
-    bl_label = "Suggest Shade from External Style"
-    bl_description = (
-        "Generate a Shade style (Surface Colour + Transparency) from the external .blend style.\n\n"
-        "ALT+CLICK to apply to all styles with an external .blend style"
-    )
-    bl_options = {"REGISTER", "UNDO"}
-    material_name: bpy.props.StringProperty(name="Material Name", default="", options={"SKIP_SAVE"})
-    all_styles: bpy.props.BoolProperty(name="All Styles", default=False, options={"SKIP_SAVE"})
-    value_offset: bpy.props.FloatProperty(
-        name="Value",
-        description="Offset added to the colour's value (-1 = fully dark, 0 = unchanged, +1 = fully light)",
-        default=0.0,
-        min=-1.0,
-        max=1.0,
-        step=1,
-        precision=2,
-        options={"SKIP_SAVE"},
-    )
-    saturation_factor: bpy.props.FloatProperty(
-        name="Saturation",
-        description="Scale applied to the colour's saturation (0 = greyscale, 1 = unchanged, >1 = more saturated)",
-        default=1.0,
-        min=0.0,
-        max=2.0,
-        step=1,
-        precision=2,
-        options={"SKIP_SAVE"},
-    )
-
-    def invoke(self, context, event):
-        if event.alt:
-            self.all_styles = True
-        return context.window_manager.invoke_props_dialog(self)
-
-    def draw(self, context):
-        layout = self.layout
-        layout.prop(self, "value_offset", slider=True)
-        layout.prop(self, "saturation_factor", slider=True)
-        if self.all_styles:
-            layout.label(text="Will apply to all external styles", icon="INFO")
-
-    def _execute(self, context):
-        if self.all_styles:
-            candidates = [
-                (mat, tool.Style.get_style_elements(mat))
-                for mat in bpy.data.materials
-                if tool.Blender.get_ifc_definition_id(mat)
-            ]
-            candidates = [(mat, se) for mat, se in candidates if tool.Style.has_blender_external_style(se)]
-            wm = context.window_manager
-            wm.progress_begin(0, max(len(candidates), 1))
-            count = 0
-            color_cache: dict[tuple[str, str, str], tuple | None] = {}
-            try:
-                for i, (mat, style_elements) in enumerate(candidates):
-                    wm.progress_update(i)
-                    if self._apply_to_material(
-                        mat, style_elements, self.value_offset, self.saturation_factor, color_cache
-                    ):
-                        count += 1
-            finally:
-                wm.progress_end()
-            self.report({"INFO"}, f"Shade style generated for {count} style(s).")
-        else:
-            mat = bpy.data.materials.get(self.material_name)
-            if not mat:
-                return {"CANCELLED"}
-            style_elements = tool.Style.get_style_elements(mat)
-            if not tool.Style.has_blender_external_style(style_elements):
-                self.report({"ERROR"}, "No external .blend style assigned. Please assign an external style first.")
-                return {"CANCELLED"}
-            self._apply_to_material(mat, style_elements, self.value_offset, self.saturation_factor)
-        props = tool.Style.get_style_props()
-        if props.is_editing:
-            core.load_styles(tool.Style, style_type=props.style_type)
-
-    def _apply_to_material(
-        self,
-        material: bpy.types.Material,
-        style_elements: dict,
-        value_offset: float = 0.0,
-        saturation_factor: float = 1.0,
-        color_cache: "dict[tuple[str, str, str], tuple | None] | None" = None,
-    ) -> bool:
-        external_style = style_elements["IfcExternallyDefinedSurfaceStyle"]
-        style_path = Path(tool.Ifc.resolve_uri(external_style.Location))
-        data_block_type, data_block = external_style.Identification.split("/")
-
-        cache_key = (str(style_path), data_block_type, data_block)
-        if color_cache is not None and cache_key in color_cache:
-            cached = color_cache[cache_key]
-            if cached is None:
-                return False  # previously failed for this path
-            surface_colour, transparency = cached
-        else:
-            try:
-                db = tool.Blender.append_data_block(str(style_path), data_block_type, data_block)
-            except OSError as e:
-                self.report({"WARNING"}, f'Could not open blend file for "{material.name}": {e}')
-                if color_cache is not None:
-                    color_cache[cache_key] = None
-                return False
-            if not db["data_block"]:
-                self.report({"WARNING"}, f'Could not load external style for "{material.name}": {db["msg"]}')
-                if color_cache is not None:
-                    color_cache[cache_key] = None
-                return False
-
-            ext_mat = db["data_block"]
-            surface_colour, transparency = tool.Style.get_representative_material_color(ext_mat)
-            bpy.data.materials.remove(ext_mat)
-            if color_cache is not None:
-                color_cache[cache_key] = (surface_colour, transparency)
-
-        if value_offset != 0.0 or saturation_factor != 1.0:
-            h, s, v = colorsys.rgb_to_hsv(*surface_colour)
-            v = max(0.0, min(1.0, v + value_offset))
-            s = max(0.0, min(1.0, s * saturation_factor))
-            surface_colour = colorsys.hsv_to_rgb(h, s, v)
-
-        ifc_style = tool.Ifc.get_entity(material)
-        attributes: dict = {
-            "SurfaceColour": {
-                "Name": None,
-                "Red": surface_colour[0],
-                "Green": surface_colour[1],
-                "Blue": surface_colour[2],
-            },
-        }
-        if tool.Ifc.get_schema() != "IFC2X3":
-            attributes["Transparency"] = transparency
-
-        shading_style = style_elements.get("IfcSurfaceStyleShading")
-        if shading_style:
-            tool.Ifc.run("style.edit_surface_style", style=shading_style, attributes=attributes)
-        else:
-            tool.Ifc.run(
-                "style.add_surface_style",
-                style=ifc_style,
-                ifc_class="IfcSurfaceStyleShading",
-                attributes=attributes,
-            )
-        material.diffuse_color = (*surface_colour, 1.0 - transparency)
-        tool.Style.sync_flat_branch_shading(material, surface_colour, transparency)
-        return True
-
-
 class DisableEditingStyles(bpy.types.Operator):
     bl_idname = "bim.disable_editing_styles"
     bl_options = {"REGISTER", "UNDO"}

src/bonsai/bonsai/bim/module/style/prop.py

@@ -372,16 +372,6 @@ def update_shading_style(self: "BIMStyleProperties", context: bpy.types.Context)
         tool.Style.switch_shading(blender_material, self.active_style_type)
 
 
-def update_prefer_ifc_shading(self: "BIMStyleProperties", context: bpy.types.Context) -> None:
-    style_elements = tool.Style.get_style_elements(self.id_data)
-    has_external = tool.Style.has_blender_external_style(style_elements)
-    if self.prefer_ifc_shading or not has_external:
-        self.active_style_type = "Shading"
-    else:
-        self.active_style_type = "External"
-    self.id_data.update_tag()
-
-
 class BIMStyleProperties(PropertyGroup):
     ifc_definition_id: IntProperty(name="IFC Definition ID")
     active_style_type: EnumProperty(
@@ -391,19 +381,9 @@ class BIMStyleProperties(PropertyGroup):
         default="Shading",
         update=update_shading_style,
     )
-    prefer_ifc_shading: BoolProperty(
-        name="Flat / Pretty",
-        description=(
-            "Toggle between Flat (IFC-native shading) and Pretty (external .blend style). "
-            "When set to Flat, viewport switches to Material Preview or Rendered will not activate the external style."
-        ),
-        default=False,
-        update=update_prefer_ifc_shading,
-    )
     is_renaming: BoolProperty(description="Used to prevent triggering handler callback.", default=False)
 
     if TYPE_CHECKING:
         ifc_definition_id: int
         active_style_type: tool.Style.StyleType
-        prefer_ifc_shading: bool
         is_renaming: bool

src/bonsai/bonsai/bim/module/style/ui.py

@@ -110,11 +110,6 @@ class BIM_PT_styles(Panel):
                 op = row.operator("bim.update_current_style", icon="FILE_REFRESH", text="")
                 op.style_id = style.ifc_definition_id
 
-            if active_style and self.props.style_type == "IfcSurfaceStyle":
-                if material := style.blender_material:
-                    msprops = tool.Style.get_material_style_props(material)
-                    self.draw_style_status_row(material, msprops)
-
             if self.props.style_type == "IfcSurfaceStyle":
                 self.layout.label(text="Surface Style Element:")
                 col = self.layout.column(align=True)
@@ -166,66 +161,6 @@ class BIM_PT_styles(Panel):
                 edit_label = "Save Lighting Style"
             self.draw_edit_ui(edit_label)
 
-    def draw_style_status_row(self, material: bpy.types.Material, msprops) -> None:
-        space = tool.Blender.get_view3d_space()
-        box = self.layout.box()
-
-        obj = bpy.context.active_object
-
-        parts = []
-        if space:
-            shading_type = space.shading.type
-            shading_labels = {
-                "SOLID": "Solid",
-                "MATERIAL": "Material Preview",
-                "RENDERED": "Rendered",
-                "WIREFRAME": "Wireframe",
-            }
-            parts.append(f"Viewport: {shading_labels.get(shading_type, shading_type)}")
-        else:
-            parts.append("No 3D viewport")
-            shading_type = None
-
-        if obj:
-            obj_has_uv = isinstance(obj.data, bpy.types.Mesh) and bool(obj.data.uv_layers)
-            uv_label = "UV \u2713" if obj_has_uv else "UV \u2717"
-            parts.append(f"Selected Object: {obj.name} {uv_label}")
-        else:
-            parts.append("Selected Object: None")
-
-        if shading_type == "SOLID":
-            is_flat = space.shading.color_type != "TEXTURE"
-            mode_label = "Flat"
-            dep_label = "Shade"
-            if not is_flat:
-                mode_label = "Pretty"
-                dep_label = "Texture \u2192 Shade"
-        elif shading_type in ("MATERIAL", "RENDERED"):
-            is_flat = msprops.prefer_ifc_shading
-            mode_label = "Flat"
-            dep_label = "Render+Texture \u2192 Render \u2192 Shade"
-            if not is_flat:
-                mode_label = "Pretty"
-                dep_label = "External \u2192 Render+Texture \u2192 Render \u2192 Shade"
-        else:
-            is_flat = False
-            mode_label = ""
-            dep_label = ""
-
-        row1 = box.row(align=True)
-        row1.label(text=" | ".join(parts))
-
-        if mode_label:
-            row2 = box.row(align=True)
-            row2.label(text=f"Current Mode: {mode_label} \u2014 {dep_label}")
-
-        row3 = box.row(align=True)
-        row3.alignment = "RIGHT"
-        op = row3.operator("bim.suggest_shade_from_external_style", text="", icon="BRUSHES_ALL")
-        op.material_name = material.name
-        op = row3.operator("bim.toggle_prefer_ifc_shading", text="", icon="UV_SYNC_SELECT")
-        op.material_name = material.name
-
     def draw_surface_style_shading(self):
         row = self.layout.row()
         row.prop(self.props, "surface_colour")

src/bonsai/bonsai/bim/prop.py

@@ -21,6 +21,7 @@ import os
 from typing import TYPE_CHECKING, Any, Literal, Union, assert_never, get_args
 
 import bpy
+import ifcopenshell.util.unit
 from bpy.props import (
     BoolProperty,
     CollectionProperty,
@@ -33,8 +34,6 @@ from bpy.props import (
 )
 from bpy.types import PropertyGroup
 
-import ifcopenshell.util.unit
-
 import bonsai.bim
 import bonsai.bim.handler
 import bonsai.tool as tool
@@ -123,57 +122,6 @@ def get_attribute_enum_values(prop: "Attribute", context: bpy.types.Context) ->
     return items
 
 
-def get_unit_enum_items_for_special_type(
-    special_type: str, ifc_file: Union[ifcopenshell.file, None]
-) -> tool.Blender.BLENDER_ENUM_ITEMS:
-    """Items for a unit-override picker: "Default (<symbol>)" plus every candidate unit
-    matching `special_type`, filtered per-caller since candidates depend on the measure type
-    in question (unlike the globally-shared lists in `bonsai.bim.ui.EnumData`).
-    """
-    if not ifc_file or not tool.Pset.is_measurable_special_type(special_type):
-        return [(cache_string("0"), cache_string("Default"), "")]
-
-    default_symbol = tool.Pset.get_unit_symbol_for_special_type(special_type, ifc_file)
-    items: list[tuple[str, str, str]] = [
-        (cache_string("0"), cache_string(f"Default ({default_symbol})" if default_symbol else "Default"), "")
-    ]
-    for unit in tool.Pset.get_candidate_units_for_special_type(special_type, ifc_file):
-        name = getattr(unit, "Name", None) or unit.is_a()
-        symbol = ifcopenshell.util.unit.get_unit_symbol(unit)
-        label = f"{name} ({symbol})" if symbol else name
-        items.append((cache_string(str(unit.id())), cache_string(label), ""))
-    return items
-
-
-def get_attribute_unit_enum_items(prop: "Attribute", context: bpy.types.Context) -> tool.Blender.BLENDER_ENUM_ITEMS:
-    """Items for `Attribute.unit_id_enum`. Wraps `get_unit_enum_items_for_special_type` with a
-    defensive addition: real-world files sometimes carry a Unit that doesn't cleanly match our
-    candidate-matching logic (e.g. a mismatched UnitType). Always keep the attribute's own
-    current override selectable/representable, however unusual, so setting unit_id_enum to
-    match an already-seeded unit_id can never raise "enum not found".
-    """
-    ifc_file = tool.Ifc.get()
-    items = get_unit_enum_items_for_special_type(prop.special_type, ifc_file)
-
-    if prop.unit_id and prop.unit_id not in {int(i[0]) for i in items}:
-        own_unit = ifc_file.by_id(prop.unit_id)
-        name = getattr(own_unit, "Name", None) or own_unit.is_a()
-        symbol = ifcopenshell.util.unit.get_unit_symbol(own_unit)
-        label = f"{name} ({symbol})" if symbol else name
-        items.append((cache_string(str(prop.unit_id)), cache_string(label), ""))
-
-    return items
-
-
-def update_attribute_unit_id(self: "Attribute", context: bpy.types.Context) -> None:
-    new_unit_id = int(tool.Blender.get_enum_safe(self, "unit_id_enum") or "0")
-    if ifc_file := tool.Ifc.get():
-        # Must run before self.unit_id is overwritten: convert_attribute_unit needs the OLD
-        # unit_id to know what unit the current value is expressed in.
-        tool.Pset.convert_attribute_unit(self, new_unit_id, ifc_file)
-    self.unit_id = new_unit_id
-
-
 def update_schema_dir(self: "BIMProperties", context: bpy.types.Context) -> None:
     import bonsai.bim.schema
 
@@ -302,33 +250,44 @@ def set_numerical_value(self: "Attribute", value_name: str, new_value: Union[flo
     self[value_name] = new_value
 
 
+def get_length_value(self: "Attribute") -> float:
+    si_conversion = ifcopenshell.util.unit.calculate_unit_scale(tool.Ifc.get())
+    return self.float_value * si_conversion
+
+
+def set_length_value(self: "Attribute", value: float) -> None:
+    si_conversion = ifcopenshell.util.unit.calculate_unit_scale(tool.Ifc.get())
+    self.float_value = value / si_conversion
+
+
 def get_display_name(self: "Attribute") -> str:
+    DISPLAY_UNIT_TYPES = ("AREA", "VOLUME", "FORCE")
     name = self.name
-    if not self.unit_symbol:
+    if not self.special_type or self.special_type not in DISPLAY_UNIT_TYPES:
         return name
-    return f"{name}, {self.unit_symbol}"
 
+    unit_type = f"{self.special_type}UNIT"
+    project_unit = ifcopenshell.util.unit.get_project_unit(tool.Ifc.get(), unit_type)
+    if not project_unit:
+        return name
 
-def get_unit_symbol(self: "Attribute") -> str:
-    """The symbol for whatever unit the value is currently expressed in: this property's own
-    override (`unit_id`) if set, else the project default for `special_type`. Computed fresh on
-    every access (rather than cached at import time) so it stays correct immediately after the
-    unit picker changes `unit_id`, and after the project's own default units are edited.
-    """
-    if not tool.Pset.is_measurable_special_type(self.special_type):
-        return ""
-    if not (ifc_file := tool.Ifc.get()):
-        return ""
-    unit = tool.Pset.resolve_effective_unit(self.special_type, self.unit_id, ifc_file)
-    return ifcopenshell.util.unit.get_unit_symbol(unit) if unit else ""
+    unit_symbol = ifcopenshell.util.unit.get_unit_symbol(project_unit)
+    return f"{name}, {unit_symbol}"
 
 
 AttributeDataType = Literal["string", "integer", "float", "boolean", "enum", "file", "list[string]"]
-# Either "", "DATE", "DATETIME", "LOGICAL", "URI", "DURATION", or an
-# IfcUnitEnum/IfcDerivedUnitEnum value with the "UNIT" suffix stripped (e.g.
-# "LENGTH", "PRESSURE", "MODULUSOFELASTICITY") as returned by
-# tool.Pset.get_special_type_for_prop().
-AttributeSpecialType = str
+AttributeSpecialType = Literal[
+    "",
+    "DATE",
+    "DATETIME",
+    "LENGTH",
+    "AREA",
+    "VOLUME",
+    "FORCE",
+    "LOGICAL",
+    "URI",
+    "DURATION",
+]
 
 
 class Attribute(PropertyGroup):
@@ -359,6 +318,9 @@ class Attribute(PropertyGroup):
         get=lambda self: float(self.get("float_value", 0.0)),
         set=set_float_value,
     )
+    length_value: FloatProperty(
+        name="Value", description=tooltip, get=get_length_value, set=set_length_value, unit="LENGTH"
+    )
     enum_items: StringProperty(name="Value")
     """Json serialized mapping of enum items:
         Typically a dictionary of string identifiers to item names.
@@ -380,10 +342,6 @@ class Attribute(PropertyGroup):
     value_max: FloatProperty(description="This is used to validate int_value and float_value")
     value_max_constraint: BoolProperty(default=False, description="True if the numerical value has an upper bound")
     special_type: StringProperty(name="Special Value Type", default="")
-    unit_symbol: StringProperty(name="Unit Symbol", get=get_unit_symbol)
-    unit_id: IntProperty(name="Unit Override", default=0)
-    """STEP id of this property/quantity's own Unit override. 0 means "use the project default"."""
-    unit_id_enum: EnumProperty(items=get_attribute_unit_enum_items, name="Unit", update=update_attribute_unit_id)
     use_explorer_ui: BoolProperty()
     metadata: StringProperty(name="Metadata", description="For storing some additional information about the attribute")
     update: StringProperty(name="Update", description="Custom update function to be executed")
@@ -399,6 +357,7 @@ class Attribute(PropertyGroup):
         bool_value: bool
         int_value: int
         float_value: float
+        length_value: float
         enum_items: str
         enum_items_dynamic: str
         enum_descriptions: bpy.types.bpy_prop_collection_idprop[StrProperty]
@@ -414,9 +373,6 @@ class Attribute(PropertyGroup):
         value_min_constraint: bool
         value_max: float
         value_max_constraint: bool
-        unit_symbol: str
-        unit_id: int
-        unit_id_enum: str
         use_explorer_ui: bool
         metadata: str
         update: str
@@ -474,6 +430,8 @@ class Attribute(PropertyGroup):
         elif data_type == "integer":
             return "int_value"
         elif data_type == "float":
+            if display_only and self.special_type == "LENGTH":
+                return "length_value"
             return "float_value"
         elif data_type == "enum":
             return "enum_value"

src/bonsai/bonsai/bim/ui.py

@@ -41,7 +41,6 @@ import bonsai.bim.helper
 import bonsai.tool as tool
 from bonsai.bim.ifc import is_cache_locked_by_other_process
 from bonsai.bim.module.bsdd.prop import BIMBSDDProperties, BSDDProperty
-from bonsai.bim.module.material.operator import SelectByMaterial
 from bonsai.bim.module.model import prop as _model_prop
 from bonsai.bim.module.model import ui as _model_ui
 from bonsai.bim.module.pset.prop import IfcProperty
@@ -1855,21 +1854,6 @@ def draw_statusbar(self, context):
 
 def draw_custom_context_menu(self: bpy.types.Menu, context: bpy.types.Context) -> None:
     # https://blender.stackexchange.com/a/275555/86891
-
-    # Context menu for material name buttons (e.g. `bim.select_by_material`),
-    # offering a quick "Rename Material" entry instead of having to look up
-    # the material in the scene Materials panel to rename it.
-    button_operator = getattr(context, "button_operator", None)
-    if button_operator is not None and button_operator.bl_rna.identifier == SelectByMaterial.bl_rna.identifier:
-        ifc_file = tool.Ifc.get()
-        material = ifc_file.by_id(button_operator.material) if ifc_file else None
-        if material is not None and material.is_a("IfcMaterial"):
-            assert self.layout
-            self.layout.separator()
-            op = self.layout.operator("bim.rename_material", text="Rename Material", icon="GREASEPENCIL")
-            op.material = material.id()
-        return
-
     if (
         not hasattr(context, "button_pointer")
         or not hasattr(context, "button_prop")

src/bonsai/bonsai/core/material.py

@@ -107,15 +107,6 @@ def disable_editing_material(material_tool: type[tool.Material]) -> None:
     material_tool.disable_editing_material()
 
 
-def rename_material(
-    ifc: type[tool.Ifc], material_tool: type[tool.Material], material: ifcopenshell.entity_instance, name: str
-) -> None:
-    ifc.run("material.edit_material", material=material, attributes={"Name": name})
-    if material_tool.is_editing_materials():
-        material_tool.import_material_definitions(material_tool.get_active_material_type())
-    material_tool.refresh()
-
-
 def assign_material(
     ifc: type[tool.Ifc],
     material_tool: type[tool.Material],

src/bonsai/bonsai/core/spatial.py

@@ -222,6 +222,7 @@ def generate_space(
 
     if element and element.is_a("IfcSpace"):
         spatial.set_space_representation_from_polygon(active_obj, element, space_polygon, h, polygon_is_si=True)
+        spatial.translate_obj_to_z_location(active_obj, z)
     else:
         if relating_type:
             name = model.generate_occurrence_name(relating_type, "IfcSpace")

src/bonsai/bonsai/core/tool.py

@@ -667,7 +667,6 @@ class Material:
     def is_editing_materials(cls): pass
     def is_material_used_in_sets(cls, material): pass
     def load_material_attributes(cls, material): pass
-    def refresh(cls): pass
     def replace_material_with_material_profile(cls, element): pass
     def update_elements_using_material(cls, material): pass
 

src/bonsai/bonsai/tool/blender.py

@@ -803,40 +803,6 @@ class Blender(bonsai.core.tool.Blender):
         # restore shader editor settings
         shader_editor.pin = previous_pin_setting
 
-    @classmethod
-    def copy_node_graph_additive(
-        cls, material_to: bpy.types.Material, material_from: bpy.types.Material
-    ) -> bpy.types.ShaderNodeOutputMaterial | None:
-        """Paste nodes from material_from alongside the existing nodes in material_to.
-
-        Unlike copy_node_graph this does NOT clear the existing node tree first.
-        Returns the OUTPUT_MATERIAL node that was added from material_from, or None.
-        """
-        temp_override = cls.get_shader_editor_context()
-        shader_editor = temp_override["space"]
-
-        before_names = {n.name for n in material_to.node_tree.nodes}
-
-        previous_pin_setting = shader_editor.pin
-        shader_editor.pin = True
-        shader_editor.node_tree = material_from.node_tree
-
-        for node in material_from.node_tree.nodes:
-            node.select = True
-        with bpy.context.temp_override(**temp_override):
-            bpy.ops.node.clipboard_copy()
-
-        shader_editor.node_tree = material_to.node_tree
-        with bpy.context.temp_override(**temp_override):
-            bpy.ops.node.clipboard_paste(offset=(0, 0))
-
-        shader_editor.pin = previous_pin_setting
-
-        for node in material_to.node_tree.nodes:
-            if node.name not in before_names and node.type == "OUTPUT_MATERIAL":
-                return node
-        return None
-
     @classmethod
     def get_material_node(
         cls, blender_material: bpy.types.Material, node_type: str, kwargs: Optional[dict] = {}

src/bonsai/bonsai/tool/bsdd.py

@@ -138,6 +138,7 @@ class Bsdd(bonsai.core.tool.Bsdd):
     def get_dictionaries(cls) -> list[bsdd.DictionaryContractV1]:
         prefs = tool.Blender.get_addon_preferences()
         baseurl = getattr(prefs, "bsdd_baseurl", "https://api.bsdd.buildingsmart.org/api/")
+        cls.client = bsdd.Client()
         if hasattr(cls.client, "baseurl"):
             cls.client.baseurl = baseurl
         response = cls.client.get_dictionary(include_test_dictionaries=prefs.bsdd_load_test_dictionaries)

src/bonsai/bonsai/tool/clip_box.py

@@ -1270,17 +1270,6 @@ class ClipBox:
     def on_depsgraph_update_caps(cls, scene, depsgraph) -> None:
         """Depsgraph entry-point — guard, then delegate to the
         modal-aware debounce in :meth:`_handle_cap_tick`."""
-        # Same file-load danger window as on_depsgraph_update: a real
-        # depsgraph tick during load (between load_pre and the new file's
-        # first paint) must not re-arm a cap-rebuild timer. _on_load_pre
-        # already cancels any in-flight timer via _cancel_pending_cap_rebuild;
-        # without this gate, a depsgraph_update_post event firing later in
-        # the same load (Blender fires these while building the new file's
-        # scene) would immediately reschedule one via _handle_cap_tick,
-        # undoing that cancellation and re-arming against regions whose GPU
-        # state is not yet wired.
-        if cls._file_loading:
-            return
         if getattr(bpy.context, "screen", None) is None:
             return
         if cls._active_scene_props(scene) is None:

src/bonsai/bonsai/tool/material.py

@@ -146,12 +146,6 @@ class Material(bonsai.core.tool.Material):
 
         MaterialsData.data["material_styles_data"] = MaterialsData.material_styles_data()
 
-    @classmethod
-    def refresh(cls) -> None:
-        from bonsai.bim.module.material.data import refresh as refresh_material_data
-
-        refresh_material_data()
-
     @classmethod
     def is_editing_materials(cls) -> bool:
         props = tool.Material.get_material_props()

src/bonsai/bonsai/tool/nest.py

@@ -47,15 +47,7 @@ class Nest(bonsai.core.tool.Nest):
             return False
         if relating_object == related_object:
             return False
-        # IfcRelNests.RelatingObject/RelatedObjects are typed as the general
-        # IfcObjectDefinition, so nesting is schema-legal both between element
-        # occurrences (the common case, e.g. a faucet nested into a sink) and
-        # between element types (e.g. an assembly type nesting its component
-        # types). Mixing an occurrence with a type isn't a real modeling
-        # pattern, so only allow same-kind pairs. See #2283.
-        is_compatible_class = (relating_object.is_a("IfcElement") and related_object.is_a("IfcElement")) or (
-            relating_object.is_a("IfcTypeProduct") and related_object.is_a("IfcTypeProduct")
-        )
+        is_compatible_class = relating_object.is_a("IfcElement") and related_object.is_a("IfcElement")
         if not is_compatible_class:
             return False
         # Prevent cyclic references: walk up the full hierarchy from the

src/bonsai/bonsai/tool/polyline.py

@@ -168,12 +168,12 @@ class Polyline(bonsai.core.tool.Polyline):
         distance = (mouse_vector - last_point).length
         if distance < 0:
             return
-        angle, orientation_angle = None, None
-        angle_round_threshold = 1000  # Avoids rounding when distance is too big
+        angle, orientation_angle, angle_round_threshold = None, None, None
         if distance > 0:
             angle = tool.Cad.angle_3_vectors(
                 second_to_last_point, last_point, mouse_vector, new_angle=None, degrees=True
             )
+            angle_round_threshold = 1000  # Avoids rounding when distance is too big
 
             # Round angle to the nearest 0.05
             angle = round(angle / 0.05) * 0.05 if distance < angle_round_threshold else angle
@@ -189,7 +189,7 @@ class Polyline(bonsai.core.tool.Polyline):
             angle = 0
             orientation_angle = 0
         if input_ui:
-            assert angle is not None and orientation_angle is not None
+            assert angle is not None and orientation_angle is not None and angle_round_threshold is not None
             if should_round:
                 angle_snap = tool.Snap.get_angle_snap_value(context)
                 angle = angle_snap * round(angle / angle_snap) if distance < angle_round_threshold else angle

src/bonsai/bonsai/tool/project.py

@@ -391,14 +391,29 @@ class Project(bonsai.core.tool.Project):
     def get_parent_library(
         cls, project_library: ifcopenshell.entity_instance
     ) -> Union[ifcopenshell.entity_instance, None]:
-        """Return the IfcContext that declares or nests ``project_library``, or ``None``
-        if neither relationship is present."""
+        """Return the IfcContext that declares or nests ``project_library``.
+
+        Returns ``None`` when ``project_library`` is itself the root of a
+        library-only file (no IfcRelNests, no IfcRelDeclares).
+        """
         if nests := project_library.Nests:
             return nests[0].RelatingObject
         if has_context := project_library.HasContext:
             return has_context[0].RelatingContext
         return None
 
+    @classmethod
+    def get_root_context(cls, ifc_file: ifcopenshell.file) -> ifcopenshell.entity_instance:
+        """Return the file's root IfcContext.
+
+        Prefers IfcProject if present, otherwise falls back to IfcProjectLibrary —
+        library-only files are valid per IFC4+ and contain no IfcProject. Caller is
+        responsible for the IFC2X3 guard; IfcContext does not exist in that schema.
+        """
+        if projects := ifc_file.by_type("IfcProject"):
+            return projects[0]
+        return ifc_file.by_type("IfcProjectLibrary")[0]
+
     @classmethod
     def get_project_hierarchy(cls, ifc_file: ifcopenshell.file) -> HiearchyDict:
         """Get project hierarchy in the following form:

src/bonsai/bonsai/tool/pset.py

@@ -26,7 +26,6 @@ import ifcopenshell
 import ifcopenshell.api.pset
 import ifcopenshell.util.attribute
 import ifcopenshell.util.element
-import ifcopenshell.util.unit
 
 import bonsai.bim.helper
 import bonsai.bim.schema
@@ -169,144 +168,42 @@ class Pset(bonsai.core.tool.Pset):
             pset_id=0, pset_name=cls.get_pset_name(obj, obj_type), pset_type="PSET", obj=obj, obj_type=obj_type
         )
 
-    # Templates for quantities can specify their kind via TemplateType (e.g.
-    # "Q_LENGTH") instead of PrimaryMeasureType. IfcQuantityCount has no
-    # associated measure/unit, so it is intentionally absent here.
-    QUANTITY_TEMPLATE_TYPE_TO_SPECIAL_TYPE = {
-        "Q_LENGTH": "LENGTH",
-        "Q_AREA": "AREA",
-        "Q_VOLUME": "VOLUME",
-        "Q_WEIGHT": "MASS",
-        "Q_TIME": "TIME",
-    }
-
     @classmethod
-    def get_special_type_for_measure_class(cls, measure_class: str) -> str:
-        """Get the ``special_type`` (an IfcUnitEnum value with "UNIT" stripped) for an IFC measure class.
-
-        :param measure_class: An IFC measure class name, e.g. "IfcLengthMeasure".
-        :return: E.g. "LENGTH", or "" if the class has no associated unit type.
-        """
-        if not measure_class.endswith("Measure"):
-            return ""
-        unit_type = ifcopenshell.util.unit.get_measure_unit_type(measure_class)
-        return unit_type[: -len("UNIT")] if unit_type.endswith("UNIT") else ""
-
-    @classmethod
-    def get_special_type_for_unit(cls, unit: ifcopenshell.entity_instance) -> str:
-        """Get the ``special_type`` (an IfcUnitEnum value with "UNIT" stripped) directly from
-        a Unit entity, for properties whose NominalValue is a generic numeric type (e.g.
-        IfcReal) rather than a proper measure class, but which still carry a real Unit.
-        """
-        unit_type = getattr(unit, "UnitType", None)
-        if unit_type and unit_type != "USERDEFINED":
-            return unit_type[: -len("UNIT")] if unit_type.endswith("UNIT") else ""
-        dimension_type = ifcopenshell.util.unit.identify_unit_dimensions(unit)
-        return dimension_type[: -len("UNIT")] if dimension_type else ""
-
-    @classmethod
-    def get_special_type_for_prop(cls, prop_or_prop_template: ifcopenshell.entity_instance) -> str:
-        """Classify a property/quantity/template by its measure type.
-
-        :return: An IfcUnitEnum value with the "UNIT" suffix stripped (e.g.
-            "LENGTH", "PRESSURE"), "URI" for IfcURIReference, or "" if the
-            value has no associated unit type.
-        """
+    def get_special_type_for_prop(
+        cls, prop_or_prop_template: ifcopenshell.entity_instance
+    ) -> Literal["LENGTH"] | Literal["AREA"] | Literal["VOLUME"] | Literal["URI"] | Literal[""]:
+        special_type = ""
         if prop_or_prop_template.is_a("IfcPropertyTemplate"):
             primary_measure_type = prop_or_prop_template.PrimaryMeasureType
-            if primary_measure_type == "IfcURIReference":
-                return "URI"
-            if primary_measure_type:
-                return cls.get_special_type_for_measure_class(primary_measure_type)
-            return cls.QUANTITY_TEMPLATE_TYPE_TO_SPECIAL_TYPE.get(prop_or_prop_template.TemplateType, "")
-        elif prop_or_prop_template.is_a("IfcPropertySingleValue"):
-            value = prop_or_prop_template.NominalValue
-            if value is not None:
-                special_type = cls.get_special_type_for_measure_class(value.is_a())
-                if special_type:
-                    return special_type
-                # Some property sets declare a generic numeric type (e.g. IfcReal) rather
-                # than a proper measure class, relying on an explicit Unit attribute alone to
-                # convey the dimension. Still measurable -- derive special_type from the Unit
-                # itself rather than (fruitlessly) from NominalValue's declared type.
-                if value.is_a() in ("IfcReal", "IfcInteger"):
-                    if unit := getattr(prop_or_prop_template, "Unit", None):
-                        return cls.get_special_type_for_unit(unit)
-        elif prop_or_prop_template.is_a("IfcPhysicalSimpleQuantity"):
-            entity = prop_or_prop_template.wrapped_data.declaration().as_entity()
-            measure_class = entity.attribute_by_index(3).type_of_attribute().declared_type().name()
-            return cls.get_special_type_for_measure_class(measure_class)
-        return ""
-
-    @classmethod
-    def get_unit_symbol_for_special_type(cls, special_type: str, ifc_file: ifcopenshell.file) -> str:
-        """Get the project's default unit symbol for a `special_type` (see `get_special_type_for_prop`).
-
-        Used where there's no property instance to check for a `Unit` override
-        (e.g. a template, or a native IFC entity attribute, neither of which
-        can carry one).
-        """
-        if not special_type or special_type == "URI":
-            return ""
-        unit = ifcopenshell.util.unit.get_project_unit(ifc_file, f"{special_type}UNIT")
-        return ifcopenshell.util.unit.get_unit_symbol(unit) if unit else ""
-
-    @classmethod
-    def get_unit_symbol_for_prop(cls, prop: ifcopenshell.entity_instance, ifc_file: ifcopenshell.file) -> str:
-        """Get the unit symbol for an existing property/quantity, respecting its own `Unit` override.
-
-        Gated on the property being classified as measurable (see `get_special_type_for_prop`,
-        which already accounts for a Unit attached to a generic numeric value) -- this only
-        excludes a Unit attached to a property whose value has no numeric/measure semantics at
-        all (e.g. text), where a stray Unit shouldn't be surfaced as a resolved unit.
-        """
-        if not cls.is_measurable_special_type(cls.get_special_type_for_prop(prop)):
-            return ""
-        unit = ifcopenshell.util.unit.get_property_unit(prop, ifc_file)
-        return ifcopenshell.util.unit.get_unit_symbol(unit) if unit else ""
-
-    # special_type values that don't denote a real unit-bearing measure (see get_special_type_for_prop).
-    NON_MEASURABLE_SPECIAL_TYPES = frozenset({"", "DATE", "DATETIME", "LOGICAL", "URI", "DURATION"})
-
-    @classmethod
-    def is_measurable_special_type(cls, special_type: str) -> bool:
-        """True if `special_type` (see `get_special_type_for_prop`) denotes a real unit-bearing measure."""
-        return special_type not in cls.NON_MEASURABLE_SPECIAL_TYPES
-
-    @classmethod
-    def get_candidate_units_for_special_type(
-        cls, special_type: str, ifc_file: ifcopenshell.file
-    ) -> list[ifcopenshell.entity_instance]:
-        """All units in the file usable as an override for a `special_type` (see `get_special_type_for_prop`)."""
-        if not cls.is_measurable_special_type(special_type):
-            return []
-        return ifcopenshell.util.unit.get_candidate_units(ifc_file, f"{special_type}UNIT")
-
-    @classmethod
-    def resolve_effective_unit(
-        cls, special_type: str, unit_id: int, ifc_file: ifcopenshell.file
-    ) -> Union[ifcopenshell.entity_instance, None]:
-        """The unit a value is currently expressed in: its own override (`unit_id`, a STEP id,
-        0 meaning "no override"), or the project default for `special_type` otherwise."""
-        if unit_id:
-            return ifc_file.by_id(unit_id)
-        return ifcopenshell.util.unit.get_project_unit(ifc_file, f"{special_type}UNIT")
-
-    @classmethod
-    def convert_attribute_unit(cls, metadata: "Attribute", new_unit_id: int, ifc_file: ifcopenshell.file) -> None:
-        """Rescale `metadata.float_value` in place so its physical quantity is preserved when
-        switching from its current effective unit to the unit named by `new_unit_id` (0 = project
-        default). No-op for non-measurable attributes or when old and new resolve to the same unit.
-        """
-        if not cls.is_measurable_special_type(metadata.special_type):
-            return
-        old_unit = cls.resolve_effective_unit(metadata.special_type, metadata.unit_id, ifc_file)
-        new_unit = cls.resolve_effective_unit(metadata.special_type, new_unit_id, ifc_file)
-        if old_unit is None or new_unit is None or old_unit == new_unit:
-            return
-        old_scale = ifcopenshell.util.unit.get_unit_scale(old_unit)
-        new_scale = ifcopenshell.util.unit.get_unit_scale(new_unit)
-        metadata.float_value = metadata.float_value * old_scale / new_scale
+            template_type = prop_or_prop_template.TemplateType
+            if primary_measure_type in ("IfcPositiveLengthMeasure", "IfcLengthMeasure") or template_type == "Q_LENGTH":
+                special_type = "LENGTH"
+            elif primary_measure_type == "IfcAreaMeasure" or template_type == "Q_AREA":
+                special_type = "AREA"
+            elif primary_measure_type == "IfcVolumeMeasure" or template_type == "Q_VOLUME":
+                special_type = "VOLUME"
+            elif primary_measure_type == "IfcURIReference":
+                special_type = "URI"
+        else:
+            if prop_or_prop_template.is_a("IfcPropertySingleValue"):
+                value = prop_or_prop_template.NominalValue
+                if value is not None:
+                    value_type = value.is_a()
+                    if value_type in ("IfcLengthMeasure", "IfcPositiveLengthMeasure"):
+                        special_type = "LENGTH"
+                    elif value_type == "IfcAreaMeasure":
+                        special_type = "AREA"
+                    elif value_type == "IfcVolumeMeasure":
+                        special_type = "VOLUME"
+            elif prop_or_prop_template.is_a("IfcPhysicalSimpleQuantity"):
+                prop_class = prop_or_prop_template.is_a()
+                if prop_class == "IfcQuantityArea":
+                    special_type = "AREA"
+                elif prop_class == "IfcQuantityVolume":
+                    special_type = "VOLUME"
+                elif prop_class == "IfcQuantityLength":
+                    special_type = "LENGTH"
+        return special_type
 
     @classmethod
     def import_pset_from_existing(
@@ -386,15 +283,6 @@ class Pset(bonsai.core.tool.Pset):
                 metadata.is_null = value is None
                 metadata.is_optional = True
                 metadata.special_type = cls.get_special_type_for_prop(prop)
-                # The prop's OWN Unit override only -- metadata.unit_symbol is computed fresh from
-                # special_type/unit_id on every access (see Attribute.get_unit_symbol), so it
-                # already accounts for the project-default fallback once unit_id is set below.
-                # Some real-world files (e.g. certain exporters) set Unit on properties that
-                # aren't actually measures -- ignore it there, since we only ever treat Unit as
-                # meaningful for measurable special_types (matching the UI picker's own gating).
-                own_unit = getattr(prop, "Unit", None) if cls.is_measurable_special_type(metadata.special_type) else None
-                metadata.unit_id = own_unit.id() if own_unit else 0
-                metadata.unit_id_enum = str(metadata.unit_id)
                 metadata.set_value(metadata.get_value_default() if metadata.is_null else value)
                 process_prop_description(metadata)
 
@@ -521,8 +409,6 @@ class Pset(bonsai.core.tool.Pset):
                 cls.import_single_value_from_template(pset_template, prop_template, simplified_data, props)
 
             elif prop_template.TemplateType.startswith("Q_"):
-                if prop_data:
-                    continue  # Existing quantity will be added later by import_pset_from_existing.
                 cls.import_single_value_from_template(pset_template, prop_template, simplified_data, props)
 
             elif prop_template.TemplateType == "P_ENUMERATEDVALUE":

src/bonsai/bonsai/tool/qto.py

@@ -176,19 +176,6 @@ class Qto(bonsai.core.tool.Qto):
         is_ifc4x3 = ifc_file.schema == "IFC4X3"
         return {rule_id: rule for rule_id, rule in ifc5d.qto.rules.items() if rule_id.startswith("IFC4X3") == is_ifc4x3}
 
-    @classmethod
-    def get_target_units(cls) -> dict[str, ifcopenshell.entity_instance]:
-        from bonsai.bim.module.qto.prop import MEASURE_TO_TARGET_UNIT_FIELD
-
-        props = cls.get_qto_props()
-        ifc_file = tool.Ifc.get()
-        target_units: dict[str, ifcopenshell.entity_instance] = {}
-        for measure_class, (field_name, _special_type, _label) in MEASURE_TO_TARGET_UNIT_FIELD.items():
-            unit_id = int(tool.Blender.get_enum_safe(props, field_name) or "0")
-            if unit_id:
-                target_units[measure_class] = ifc_file.by_id(unit_id)
-        return target_units
-
     @classmethod
     def get_not_quantified_elements_message(cls, not_quantified_elements: set[ifcopenshell.entity_instance]) -> str:
         not_quantified_message = ""

src/bonsai/bonsai/tool/style.py

@@ -597,146 +597,6 @@ class Style(bonsai.core.tool.Style):
         external_style = style_elements.get("IfcExternallyDefinedSurfaceStyle", None)
         return bool(external_style and external_style.Location and external_style.Location.endswith(".blend"))
 
-    @classmethod
-    def _color_from_principled(cls, node: bpy.types.Node) -> tuple[tuple[float, float, float], float]:
-        color = cls._resolve_color_socket(node.inputs["Base Color"])
-        alpha_socket = node.inputs["Alpha"]
-        alpha_source = cls._upstream_color_source(alpha_socket)
-        if alpha_source and alpha_source[0] == "IMAGE":
-            pixels = alpha_source[1].pixels[:]
-            n = len(pixels) // 4
-            step = max(1, n // 4096)
-            a_sum = sum(pixels[i * 4 + 3] for i in range(0, n, step))
-            count = len(range(0, n, step)) or 1
-            transparency = 1.0 - (a_sum / count)
-        else:
-            transparency = 1.0 - alpha_socket.default_value
-        return color, transparency
-
-    @classmethod
-    def _color_from_shader_socket(
-        cls, socket: bpy.types.NodeSocket, seen: set[str] | None = None
-    ) -> tuple[tuple[float, float, float], float] | None:
-        if seen is None:
-            seen = set()
-        for link in socket.links:
-            node = link.from_node
-            if node.name in seen:
-                continue
-            seen.add(node.name)
-            if node.type == "BSDF_PRINCIPLED":
-                return cls._color_from_principled(node)
-            if node.type in ("BSDF_DIFFUSE", "DIFFUSE_BSDF"):
-                return cls._resolve_color_socket(node.inputs["Color"]), 0.0
-            if node.type == "BSDF_GLASS":
-                return cls._resolve_color_socket(node.inputs["Color"]), 0.0
-            if node.type in ("MIX_SHADER", "ADD_SHADER"):
-                for inp in node.inputs:
-                    if inp.type == "SHADER" and inp.is_linked:
-                        result = cls._color_from_shader_socket(inp, seen)
-                        if result:
-                            return result
-        return None
-
-    @classmethod
-    def get_representative_material_color(
-        cls, material: bpy.types.Material
-    ) -> tuple[tuple[float, float, float], float]:
-        if material.node_tree:
-            nodes = material.node_tree.nodes
-            output_node = next((n for n in nodes if n.type == "OUTPUT_MATERIAL" and n.is_active_output), None) or next(
-                (n for n in nodes if n.type == "OUTPUT_MATERIAL"), None
-            )
-            if output_node:
-                result = cls._color_from_shader_socket(output_node.inputs["Surface"])
-                if result:
-                    return result
-            # Fallback: scan all shader nodes if no output node or graph traversal found nothing
-            for node in nodes:
-                if node.type == "BSDF_PRINCIPLED":
-                    return cls._color_from_principled(node)
-            for node in nodes:
-                if node.type in ("BSDF_DIFFUSE", "DIFFUSE_BSDF"):
-                    return cls._resolve_color_socket(node.inputs["Color"]), 0.0
-            for node in nodes:
-                if node.type == "BSDF_GLASS":
-                    return cls._resolve_color_socket(node.inputs["Color"]), 0.0
-        color = tuple(material.diffuse_color[:3])
-        transparency = 1.0 - material.diffuse_color[3]
-        return color, transparency
-
-    @classmethod
-    def _collect_upstream_sources(cls, socket: bpy.types.NodeSocket, seen: set[str]) -> list[tuple[str, object]]:
-        """Recursively collect all upstream colour/image sources reachable from *socket*."""
-        results = []
-        for link in socket.links:
-            node = link.from_node
-            if node.name in seen:
-                continue
-            seen.add(node.name)
-            if node.type == "TEX_IMAGE":
-                results.append(("IMAGE", node.image))
-            elif node.type == "VALTORGB":
-                results.append(("COLORRAMP", node))
-            else:
-                for inp in node.inputs:
-                    if inp.is_linked:
-                        results.extend(cls._collect_upstream_sources(inp, seen))
-        return results
-
-    @classmethod
-    def _upstream_color_source(
-        cls, socket: bpy.types.NodeSocket, seen: set[str] | None = None
-    ) -> tuple[str, object] | None:
-        sources = cls._collect_upstream_sources(socket, set() if seen is None else seen)
-        # Prefer a concrete image texture over a colour ramp (which may be greyscale/procedural).
-        for s in sources:
-            if s[0] == "IMAGE":
-                return s
-        for s in sources:
-            if s[0] == "COLORRAMP":
-                return s
-        return None
-
-    @classmethod
-    def _resolve_color_socket(cls, socket: bpy.types.NodeSocket) -> tuple[float, float, float]:
-        source = cls._upstream_color_source(socket)
-        if source is None:
-            return tuple(socket.default_value[:3])
-        kind, obj = source
-        if kind == "IMAGE":
-            return cls._average_image_color(obj)
-        if kind == "COLORRAMP":
-            return cls._average_colorramp_color(obj)
-        return tuple(socket.default_value[:3])
-
-    @staticmethod
-    def _average_image_color(image: bpy.types.Image) -> tuple[float, float, float]:
-        pixels = image.pixels[:]
-        n = len(pixels) // 4
-        if n == 0:
-            return (0.5, 0.5, 0.5)
-        step = max(1, n // 4096)
-        r_sum = g_sum = b_sum = 0.0
-        count = 0
-        for i in range(0, n, step):
-            base = i * 4
-            r_sum += pixels[base]
-            g_sum += pixels[base + 1]
-            b_sum += pixels[base + 2]
-            count += 1
-        return (r_sum / count, g_sum / count, b_sum / count)
-
-    @staticmethod
-    def _average_colorramp_color(node: bpy.types.Node) -> tuple[float, float, float]:
-        elements = node.color_ramp.elements
-        if not elements:
-            return (0.5, 0.5, 0.5)
-        r = sum(e.color[0] for e in elements) / len(elements)
-        g = sum(e.color[1] for e in elements) / len(elements)
-        b = sum(e.color[2] for e in elements) / len(elements)
-        return (r, g, b)
-
     @classmethod
     def is_editing_styles(cls) -> bool:
         props = cls.get_style_props()
@@ -814,179 +674,9 @@ class Style(bonsai.core.tool.Style):
         props = cls.get_material_style_props(blender_material)
         props.active_style_type = props.active_style_type
 
-    @classmethod
-    def get_branch_outputs(
-        cls, material: bpy.types.Material
-    ) -> tuple[bpy.types.ShaderNode | None, bpy.types.ShaderNode | None]:
-        """Return (external_output_node, flat_output_node), or (None, None) if not dual-branch."""
-        if not material.node_tree:
-            return None, None
-        ext = material.node_tree.nodes.get("BIM_Output_External")
-        fast = material.node_tree.nodes.get("BIM_Output_Flat")
-        return ext, fast
-
-    @classmethod
-    def _remove_external_branch(cls, material: bpy.types.Material) -> None:
-        """Remove all nodes reachable from BIM_Output_External (walks links backwards)."""
-        if not material.node_tree:
-            return
-        nodes = material.node_tree.nodes
-        output = nodes.get("BIM_Output_External")
-        if not output:
-            return
-        to_remove: set[str] = set()
-        stack = [output]
-        while stack:
-            node = stack.pop()
-            if node.name in to_remove:
-                continue
-            to_remove.add(node.name)
-            for inp in node.inputs:
-                for link in inp.links:
-                    stack.append(link.from_node)
-        for name in list(to_remove):
-            n = nodes.get(name)
-            if n:
-                nodes.remove(n)
-
-    @classmethod
-    def _build_flat_branch_nodes(cls, material: bpy.types.Material) -> bpy.types.ShaderNode:
-        """Add a Principled BSDF flat-branch to material's existing node tree.
-
-        Reads IfcSurfaceStyleRendering or IfcSurfaceStyleShading from the linked IFC entity.
-        Defaults to a white BSDF when no IFC shading data is available.
-        Returns the new Material Output node (named BIM_Output_Flat, is_active_output=False).
-        """
-        from mathutils import Vector
-
-        style_elements = cls.get_style_elements(material)
-        nodes = material.node_tree.nodes
-        links = material.node_tree.links
-
-        bsdf = nodes.new("ShaderNodeBsdfPrincipled")
-        bsdf.location = Vector((10, -600))
-        output = nodes.new("ShaderNodeOutputMaterial")
-        output.name = "BIM_Output_Flat"
-        output.location = Vector((300, -600))
-        output.is_active_output = False
-        links.new(bsdf.outputs["BSDF"], output.inputs["Surface"])
-
-        rendering_style = None
-        shading_only = None
-        for surface_style in style_elements.values():
-            if surface_style.is_a() == "IfcSurfaceStyleShading":
-                shading_only = surface_style
-            elif surface_style.is_a("IfcSurfaceStyleRendering"):
-                rendering_style = surface_style
-                shading_only = None
-
-        if rendering_style:
-            d = tool.Loader.surface_style_to_dict(rendering_style)
-            if d.get("DiffuseColour"):
-                ctype, cval = d["DiffuseColour"]
-                if ctype == "IfcColourRgb":
-                    bsdf.inputs["Base Color"].default_value = cval + (1,)
-                    solid_color = cval
-                else:
-                    cval = tuple(v * cval for v in d["SurfaceColour"])
-                    bsdf.inputs["Base Color"].default_value = cval + (1,)
-                    solid_color = cval
-            else:
-                r, g, b = d["SurfaceColour"]
-                bsdf.inputs["Base Color"].default_value = (r, g, b, 1.0)
-                solid_color = (r, g, b)
-            if d.get("SpecularColour"):
-                ctype, cval = d["SpecularColour"]
-                if ctype == "IfcNormalisedRatioMeasure":
-                    bsdf.inputs["Metallic"].default_value = cval
-            if d.get("SpecularHighlight"):
-                bsdf.inputs["Roughness"].default_value = d["SpecularHighlight"]
-            transparency = d.get("Transparency") or 0.0
-            bsdf.inputs["Alpha"].default_value = 1 - transparency
-            if transparency > 0:
-                material.blend_method = "BLEND"
-            material.diffuse_color = solid_color + (1.0 - transparency,)
-        elif shading_only:
-            d = tool.Loader.surface_style_to_dict(shading_only)
-            r, g, b = d["SurfaceColour"]
-            alpha = 1 - (d.get("Transparency") or 0.0)
-            bsdf.inputs["Base Color"].default_value = (r, g, b, 1.0)
-            bsdf.inputs["Alpha"].default_value = alpha
-            if alpha < 1.0:
-                material.blend_method = "BLEND"
-            material.diffuse_color = (r, g, b, alpha)
-        # else: leave default white Principled BSDF
-        return output
-
-    @classmethod
-    def setup_dual_branch(cls, material: bpy.types.Material, ext_material: bpy.types.Material) -> bool:
-        """Build a dual-branch node tree: flat branch from IFC data + external branch from ext_material.
-
-        Clears any existing nodes and builds both branches from scratch.
-        External branch output (BIM_Output_External) is set active — Pretty mode.
-        Flat branch output (BIM_Output_Flat) is inactive — Flat mode.
-        Returns True on success; False if no shader editor is available (falls back to single-branch).
-        """
-        cls.set_use_nodes(material, True)
-        for n in material.node_tree.nodes[:]:
-            material.node_tree.nodes.remove(n)
-
-        cls._build_flat_branch_nodes(material)
-
-        ext_output = tool.Blender.copy_node_graph_additive(material, ext_material)
-        if not ext_output:
-            # No shader editor available: fall back to single-branch
-            tool.Blender.copy_node_graph(material, ext_material)
-            return False
-
-        ext_output.name = "BIM_Output_External"
-        ext_output.is_active_output = True
-        material["bim_dual_branch"] = True
-        return True
-
-    @classmethod
-    def update_external_branch(cls, material: bpy.types.Material, ext_material: bpy.types.Material) -> None:
-        """Replace the external-branch nodes of an already dual-branch material."""
-        cls._remove_external_branch(material)
-        ext_output = tool.Blender.copy_node_graph_additive(material, ext_material)
-        if ext_output:
-            ext_output.name = "BIM_Output_External"
-            ext_output.is_active_output = True
-            fast = material.node_tree.nodes.get("BIM_Output_Flat")
-            if fast:
-                fast.is_active_output = False
-
-    @classmethod
-    def sync_flat_branch_shading(
-        cls, material: bpy.types.Material, surface_colour: tuple[float, float, float], transparency: float
-    ) -> None:
-        """Update the flat-branch Principled BSDF with new shading values.
-
-        Call this after creating or editing IfcSurfaceStyleShading so the flat branch
-        stays in sync without requiring a full setup_dual_branch rebuild.
-        """
-        if not material.node_tree:
-            return
-        fast_output = material.node_tree.nodes.get("BIM_Output_Flat")
-        if not fast_output:
-            return
-        for link in fast_output.inputs["Surface"].links:
-            if link.from_node.type == "BSDF_PRINCIPLED":
-                bsdf = link.from_node
-                r, g, b = surface_colour
-                bsdf.inputs["Base Color"].default_value = (r, g, b, 1.0)
-                bsdf.inputs["Alpha"].default_value = 1.0 - transparency
-                break
-
     @classmethod
     def switch_shading(cls, blender_material: bpy.types.Material, style_type: StyleType) -> None:
         if style_type == "External":
-            ext, fast = cls.get_branch_outputs(blender_material)
-            if ext and fast:
-                ext.is_active_output = True
-                fast.is_active_output = False
-                blender_material.update_tag()
-                return
             try:
                 bpy.ops.bim.activate_external_style(material_name=blender_material.name)
             except RuntimeError as error:
@@ -994,41 +684,21 @@ class Style(bonsai.core.tool.Style):
                     return
                 raise error
         elif style_type == "Shading":
-            ext, fast = cls.get_branch_outputs(blender_material)
-            if ext and fast:
-                fast.is_active_output = True
-                ext.is_active_output = False
-                blender_material.update_tag()
-                return
             style_elements = tool.Style.get_style_elements(blender_material)
             rendering_style = None
             texture_style = None
-            shading_only_style = None
 
             for surface_style in style_elements.values():
                 if surface_style.is_a() == "IfcSurfaceStyleShading":
-                    shading_only_style = surface_style
                     tool.Loader.create_surface_style_shading(blender_material, surface_style)
                 elif surface_style.is_a("IfcSurfaceStyleRendering"):
                     rendering_style = surface_style
-                    shading_only_style = None  # rendering overrides shading-only path
                     tool.Loader.create_surface_style_rendering(blender_material, surface_style)
                 elif surface_style.is_a("IfcSurfaceStyleWithTextures"):
                     texture_style = surface_style
 
             if rendering_style and texture_style:
                 tool.Loader.create_surface_style_with_textures(blender_material, rendering_style, texture_style)
-            elif shading_only_style and not rendering_style:
-                # create a minimal Principled BSDF so Material Preview/Rendered shows the colour instead of white.
-                tool.Style.set_use_nodes(blender_material, True)
-                tool.Loader.restart_material_node_tree(blender_material)
-                bsdf = tool.Blender.get_material_node(blender_material, "BSDF_PRINCIPLED")
-                if bsdf:
-                    r, g, b, a = blender_material.diffuse_color
-                    bsdf.inputs["Base Color"].default_value = (r, g, b, 1)
-                    bsdf.inputs["Alpha"].default_value = a
-                    if a < 1.0:
-                        blender_material.blend_method = "BLEND"
         else:
             assert False, f"Invalid style type found: {style_type}"
 
@@ -1074,36 +744,3 @@ class Style(bonsai.core.tool.Style):
         elements = ifcopenshell.util.element.get_elements_by_style(tool.Ifc.get(), style)
         objects = [tool.Ifc.get_object(e) for e in elements]
         tool.Geometry.reload_representation(objects)
-
-    _last_shading_type: str | None = None
-
-    @classmethod
-    def restore_material_style_types(cls, shading_type: str) -> None:
-        """Set each IFC material's active_style_type to the richest available for the given viewport mode.
-
-        In SOLID mode all materials use "Shading".
-        In MATERIAL_PREVIEW / RENDERED, materials with an external .blend style use "External"
-        unless prefer_ifc_shading is set on that material.
-        """
-        if cls._last_shading_type == shading_type:
-            return
-        cls._last_shading_type = shading_type
-
-        for material in bpy.data.materials:
-            if not tool.Blender.get_ifc_definition_id(material):
-                continue
-            props = cls.get_material_style_props(material)
-            style_elements = cls.get_style_elements(material)
-            if shading_type == "SOLID":
-                props.active_style_type = "Shading"
-                shading = style_elements.get("IfcSurfaceStyleRendering") or style_elements.get("IfcSurfaceStyleShading")
-                if shading:
-                    d = tool.Loader.surface_style_to_dict(shading)
-                    alpha = 1.0 - (d.get("Transparency") or 0.0)
-                    material.diffuse_color = d["SurfaceColour"] + (alpha,)
-            else:  # MATERIAL_PREVIEW or RENDERED
-                if cls.has_blender_external_style(style_elements) and not props.prefer_ifc_shading:
-                    props.active_style_type = "External"
-                else:
-                    props.active_style_type = "Shading"
-            material.update_tag()

src/bonsai/scripts/dev_environment.py

@@ -15,7 +15,6 @@ Example usage:
 
 """
 
-import argparse
 import shutil
 import subprocess
 import sys
@@ -28,17 +27,6 @@ if sys.platform not in available_platforms:
     print(f"Currently only available on {', '.join(available_platforms)}. Not available on {sys.platform}.")
     exit(1)
 
-parser = argparse.ArgumentParser(description=__doc__)
-parser.add_argument(
-    "--skip-binaries",
-    action="store_true",
-    help=(
-        "Skip copying compiled dependencies (e.g. ifcopenshell_wrapper) to the repo. "
-        "Useful if you already have the latest binaries in the repo and don't want them to be overridden."
-    ),
-)
-args = parser.parse_args()
-
 # ---------------------------
 # SETTINGS.
 # ---------------------------
@@ -137,20 +125,17 @@ def main() -> None:
         path.unlink()
     subprocess.check_call(("git", "checkout", "--", symlinks_glob), cwd=REPO_PATH)
 
-    if args.skip_binaries:
-        print("Skipping copying compiled dependencies to the repo...")
-    else:
-        print("Copying compiled dependencies to the repo...")
-        dest = REPO_PATH / "src" / "ifcopenshell-python" / "ifcopenshell"
-        for path in PACKAGE_PATH.glob("ifcopenshell/*_wrapper*"):
-            if path.suffix.lower() == ".pyi":
-                continue
-            dest_ = dest / path.name
-            print(f"Copying {path} -> {dest_}")
-            try:
-                shutil.copy(path, dest_)
-            except shutil.SameFileError:
-                pass
+    print("Copying compiled dependencies to the repo...")
+    dest = REPO_PATH / "src" / "ifcopenshell-python" / "ifcopenshell"
+    for path in PACKAGE_PATH.glob("ifcopenshell/*_wrapper*"):
+        if path.suffix.lower() == ".pyi":
+            continue
+        dest_ = dest / path.name
+        print(f"Copying {path} -> {dest_}")
+        try:
+            shutil.copy(path, dest_)
+        except shutil.SameFileError:
+            pass
 
     print("Symlinking extension to the git repo...")
     # fmt: off

src/bonsai/test/bim/feature/boolean.feature

@@ -12,19 +12,16 @@ Scenario: Ensure added booleans are marked as manual
     And I click "OK"
     And the object "IfcFurniture/Unnamed" exists
     And I toggle edit mode
-    And the variable "extrusion" is "{ifc}.by_type('IfcExtrudedAreaSolid')[0].id()"
-    And the object "Item/IfcExtrudedAreaSolid/{extrusion}" exists
+    And the object "Item/IfcExtrudedAreaSolid/73" exists
     And I open the "Add Item" menu
     When I click "Half Space Solid"
-    And the variable "half_space" is "{ifc}.by_type('IfcHalfSpaceSolid')[0].id()"
-    And the variable "boolean" is "{ifc}.by_type('IfcBooleanResult')[0].id()"
-    And the object "Item/IfcHalfSpaceSolid/{half_space}" exists
+    And the object "Item/IfcHalfSpaceSolid/90" exists
     And I deselect all objects
     And I toggle edit mode
     And I select the object "IfcFurniture/Unnamed"
     And I look at the "Property Sets" panel
     Then I see "BBIM_Boolean"
-    And I see "[{boolean}]"
+    And I see "[91]"
 
 Scenario: Ensure removed booleans are unmarked as manual
     Given an empty IFC project
@@ -36,20 +33,17 @@ Scenario: Ensure removed booleans are unmarked as manual
     And I click "OK"
     And the object "IfcFurniture/Unnamed" exists
     And I toggle edit mode
-    And the variable "extrusion" is "{ifc}.by_type('IfcExtrudedAreaSolid')[0].id()"
-    And the object "Item/IfcExtrudedAreaSolid/{extrusion}" exists
+    And the object "Item/IfcExtrudedAreaSolid/73" exists
     And I open the "Add Item" menu
     And I click "Half Space Solid"
-    And the variable "half_space" is "{ifc}.by_type('IfcHalfSpaceSolid')[0].id()"
-    And the variable "boolean" is "{ifc}.by_type('IfcBooleanResult')[0].id()"
     And I deselect all objects
     And I toggle edit mode
     And I select the object "IfcFurniture/Unnamed"
     And I toggle edit mode
-    And I select the object "Item/IfcHalfSpaceSolid/{half_space}"
+    And I select the object "Item/IfcHalfSpaceSolid/90"
     When I delete the selected objects
     And I toggle edit mode
     And I select the object "IfcFurniture/Unnamed"
     And I look at the "Property Sets" panel
     Then I don't see "BBIM_Boolean"
-    And I don't see "[{boolean}]"
+    And I don't see "[91]"

src/bonsai/test/bim/feature/pset.feature

@@ -361,22 +361,31 @@ Scenario: Edit pset length property
     Given an empty IFC project
     And I press "mesh.add_stair"
     And the variable "pset" is "tool.Pset.get_element_pset(tool.Ifc.get_entity(bpy.context.active_object), 'Pset_StairFlightCommon').id()"
+    And the variable "si_conversion" is "ifcopenshell.util.unit.calculate_unit_scale(tool.Ifc.get())"
     And I press "bim.enable_pset_editing(pset_id={pset}, obj='IfcStairFlight/StairFlight', obj_type='Object')"
 
     # Testing IfcPositiveLengthMeasure type of prop
     Then "active_object.PsetProperties.properties['TreadLength'].metadata.special_type" is "LENGTH"
     And "active_object.PsetProperties.properties['TreadLength'].metadata.float_value" is "250"
+    And "active_object.PsetProperties.properties['TreadLength'].metadata.length_value" is roughly "0.25"
 
     When I set "active_object.PsetProperties.properties['TreadLength'].metadata.float_value" to "350"
     Then "active_object.PsetProperties.properties['TreadLength'].metadata.float_value" is roughly "350"
 
+    When I set "active_object.PsetProperties.properties['TreadLength'].metadata.length_value" to "0.45"
+    Then "active_object.PsetProperties.properties['TreadLength'].metadata.float_value" is roughly "450"
+
     # Testing IfcLengthMeasure type of prop
     Then "active_object.PsetProperties.properties['NosingLength'].metadata.special_type" is "LENGTH"
     And "active_object.PsetProperties.properties['NosingLength'].metadata.float_value" is "0.0"
+    And "active_object.PsetProperties.properties['NosingLength'].metadata.length_value" is roughly "0.0"
 
     When I set "active_object.PsetProperties.properties['NosingLength'].metadata.float_value" to "350"
     Then "active_object.PsetProperties.properties['NosingLength'].metadata.float_value" is roughly "350"
 
+    When I set "active_object.PsetProperties.properties['NosingLength'].metadata.length_value" to "0.45"
+    Then "active_object.PsetProperties.properties['NosingLength'].metadata.float_value" is roughly "450"
+
     When I press "bim.edit_pset(obj='IfcStairFlight/StairFlight', obj_type='Object')"
     Then nothing happens
 
@@ -385,14 +394,19 @@ Scenario: Edit qset length property
     And I press "mesh.add_stair"
     And I press "bim.perform_quantity_take_off"
     And the variable "pset" is "tool.Pset.get_element_pset(tool.Ifc.get_entity(bpy.context.active_object), 'Qto_StairFlightBaseQuantities').id()"
+    And the variable "si_conversion" is "ifcopenshell.util.unit.calculate_unit_scale(tool.Ifc.get())"
     And I press "bim.enable_pset_editing(pset_id={pset}, obj='IfcStairFlight/StairFlight', obj_type='Object')"
 
     # Testing Q_LENGTH type of prop
     Then "active_object.PsetProperties.properties['Length'].metadata.special_type" is "LENGTH"
     And "active_object.PsetProperties.properties['Length'].metadata.float_value" is roughly "2156.485"
+    And "active_object.PsetProperties.properties['Length'].metadata.length_value" is roughly "2.156"
 
     When I set "active_object.PsetProperties.properties['Length'].metadata.float_value" to "350"
-    Then "active_object.PsetProperties.properties['Length'].metadata.float_value" is roughly "350"
+    Then "active_object.PsetProperties.properties['Length'].metadata.length_value" is roughly "0.35"
+
+    When I set "active_object.PsetProperties.properties['Length'].metadata.length_value" to "0.45"
+    Then "active_object.PsetProperties.properties['Length'].metadata.float_value" is roughly "450"
 
     When I press "bim.edit_pset(obj='IfcStairFlight/StairFlight', obj_type='Object')"
     Then nothing happens

src/bonsai/test/bim/module/project/test_project_library_data.py

@@ -32,91 +32,64 @@ from test.bim.bootstrap import NewIfc
 pytestmark = pytest.mark.project
 
 
-def _make_library_file(*, with_child: bool = False) -> ifcopenshell.file:
-    """Build a spec-valid IFC4 library file: IfcProject + IfcProjectLibrary declared to it.
+def _make_library_only_file(*, with_child: bool = False) -> ifcopenshell.file:
+    """Build a minimal IFC4 file containing only an IfcProjectLibrary (no IfcProject).
 
-    Per the IFC Project Context concept template, every project data set (library
-    files included) shall contain exactly one IfcProject, and IfcProjectLibrary
-    instances are assigned to it via IfcRelDeclares. This matches how every library
-    file shipped in bonsai/bim/data/libraries is actually authored. ``with_child=True``
-    also nests a sub-library under the root via IfcRelNests.
+    Per IFC4+, a file must contain at least one IfcContext; IfcProjectLibrary is a
+    valid root on its own. ``with_child=True`` nests a sub-library under the root via
+    IfcRelNests, mirroring real authored library files.
     """
     library_file = ifcopenshell.api.project.create_file(version="IFC4")
-    project = ifcopenshell.api.root.create_entity(library_file, ifc_class="IfcProject", name="Demo Project")
     root = ifcopenshell.api.root.create_entity(library_file, ifc_class="IfcProjectLibrary", name="RootLib")
-    ifcopenshell.api.project.assign_declaration(library_file, definitions=[root], relating_context=project)
     if with_child:
         child = ifcopenshell.api.root.create_entity(library_file, ifc_class="IfcProjectLibrary", name="ChildLib")
         ifcopenshell.api.nest.assign_object(library_file, [child], root)
     return library_file
 
 
-class TestLibraryFile(NewIfc):
-    """Project-library UI code operating on a spec-valid model (IfcProject root).
+class TestLibraryOnlyFile(NewIfc):
+    def test_get_root_context_returns_project_library_when_no_project(self):
+        library_file = _make_library_only_file()
+        assert not library_file.by_type("IfcProject")
 
-    A file containing only IfcProjectLibrary and no IfcProject is not valid IFC and
-    is not supported; see test_parent_libraries_enum_raises_for_a_file_without_a_project.
-    """
+        root = tool.Project.get_root_context(library_file)
 
-    def test_parent_libraries_enum_raises_for_a_file_without_a_project(self):
-        library_file = ifcopenshell.api.project.create_file(version="IFC4")
-        ifcopenshell.api.root.create_entity(library_file, ifc_class="IfcProjectLibrary", name="RootLib")
-        IfcStore.library_file = library_file
-        try:
-            with pytest.raises(IndexError):
-                ProjectLibraryData.parent_libraries_enum()
-        finally:
-            IfcStore.library_file = None
+        assert root.is_a("IfcProjectLibrary")
+        assert root.Name == "RootLib"
 
-    def test_get_parent_library_returns_project_for_declared_root_library(self):
-        library_file = _make_library_file()
-        project = library_file.by_type("IfcProject")[0]
+    def test_get_parent_library_returns_none_for_root_library(self):
+        library_file = _make_library_only_file()
         root = library_file.by_type("IfcProjectLibrary")[0]
 
-        assert tool.Project.get_parent_library(root) == project
+        assert tool.Project.get_parent_library(root) is None
 
-    def test_get_parent_library_returns_the_library_for_a_nested_sub_library(self):
-        library_file = _make_library_file(with_child=True)
-        root = next(lib for lib in library_file.by_type("IfcProjectLibrary") if lib.Name == "RootLib")
-        child = next(lib for lib in library_file.by_type("IfcProjectLibrary") if lib.Name == "ChildLib")
-
-        assert tool.Project.get_parent_library(child) == root
-
-    def test_get_parent_library_returns_none_for_an_orphaned_library(self):
-        library_file = ifcopenshell.api.project.create_file(version="IFC4")
-        orphan = ifcopenshell.api.root.create_entity(library_file, ifc_class="IfcProjectLibrary", name="Orphan")
-
-        assert tool.Project.get_parent_library(orphan) is None
-
-    def test_get_project_hierarchy_roots_libraries_under_the_project(self):
-        library_file = _make_library_file(with_child=True)
-        project = library_file.by_type("IfcProject")[0]
+    def test_get_project_hierarchy_skips_root_library(self):
+        library_file = _make_library_only_file(with_child=True)
         root = next(lib for lib in library_file.by_type("IfcProjectLibrary") if lib.Name == "RootLib")
         child = next(lib for lib in library_file.by_type("IfcProjectLibrary") if lib.Name == "ChildLib")
 
         hierarchy = tool.Project.get_project_hierarchy(library_file)
 
-        assert root in hierarchy[project]
+        assert root in hierarchy
         assert child in hierarchy[root]
 
-    def test_project_library_data_loads_with_unique_enum_keys(self):
-        IfcStore.library_file = _make_library_file(with_child=True)
+    def test_project_library_data_loads_without_crash(self):
+        IfcStore.library_file = _make_library_only_file()
         try:
             ProjectLibraryData.is_loaded = False
             ProjectLibraryData.load()
             assert ProjectLibraryData.is_loaded
             enum = ProjectLibraryData.data["parent_libraries_enum"]
-            keys = [entry[0] for entry in enum]
-            assert len(keys) == len(set(keys))
-            assert enum[0][1].startswith("IfcProject ")
+            assert len(enum) == 1
+            assert enum[0][1].startswith("IfcProjectLibrary ")
         finally:
             IfcStore.library_file = None
             ProjectLibraryData.is_loaded = False
 
-    def test_refresh_library_succeeds(self):
+    def test_refresh_library_succeeds_on_library_only_file(self):
         import bpy
 
-        IfcStore.library_file = _make_library_file(with_child=True)
+        IfcStore.library_file = _make_library_only_file(with_child=True)
         try:
             result = bpy.ops.bim.refresh_library()
             assert result == {"FINISHED"}
@@ -124,65 +97,13 @@ class TestLibraryFile(NewIfc):
             IfcStore.library_file = None
             ProjectLibraryData.is_loaded = False
 
-    def test_edit_project_library_moves_a_project_declared_library_under_another_library(self):
+    def test_add_project_library_nests_under_root_when_no_project(self):
         import bpy
 
-        library_file = _make_library_file()
-        project = library_file.by_type("IfcProject")[0]
-        root = library_file.by_type("IfcProjectLibrary")[0]
-        target = ifcopenshell.api.root.create_entity(library_file, ifc_class="IfcProjectLibrary", name="TargetLib")
-        ifcopenshell.api.project.assign_declaration(library_file, definitions=[target], relating_context=project)
-        IfcStore.library_file = library_file
-        try:
-            props = tool.Project.get_project_props()
-            props.selected_project_library = str(root.id())
-            props.is_editing_project_library = True
-            props.parent_library = str(target.id())
-
-            result = bpy.ops.bim.edit_project_library()
-
-            assert result == {"FINISHED"}
-            assert tool.Project.get_parent_library(root) == target
-            assert root.Nests and root.Nests[0].RelatingObject == target
-            assert not root.HasContext
-        finally:
-            if props.is_editing_project_library:
-                props.is_editing_project_library = False
-            IfcStore.library_file = None
-            ProjectLibraryData.is_loaded = False
-
-    def test_edit_project_library_moves_a_nested_library_back_under_the_project(self):
-        import bpy
-
-        library_file = _make_library_file(with_child=True)
-        project = library_file.by_type("IfcProject")[0]
-        child = next(lib for lib in library_file.by_type("IfcProjectLibrary") if lib.Name == "ChildLib")
-        IfcStore.library_file = library_file
-        try:
-            props = tool.Project.get_project_props()
-            props.selected_project_library = str(child.id())
-            props.is_editing_project_library = True
-            props.parent_library = str(project.id())
-
-            result = bpy.ops.bim.edit_project_library()
-
-            assert result == {"FINISHED"}
-            assert tool.Project.get_parent_library(child) == project
-            assert child.HasContext and child.HasContext[0].RelatingContext == project
-            assert not child.Nests
-        finally:
-            if props.is_editing_project_library:
-                props.is_editing_project_library = False
-            IfcStore.library_file = None
-            ProjectLibraryData.is_loaded = False
-
-    def test_add_project_library_declares_new_library_under_the_project_root(self):
-        import bpy
-
-        IfcStore.library_file = _make_library_file()
+        IfcStore.library_file = _make_library_only_file()
         library_file = IfcStore.library_file
         try:
-            project = library_file.by_type("IfcProject")[0]
+            root = library_file.by_type("IfcProjectLibrary")[0]
             before = set(library_file.by_type("IfcProjectLibrary"))
 
             result = bpy.ops.bim.add_project_library()
@@ -192,9 +113,9 @@ class TestLibraryFile(NewIfc):
             new_libraries = after - before
             assert len(new_libraries) == 1
             new_library = next(iter(new_libraries))
-            assert new_library.HasContext
-            assert new_library.HasContext[0].RelatingContext == project
-            assert not new_library.Nests
+            assert new_library.Nests
+            assert new_library.Nests[0].RelatingObject == root
+            assert not new_library.HasContext
         finally:
             IfcStore.library_file = None
             ProjectLibraryData.is_loaded = False

src/bonsai/test/bim/stub.py

@@ -21,10 +21,6 @@ from typing import Any, Optional, Union
 
 
 class bSDDClientStub:
-    def __init__(self):
-        # Mirrors bsdd.Client so tool.Bsdd.identifier_url() works against the stub.
-        self.baseurl = "https://api.bsdd.buildingsmart.org/api/"
-
     def get_dictionary(self, dictionary_uri=None, include_test_dictionaries=False):
         dicts = {
             "dictionaries": [

src/bonsai/test/bim/test_feature.py

@@ -187,14 +187,7 @@ class PanelSpy:
             after = ""
             if self.spied_labels:
                 after = self.spied_labels[-1]
-            spied_operator = {
-                "operator": operator,
-                "icon": icon,
-                "text": text,
-                "kwargs": {},
-                "after": after,
-                "bl_idname": bl_idname,
-            }
+            spied_operator = {"operator": operator, "icon": icon, "text": text, "kwargs": {}, "after": after}
             self.spied_operators.append(spied_operator)
             return OperatorSpy(spied_operator)
         elif self.spied_attr == "panel":
@@ -217,14 +210,6 @@ class OperatorSpy:
         else:
             self.spied_data["kwargs"][name] = value
 
-    @property
-    def bl_rna(self) -> Any:
-        # Mirror the real `UILayout.operator()` return value (an OperatorProperties
-        # instance), which exposes `.bl_rna` so panel code such as
-        # `"module" in op.bl_rna.properties` (bonsai/bim/helper.py) also works when
-        # drawing is spied on during BDD tests.
-        return getattr(bpy.types, self.spied_data["bl_idname"]).bl_rna
-
 
 class TemplateListSpy(PanelSpy):
     items: bpy.types.bpy_prop_collection_idprop[bpy.types.PropertyGroup]
@@ -468,7 +453,6 @@ def i_trigger_operator(operator):
 @then(parsers.parse('I see "{text}"'))
 def i_see_text(text):
     assert panel_spy
-    text = replace_variables(text)
     panel_spy.refresh_spy()
     assert [l for l in panel_spy.spied_labels if text in l], f"Text {text} not found in {panel_spy.spied_labels}"
 
@@ -603,7 +587,6 @@ def i_select_the_row_where_i_see_text_in_the_nth_list(text, nth):
 @then(parsers.parse('I don\'t see "{text}"'))
 def i_dont_see_text(text):
     assert panel_spy
-    text = replace_variables(text)
     panel_spy.refresh_spy()
     assert not [l for l in panel_spy.spied_labels if text in l], f"Text {text} found in {panel_spy.spied_labels}"
 
@@ -790,11 +773,7 @@ def i_create_default_mep_types():
     with bpy.context.temp_override(active_object=bpy.data.objects["IfcActuatorType/ACTUATOR"]):
         bpy.ops.bim.add_port()
         # port at cube's left side
-        # Newly created ports are never given an explicit IFC `.Name` (see
-        # `core/system.py:create_port_at_cursor` / `tool/system.py`), so
-        # `tool.Loader.get_name()` falls back to the standard "Unnamed" convention
-        # used throughout Bonsai for freshly-created, not-yet-named elements.
-        bpy.data.objects["IfcDistributionPort/Unnamed"].location = (-0.5, 0, 0)
+        bpy.data.objects["IfcDistributionPort/Port"].location = (-0.5, 0, 0)
         bpy.ops.bim.hide_ports()
 
 
@@ -1094,7 +1073,6 @@ def then_the_object_name_is_placed_in_the_collection_collection(name: str, colle
 @given(parsers.parse('additionally the object "{name}" is selected'))
 @when(parsers.parse('additionally the object "{name}" is selected'))
 def additionally_the_object_name_is_selected(name):
-    name = replace_variables(name)
     obj = bpy.context.scene.objects.get(name)
     if not obj:
         total = len(bpy.context.scene.objects)
@@ -1175,7 +1153,6 @@ def nothing_happens():
 @when(parsers.parse('the object "{name}" exists'))
 @then(parsers.parse('the object "{name}" exists'))
 def the_object_name_exists(name: str) -> bpy.types.Object:
-    name = replace_variables(name)
     # Some objects from linked collections may share the same name. This disambiguates them.
     if name.startswith("Col:"):
         _, collection_name, name = name.split(":")
@@ -1190,7 +1167,6 @@ def the_object_name_exists(name: str) -> bpy.types.Object:
 
 @then(parsers.parse('the object "{name}" does not exist'))
 def the_object_name_does_not_exist(name) -> None:
-    name = replace_variables(name)
     obj = bpy.data.objects.get(name)
     assert obj is None, f'The object "{name}" exists'
 

src/bonsai/test/bim/test_prop.py

@@ -1,328 +0,0 @@
-# Bonsai - OpenBIM Blender Add-on
-# Copyright (C) 2026
-#
-# This file is part of Bonsai.
-#
-# Bonsai is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# Bonsai is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with Bonsai.  If not, see <http://www.gnu.org/licenses/>.
-
-import bpy
-import ifcopenshell
-import ifcopenshell.api.pset
-import ifcopenshell.api.root
-import ifcopenshell.api.unit
-import pytest
-
-import bonsai.bim.prop
-import bonsai.tool as tool
-from test.bim.bootstrap import NewFile
-
-
-def import_single_property(ifc, element, prop):
-    """Import a single existing IfcProperty into a real, addon-registered
-    PsetProperties collection, exactly as the property editor does, and
-    return its `metadata` (an `Attribute`)."""
-    pset = ifcopenshell.api.pset.add_pset(ifc, product=element, name="Pset_Test")
-    pset.HasProperties = [prop]
-    obj = bpy.data.objects.new(prop.Name, None)
-    tool.Ifc.link(element, obj)
-    props = obj.PsetProperties
-    tool.Pset.import_pset_from_existing(pset, props, None)
-    return props.properties[prop.Name].metadata
-
-
-class TestGetDisplayName(NewFile):
-    def test_appends_the_resolved_unit_symbol(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        pressure = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="PRESSUREUNIT")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[pressure])
-
-        element = ifc.createIfcWall()
-        prop = ifc.createIfcPropertySingleValue(Name="Foo", NominalValue=ifc.createIfcPressureMeasure(5.0))
-        metadata = import_single_property(ifc, element, prop)
-
-        assert metadata.display_name == "Foo, Pa"
-
-    def test_falls_back_to_the_plain_name_when_no_unit_is_resolvable(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        # No units assigned to the project at all -- nothing to resolve.
-
-        element = ifc.createIfcWall()
-        prop = ifc.createIfcPropertySingleValue(Name="Foo", NominalValue=ifc.createIfcPressureMeasure(5.0))
-        metadata = import_single_property(ifc, element, prop)
-
-        assert metadata.unit_symbol == ""
-        assert metadata.display_name == "Foo"
-
-    def test_falls_back_to_the_plain_name_for_a_non_measure_property(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-
-        element = ifc.createIfcWall()
-        prop = ifc.createIfcPropertySingleValue(Name="Foo", NominalValue=ifc.createIfcText("Bar"))
-        metadata = import_single_property(ifc, element, prop)
-
-        assert metadata.unit_symbol == ""
-        assert metadata.display_name == "Foo"
-
-    def test_resolves_a_unit_explicitly_attached_to_a_generic_numeric_value(self):
-        # A generic IfcReal has no unit semantics per its own declared type, but a property
-        # set may still explicitly attach a real Unit to a specific instance to convey the
-        # dimension the spec's generic typing doesn't. That explicit Unit is real, deliberate
-        # data (not incidental/stray), so it should resolve normally.
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_mm = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT", prefix="MILLI")
-
-        element = ifc.createIfcWall()
-        prop = ifc.createIfcPropertySingleValue(Name="Foo", NominalValue=ifc.createIfcReal(150.0), Unit=length_mm)
-        metadata = import_single_property(ifc, element, prop)
-
-        assert metadata.unit_symbol == "mm"
-        assert metadata.display_name == "Foo, mm"
-
-
-class TestGetAttributeUnitEnumItems(NewFile):
-    def test_returns_default_plus_one_per_candidate_for_a_measurable_type(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_mm = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT", prefix="MILLI")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[length_mm])
-        length_m = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-
-        element = ifc.createIfcWall()
-        prop = ifc.createIfcPropertySingleValue(Name="Foo", NominalValue=ifc.createIfcLengthMeasure(2.5))
-        metadata = import_single_property(ifc, element, prop)
-
-        items = bonsai.bim.prop.get_attribute_unit_enum_items(metadata, bpy.context)
-        identifiers = [i[0] for i in items]
-        assert identifiers[0] == "0"
-        assert str(length_mm.id()) in identifiers
-        assert str(length_m.id()) in identifiers
-        assert len(items) == 3  # Default + mm + m
-
-    def test_returns_just_default_for_non_measurable_types(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-
-        element = ifc.createIfcWall()
-        prop = ifc.createIfcPropertySingleValue(Name="Foo", NominalValue=ifc.createIfcText("Bar"))
-        metadata = import_single_property(ifc, element, prop)
-
-        items = bonsai.bim.prop.get_attribute_unit_enum_items(metadata, bpy.context)
-        assert [i[0] for i in items] == ["0"]
-
-
-class TestGetUnitEnumItemsForSpecialType(NewFile):
-    def test_matches_the_attribute_wrapper_output(self):
-        # Regression test for extracting get_unit_enum_items_for_special_type out of
-        # get_attribute_unit_enum_items: the wrapper must still produce identical items for
-        # the plain (no own-unit-fallback-needed) case.
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_mm = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT", prefix="MILLI")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[length_mm])
-        ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-
-        element = ifc.createIfcWall()
-        prop = ifc.createIfcPropertySingleValue(Name="Foo", NominalValue=ifc.createIfcLengthMeasure(2.5))
-        metadata = import_single_property(ifc, element, prop)
-
-        direct_items = bonsai.bim.prop.get_unit_enum_items_for_special_type(metadata.special_type, ifc)
-        wrapper_items = bonsai.bim.prop.get_attribute_unit_enum_items(metadata, bpy.context)
-        assert direct_items == wrapper_items
-
-    def test_returns_just_default_when_ifc_file_is_none(self):
-        assert bonsai.bim.prop.get_unit_enum_items_for_special_type("LENGTH", None) == [("0", "Default", "")]
-
-
-class TestUnitSymbolWithAreaVolumeDerivedFromLength(NewFile):
-    """Regression test: AREAUNIT/VOLUMEUNIT have no IfcDerivedUnitEnum member, so a project
-    whose area/volume default is an IfcDerivedUnit rather than a literal-UnitType-matching
-    IfcSIUnit/IfcConversionBasedUnit has no literal UnitType to match on.
-    ifcopenshell.util.unit.get_project_unit() used to only match by literal UnitType, so the
-    read-only unit symbol and the edit-mode "Default (<symbol>)" picker entry both silently
-    fell back to no symbol at all in that case.
-    """
-
-    def setup_project_with_derived_area_and_volume(self, ifc):
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-        area = ifcopenshell.api.unit.add_derived_unit(ifc, "USERDEFINED", "area-ish", {length: 2})
-        volume = ifcopenshell.api.unit.add_derived_unit(ifc, "USERDEFINED", "volume-ish", {length: 3})
-        ifcopenshell.api.unit.assign_unit(ifc, units=[length, area, volume])
-
-    def test_default_picker_entry_shows_the_resolved_symbol(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        self.setup_project_with_derived_area_and_volume(ifc)
-
-        element = ifc.createIfcWall()
-        prop = ifc.createIfcPropertySingleValue(Name="Foo", NominalValue=ifc.createIfcAreaMeasure(5.0))
-        metadata = import_single_property(ifc, element, prop)
-
-        items = bonsai.bim.prop.get_attribute_unit_enum_items(metadata, bpy.context)
-        assert items[0][1] == "Default (m2)"
-
-    def test_read_only_display_resolves_the_symbol(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        self.setup_project_with_derived_area_and_volume(ifc)
-
-        element = ifc.createIfcWall()
-        prop = ifc.createIfcPropertySingleValue(Name="Foo", NominalValue=ifc.createIfcVolumeMeasure(5.0))
-        metadata = import_single_property(ifc, element, prop)
-
-        assert metadata.unit_symbol == "m3"
-        assert metadata.display_name == "Foo, m3"
-
-
-class TestUpdateAttributeUnitId(NewFile):
-    def test_syncs_unit_id_and_converts_float_value_when_unit_id_enum_changes(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_mm = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT", prefix="MILLI")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[length_mm])
-        length_m = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-
-        element = ifc.createIfcWall()
-        prop = ifc.createIfcPropertySingleValue(Name="Foo", NominalValue=ifc.createIfcLengthMeasure(2500.0))
-        metadata = import_single_property(ifc, element, prop)
-        assert metadata.unit_id == 0
-        assert metadata.float_value == 2500.0
-        assert metadata.unit_symbol == "mm"
-        assert metadata.display_name == "Foo, mm"
-
-        metadata.unit_id_enum = str(length_m.id())
-
-        assert metadata.unit_id == length_m.id()
-        assert metadata.float_value == pytest.approx(2.5)  # converted, not just relabeled
-        # Regression test: unit_symbol/display_name used to be a snapshot taken once at import
-        # time, so picking a different unit converted the value but left the label showing the
-        # old unit's symbol.
-        assert metadata.unit_symbol == "m"
-        assert metadata.display_name == "Foo, m"
-
-
-class TestUnitSymbolReflectsLiveProjectState(NewFile):
-    def test_symbol_updates_after_a_project_default_unit_is_assigned_later(self):
-        # Regression test: unit_symbol used to be a snapshot computed once at import time, so a
-        # property/quantity imported before its measure type had a project default unit assigned
-        # kept showing no symbol even after one was added, unless the panel was closed and
-        # reopened (re-triggering import). It's now computed fresh on every access.
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        # No AREAUNIT assigned yet.
-
-        element = ifc.createIfcWall()
-        prop = ifc.createIfcPropertySingleValue(Name="Foo", NominalValue=ifc.createIfcAreaMeasure(5.0))
-        metadata = import_single_property(ifc, element, prop)
-        assert metadata.unit_symbol == ""
-        assert metadata.display_name == "Foo"
-
-        area = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="AREAUNIT")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[area])
-
-        assert metadata.unit_symbol == "m2"
-        assert metadata.display_name == "Foo, m2"
-
-
-class TestImportPsetFromExistingWithAGenericNumericValueAndAnExplicitUnit(NewFile):
-    def test_run(self):
-        # Regression test: some property set specifications declare a property as a generic
-        # IfcReal rather than a proper measure class, relying on an explicit Unit attribute
-        # alone to convey the dimension. get_property_unit() already handled this fine for
-        # display (it checks prop.Unit before looking at NominalValue's type at all), but
-        # get_special_type_for_prop() only looked at NominalValue's class ending in "Measure"
-        # -- so special_type came back "", the picker never appeared, and the real Unit
-        # override never got seeded into unit_id even though it was legitimately set.
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_mm = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT", prefix="MILLI")
-
-        element = ifc.createIfcWall()
-        prop = ifc.createIfcPropertySingleValue(
-            Name="Foo", NominalValue=ifc.createIfcReal(150.0), Unit=length_mm
-        )
-        metadata = import_single_property(ifc, element, prop)
-
-        assert metadata.special_type == "LENGTH"
-        assert tool.Pset.is_measurable_special_type(metadata.special_type)
-        assert metadata.unit_symbol == "mm"
-        assert metadata.unit_id == length_mm.id()
-        assert metadata.unit_id_enum == str(length_mm.id())
-
-        items = bonsai.bim.prop.get_attribute_unit_enum_items(metadata, bpy.context)
-        assert str(length_mm.id()) in [i[0] for i in items]
-
-
-class TestImportPsetFromExistingWithAStrayUnitOnANonMeasureProperty(NewFile):
-    def test_run(self):
-        # Regression test: some real-world exporters set a Unit on a property whose
-        # NominalValue isn't actually a measure (e.g. a text classification), which used
-        # to crash import_pset_from_existing with "enum '<id>' not found in ('0')" -- unit_id
-        # was seeded from prop.Unit unconditionally, before the special_type gate that decides
-        # whether Unit is even meaningful for this property.
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_m = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-
-        element = ifc.createIfcWall()
-        prop = ifc.createIfcPropertySingleValue(
-            Name="Foo", NominalValue=ifc.createIfcLabel("Bar"), Unit=length_m
-        )
-        metadata = import_single_property(ifc, element, prop)  # must not raise
-
-        assert metadata.special_type == ""
-        assert metadata.unit_id == 0
-        assert metadata.unit_id_enum == "0"
-
-
-class TestGetAttributeUnitEnumItemsWithAMismatchedUnit(NewFile):
-    def test_own_unit_is_always_representable_even_if_not_a_normal_candidate(self):
-        # Regression test: a property's own Unit might not satisfy
-        # get_candidate_units_for_special_type's matching (e.g. mismatched UnitType in messy
-        # real-world data). Seeding must never crash trying to select it, and it should still
-        # show up in the picker so the user can see/change it.
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        # A LENGTHUNIT attached to a PRESSURE-typed property -- a real mismatch, not a candidate
-        # get_candidate_units_for_special_type("PRESSURE", ...) would ever return.
-        mismatched_unit = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-
-        element = ifc.createIfcWall()
-        prop = ifc.createIfcPropertySingleValue(
-            Name="Foo", NominalValue=ifc.createIfcPressureMeasure(5.0), Unit=mismatched_unit
-        )
-        metadata = import_single_property(ifc, element, prop)  # must not raise
-
-        assert metadata.special_type == "PRESSURE"
-        assert metadata.unit_id == mismatched_unit.id()
-        assert metadata.unit_id_enum == str(mismatched_unit.id())
-
-        items = bonsai.bim.prop.get_attribute_unit_enum_items(metadata, bpy.context)
-        assert str(mismatched_unit.id()) in [i[0] for i in items]

src/bonsai/test/core/test_material.py

@@ -93,22 +93,6 @@ class TestRemoveMaterialSet:
         subject.remove_material_set(ifc, material, material="material")
 
 
-class TestRenameMaterial:
-    def test_renaming_a_material(self, ifc, material):
-        ifc.run("material.edit_material", material="material", attributes={"Name": "name"}).should_be_called()
-        material.is_editing_materials().should_be_called().will_return(False)
-        material.refresh().should_be_called()
-        subject.rename_material(ifc, material, material="material", name="name")
-
-    def test_renaming_a_material_and_reloading_imported_materials(self, ifc, material):
-        ifc.run("material.edit_material", material="material", attributes={"Name": "name"}).should_be_called()
-        material.is_editing_materials().should_be_called().will_return(True)
-        material.get_active_material_type().should_be_called().will_return("material_type")
-        material.import_material_definitions("material_type").should_be_called()
-        material.refresh().should_be_called()
-        subject.rename_material(ifc, material, material="material", name="name")
-
-
 class TestLoadMaterials:
     def test_run(self, material):
         material.import_material_definitions("material_type").should_be_called()

src/bonsai/test/tool/test_ifcgit.py

@@ -497,8 +497,6 @@ class TestGitMergetool:
         with tempfile.TemporaryDirectory() as tmpdir:
             ifc_path = os.path.join(tmpdir, "model.ifc")
             mock_repo = mock.MagicMock()
-            # A clean mergetool resolution leaves no unmerged blobs in the index.
-            mock_repo.index.unmerged_blobs.return_value = {}
             IfcGitRepo.repo = mock_repo
             result = IfcGit.git_mergetool("ifcmerge", ifc_path)
             assert result is None
@@ -513,8 +511,6 @@ class TestGitMergetool:
             report_path = ifc_path + ".ifcmerge"
             open(report_path, "w").close()
             mock_repo = mock.MagicMock()
-            # A clean mergetool resolution leaves no unmerged blobs in the index.
-            mock_repo.index.unmerged_blobs.return_value = {}
             IfcGitRepo.repo = mock_repo
             result = IfcGit.git_mergetool("ifcmerge", ifc_path)
             assert result is None

src/bonsai/test/tool/test_material.py

@@ -139,17 +139,6 @@ class TestImportMaterialDefinitions(NewFile):
         assert props.materials[0].total_elements == 0
 
 
-class TestRefresh(NewFile):
-    def test_run(self):
-        from bonsai.bim.module.material.data import MaterialsData, ObjectMaterialData
-
-        MaterialsData.is_loaded = True
-        ObjectMaterialData.is_loaded = True
-        subject.refresh()
-        assert MaterialsData.is_loaded is False
-        assert ObjectMaterialData.is_loaded is False
-
-
 class TestIsEditingMaterials(NewFile):
     def test_run(self):
         props = tool.Material.get_material_props()

src/bonsai/test/tool/test_polyline.py

@@ -56,31 +56,3 @@ class TestValidateInput(NewFile):
 
         # Angle.
         assert subject.validate_input("25", "A") == (True, "25.0")
-
-
-class TestCalculateDistanceAndAngle(NewFile):
-    def test_it_does_not_crash_when_distance_is_zero_and_should_round(self, monkeypatch):
-        # Regression test for #8597: right after placing the first polyline
-        # point, the initial mouse sample can equal the last placed point
-        # (distance == 0), e.g. entering the viewport on a YZ plane wall.
-        # angle_round_threshold used to only be assigned in the
-        # `distance > 0` branch, crashing when should_round reads it here.
-        # get_increment_snap_value requires a real 3D viewport rv3d, which
-        # is unrelated to this bug, so it's stubbed out for a headless run.
-        monkeypatch.setattr(tool.Snap, "get_increment_snap_value", classmethod(lambda cls, context: 1.0))
-
-        polyline_props = tool.Model.get_polyline_props()
-        mouse_point = polyline_props.snap_mouse_point.add()
-        mouse_point.x, mouse_point.y, mouse_point.z = 0, 0, 0
-
-        tool_state = subject.create_tool_state()
-        tool_state.is_input_on = False
-        tool_state.use_default_container = False
-        tool_state.plane_method = "YZ"
-
-        input_ui = subject.create_input_ui(input_options=["D", "A", "X", "Y", "Z"])
-
-        subject.calculate_distance_and_angle(bpy.context, input_ui, tool_state, should_round=True)
-
-        assert input_ui.get_number_value("D") == 0
-        assert input_ui.get_number_value("A") == 0

src/bonsai/test/tool/test_pset.py

@@ -20,9 +20,6 @@ import bpy
 import ifcopenshell
 import ifcopenshell.api
 import ifcopenshell.api.pset
-import ifcopenshell.api.root
-import ifcopenshell.api.unit
-import pytest
 
 import bonsai.core.tool
 import bonsai.tool as tool
@@ -55,340 +52,3 @@ class TestIsPsetEmpty(NewFile):
         assert subject.is_pset_empty(pset) is False
         ifcopenshell.api.pset.edit_pset(ifc, pset=pset, properties={"Foo": None})
         assert subject.is_pset_empty(pset) is True
-
-
-class TestEditingAnOverriddenUnitPropertyRoundTrips(NewFile):
-    def test_run(self):
-        # Project default is mm, but this property is authored directly in m.
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_mm = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT", prefix="MILLI")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[length_mm])
-        length_m = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-
-        element = ifc.createIfcWall()
-        pset = ifcopenshell.api.pset.add_pset(ifc, product=element, name="Pset_Test")
-        prop = ifc.createIfcPropertySingleValue(
-            Name="Foo", NominalValue=ifc.createIfcLengthMeasure(2.5), Unit=length_m
-        )
-        pset.HasProperties = [prop]
-
-        obj = bpy.data.objects.new("Wall", None)
-        tool.Ifc.link(element, obj)
-        blender_props = obj.PsetProperties
-        subject.import_pset_from_existing(pset, blender_props, None)
-
-        metadata = blender_props.properties["Foo"].metadata
-        assert metadata.unit_symbol == "m"
-        assert metadata.float_value == 2.5  # raw stored value, not rescaled to the project's mm
-
-        # Simulate a user edit in the property editor.
-        metadata.float_value = 3.5
-
-        # Simulate what EditPset.execute() does: collect the raw value straight
-        # off the metadata and write it back, with no rescaling step.
-        properties = {"Foo": metadata.get_value()}
-        ifcopenshell.api.pset.edit_pset(ifc, pset=pset, properties=properties)
-
-        assert prop.NominalValue.wrappedValue == 3.5  # not rescaled to 3500mm
-        assert prop.Unit == length_m  # override preserved
-
-
-class TestImportingATemplatedQuantityRespectsItsOwnUnitOverride(NewFile):
-    def test_run(self):
-        # Regression test: import_pset_from_template's Q_ branch used to
-        # unconditionally re-template existing quantities, which shadowed
-        # their own Unit override with the project default -- edit mode
-        # showed "m" while the read-only panel correctly showed "mm".
-        # Project default is m, but this quantity is authored directly in mm.
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_m = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[length_m])
-        length_mm = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT", prefix="MILLI")
-
-        element = ifc.createIfcBeam()
-        qto = ifcopenshell.api.pset.add_qto(ifc, product=element, name="Qto_Test")
-        quantity = ifc.createIfcQuantityLength(Name="Foo", Unit=length_mm, LengthValue=2500.0)
-        qto.Quantities = [quantity]
-
-        pset_template = ifc.createIfcPropertySetTemplate(
-            Name="Qto_Test",
-            TemplateType="PSET_TYPEDRIVENOVERRIDE",
-            ApplicableEntity="IfcBeam",
-            HasPropertyTemplates=[ifc.createIfcSimplePropertyTemplate(Name="Foo", TemplateType="Q_LENGTH")],
-        )
-
-        obj = bpy.data.objects.new("Beam", None)
-        tool.Ifc.link(element, obj)
-        blender_props = obj.PsetProperties
-        # Mirrors core/pset.py's enable_pset_editing: template pass, then existing-data pass.
-        subject.import_pset_from_template(pset_template, qto, blender_props)
-        subject.import_pset_from_existing(qto, blender_props, pset_template)
-
-        assert len(blender_props.properties) == 1  # not duplicated by the template pass
-        metadata = blender_props.properties["Foo"].metadata
-        assert metadata.unit_symbol == "mm"  # the quantity's own override, not the project default "m"
-        assert metadata.float_value == 2500.0  # raw stored value, not rescaled
-
-
-class TestIsMeasurableSpecialType(NewFile):
-    def test_run(self):
-        for special_type in ("", "DATE", "DATETIME", "LOGICAL", "URI", "DURATION"):
-            assert subject.is_measurable_special_type(special_type) is False
-        assert subject.is_measurable_special_type("LENGTH") is True
-        assert subject.is_measurable_special_type("PRESSURE") is True
-
-
-class TestGetCandidateUnitsForSpecialType(NewFile):
-    def test_returns_candidates_matching_the_special_type(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_mm = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT", prefix="MILLI")
-        length_m = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-        ifcopenshell.api.unit.add_si_unit(ifc, unit_type="AREAUNIT")
-        assert set(subject.get_candidate_units_for_special_type("LENGTH", ifc)) == {length_mm, length_m}
-
-    def test_gating_returns_empty_for_non_measurable_special_types(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-        assert subject.get_candidate_units_for_special_type("", ifc) == []
-        assert subject.get_candidate_units_for_special_type("URI", ifc) == []
-
-
-class TestResolveEffectiveUnit(NewFile):
-    def test_own_override_takes_precedence_over_project_default(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_mm = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT", prefix="MILLI")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[length_mm])
-        length_m = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-        assert subject.resolve_effective_unit("LENGTH", length_m.id(), ifc) == length_m
-
-    def test_falls_back_to_project_default_when_unit_id_is_zero(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_mm = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT", prefix="MILLI")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[length_mm])
-        assert subject.resolve_effective_unit("LENGTH", 0, ifc) == length_mm
-
-
-class TestConvertAttributeUnit(NewFile):
-    def _new_metadata(self, ifc: ifcopenshell.file):
-        element = ifc.createIfcWall()
-        obj = bpy.data.objects.new("Wall", None)
-        tool.Ifc.link(element, obj)
-        props = obj.PsetProperties
-        new_prop = props.properties.add()
-        new_prop.name = "Foo"
-        return new_prop.metadata
-
-    def test_converts_value_between_two_explicit_units(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_mm = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT", prefix="MILLI")
-        length_m = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-
-        metadata = self._new_metadata(ifc)
-        metadata.special_type = "LENGTH"
-        metadata.unit_id = length_mm.id()
-        metadata.float_value = 2500.0
-
-        subject.convert_attribute_unit(metadata, length_m.id(), ifc)
-        assert metadata.float_value == pytest.approx(2.5)
-
-    def test_converts_value_when_switching_to_and_from_the_project_default(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_m = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[length_m])
-        length_ft = ifcopenshell.api.unit.add_conversion_based_unit(ifc, name="foot")
-
-        metadata = self._new_metadata(ifc)
-        metadata.special_type = "LENGTH"
-        metadata.unit_id = length_ft.id()
-        metadata.float_value = 10.0  # 10 ft
-
-        subject.convert_attribute_unit(metadata, 0, ifc)  # 0 = switch to project default (m)
-        assert metadata.float_value == pytest.approx(3.048)
-
-    def test_noop_when_old_and_new_resolve_to_the_same_unit(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_m = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[length_m])
-
-        metadata = self._new_metadata(ifc)
-        metadata.special_type = "LENGTH"
-        metadata.unit_id = 0  # already resolves to length_m (the project default)
-        metadata.float_value = 5.0
-
-        subject.convert_attribute_unit(metadata, length_m.id(), ifc)
-        assert metadata.float_value == 5.0
-
-    def test_noop_for_a_non_measurable_special_type(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_m = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-
-        metadata = self._new_metadata(ifc)
-        metadata.special_type = ""
-        metadata.unit_id = 0
-        metadata.float_value = 5.0
-
-        subject.convert_attribute_unit(metadata, length_m.id(), ifc)
-        assert metadata.float_value == 5.0
-
-
-def _build_wrapped_properties_from_ui(blender_props) -> dict:
-    """Mirrors EditPset._execute()'s properties-building loop (operator.py)."""
-    properties = {}
-    for entry in blender_props.properties:
-        metadata = entry.metadata
-        value = metadata.get_value()
-        if value is not None and subject.is_measurable_special_type(metadata.special_type):
-            unit = tool.Ifc.get().by_id(metadata.unit_id) if metadata.unit_id else None
-            value = {"NominalValue": value, "Unit": unit}
-        properties[metadata.name] = value
-    return properties
-
-
-class TestEditPsetWithUnitOverridePicker(NewFile):
-    def test_picking_a_different_unit_converts_the_displayed_value_and_writes_it_back(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_mm = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT", prefix="MILLI")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[length_mm])
-        length_m = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-
-        element = ifc.createIfcWall()
-        pset = ifcopenshell.api.pset.add_pset(ifc, product=element, name="Pset_Test")
-        prop = ifc.createIfcPropertySingleValue(Name="Foo", NominalValue=ifc.createIfcLengthMeasure(2500.0))
-        pset.HasProperties = [prop]
-
-        obj = bpy.data.objects.new("Wall", None)
-        tool.Ifc.link(element, obj)
-        blender_props = obj.PsetProperties
-        subject.import_pset_from_existing(pset, blender_props, None)
-
-        metadata = blender_props.properties["Foo"].metadata
-        assert metadata.unit_id == 0
-        assert metadata.float_value == 2500.0
-
-        # Simulate the user picking "m" in the unit picker dropdown.
-        metadata.unit_id_enum = str(length_m.id())
-        assert metadata.float_value == pytest.approx(2.5)  # converted live, not just relabeled
-        assert metadata.unit_id == length_m.id()
-
-        properties = _build_wrapped_properties_from_ui(blender_props)
-        ifcopenshell.api.pset.edit_pset(ifc, pset=pset, properties=properties)
-
-        assert prop.NominalValue.wrappedValue == pytest.approx(2.5)
-        assert prop.Unit == length_m
-
-    def test_picking_default_after_an_override_converts_back_and_clears_the_unit(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_mm = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT", prefix="MILLI")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[length_mm])
-        length_m = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-
-        element = ifc.createIfcWall()
-        pset = ifcopenshell.api.pset.add_pset(ifc, product=element, name="Pset_Test")
-        prop = ifc.createIfcPropertySingleValue(
-            Name="Foo", NominalValue=ifc.createIfcLengthMeasure(2.5), Unit=length_m
-        )
-        pset.HasProperties = [prop]
-
-        obj = bpy.data.objects.new("Wall", None)
-        tool.Ifc.link(element, obj)
-        blender_props = obj.PsetProperties
-        subject.import_pset_from_existing(pset, blender_props, None)
-
-        metadata = blender_props.properties["Foo"].metadata
-        assert metadata.unit_id == length_m.id()
-        assert metadata.float_value == 2.5
-
-        # Simulate picking "Default" (mm).
-        metadata.unit_id_enum = "0"
-        assert metadata.float_value == pytest.approx(2500.0)
-        assert metadata.unit_id == 0
-
-        properties = _build_wrapped_properties_from_ui(blender_props)
-        ifcopenshell.api.pset.edit_pset(ifc, pset=pset, properties=properties)
-
-        assert prop.Unit is None
-        assert prop.NominalValue.wrappedValue == pytest.approx(2500.0)
-
-    def test_editing_an_unrelated_sibling_property_does_not_disturb_this_ones_override(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject")
-        length_m = ifcopenshell.api.unit.add_si_unit(ifc, unit_type="LENGTHUNIT")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[length_m])
-        length_ft = ifcopenshell.api.unit.add_conversion_based_unit(ifc, name="foot")
-
-        element = ifc.createIfcWall()
-        pset = ifcopenshell.api.pset.add_pset(ifc, product=element, name="Pset_Test")
-        overridden_prop = ifc.createIfcPropertySingleValue(
-            Name="Foo", NominalValue=ifc.createIfcLengthMeasure(10.0), Unit=length_ft
-        )
-        untouched_prop = ifc.createIfcPropertySingleValue(Name="Bar", NominalValue=ifc.createIfcLengthMeasure(3.0))
-        pset.HasProperties = [overridden_prop, untouched_prop]
-
-        obj = bpy.data.objects.new("Wall", None)
-        tool.Ifc.link(element, obj)
-        blender_props = obj.PsetProperties
-        subject.import_pset_from_existing(pset, blender_props, None)
-
-        # Edit only "Bar", never touching "Foo"'s unit dropdown.
-        blender_props.properties["Bar"].metadata.float_value = 4.0
-
-        properties = _build_wrapped_properties_from_ui(blender_props)
-        ifcopenshell.api.pset.edit_pset(ifc, pset=pset, properties=properties)
-
-        assert overridden_prop.Unit == length_ft  # untouched override survives
-        assert overridden_prop.NominalValue.wrappedValue == 10.0
-        assert untouched_prop.NominalValue.wrappedValue == 4.0
-
-
-class TestEditQtoRoundingLoopPreservesUnitWrappedValues(NewFile):
-    def test_run(self):
-        # Regression test for EditPset._execute()'s qto post-processing loop: it must reach
-        # into {"Unit": ..., "NominalValue": ...}-wrapped values to round them, rather than
-        # treating the whole dict as a bare float/int (which would zero it out).
-        properties = {
-            "Foo": {"NominalValue": 2.123456, "Unit": None},
-            "Bar": 3,
-        }
-        for key, value in properties.items():
-            if value is None:
-                continue
-            is_wrapped = isinstance(value, dict) and "Unit" in value
-            raw = value["NominalValue"] if is_wrapped else value
-            if raw is None:
-                continue
-            if isinstance(raw, float):
-                raw = round(raw, 4)
-            elif not isinstance(raw, int):
-                raw = 0
-            if is_wrapped:
-                value["NominalValue"] = raw
-            else:
-                properties[key] = raw
-        assert properties["Foo"]["NominalValue"] == 2.1235
-        assert properties["Foo"]["Unit"] is None
-        assert properties["Bar"] == 3

src/bonsai/test/tool/test_qto.py

@@ -181,42 +181,6 @@ class TestGetCalculatedObjectQuantities(test.bim.bootstrap.NewFile):
         assert quantities["NetVolume"] == 282.517
 
 
-class TestGetTargetUnits(test.bim.bootstrap.NewFile):
-    def setup_file(self):
-        ifc = ifcopenshell.file()
-        tool.Ifc.set(ifc)
-        ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcProject", name="Test")
-        return ifc
-
-    def test_default_scene_state_returns_nothing(self):
-        self.setup_file()
-        assert subject.get_target_units() == {}
-
-    def test_setting_a_field_maps_it_to_its_measure_class(self):
-        ifc = self.setup_file()
-        metre = ifc.createIfcSIUnit(None, "LENGTHUNIT", None, "METRE")
-        millimetre = ifc.createIfcSIUnit(None, "LENGTHUNIT", "MILLI", "METRE")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[metre])
-
-        props = tool.Qto.get_qto_props()
-        props.target_unit_length = str(millimetre.id())
-
-        assert subject.get_target_units() == {"IfcLengthMeasure": millimetre}
-
-    def test_untouched_fields_are_excluded(self):
-        ifc = self.setup_file()
-        metre = ifc.createIfcSIUnit(None, "LENGTHUNIT", None, "METRE")
-        millimetre = ifc.createIfcSIUnit(None, "LENGTHUNIT", "MILLI", "METRE")
-        gram = ifc.createIfcSIUnit(None, "MASSUNIT", None, "GRAM")
-        ifcopenshell.api.unit.assign_unit(ifc, units=[metre, gram])
-
-        props = tool.Qto.get_qto_props()
-        props.target_unit_length = str(millimetre.id())
-        props.target_unit_mass = "0"  # explicitly left at "Default"
-
-        assert subject.get_target_units() == {"IfcLengthMeasure": millimetre}
-
-
 class TestGetBaseQto(test.bim.bootstrap.NewFile):
     def test_run(self):
         ifc = ifcopenshell.file()

src/bonsai/test/tool/test_spatial.py

@@ -288,27 +288,3 @@ class TestGenerateSpace(NewFile):
             )
         )
         assert np.allclose(TEST_VERTS, sorted([tuple(v.co) for v in mesh.vertices]))
-
-    def test_regenerate_space_preserves_z_location(self):
-        bpy.ops.bim.create_project()
-        ifc = tool.Ifc.get()
-        scene = bpy.context.scene
-        product = ifcopenshell.api.root.create_entity(ifc, ifc_class="IfcWall")
-        bpy.ops.mesh.primitive_cube_add(size=10, location=(0, 0, 4))
-        obj = bpy.data.objects["Cube"]
-        scene.collection.objects.link(obj)
-        tool.Ifc.link(product, obj)
-        scene.cursor.location = (0, 0, 0)
-
-        bpy.ops.bim.generate_space()
-        space = bpy.data.objects["IfcSpace/Space"]
-        space.location.z = 5
-        bpy.context.view_layer.update()
-
-        bpy.context.view_layer.objects.active = space
-        space.select_set(True)
-        obj.select_set(False)
-
-        bpy.ops.bim.generate_space()
-
-        assert np.isclose(space.location.z, 5), f"Expected z=5, got {space.location.z}"