build(deps): bump docker/login-action from 4 to 4.5.2

Bumps [docker/login-action](https://github.com/docker/login-action) from 4 to 4.5.2. - [Release notes](https://github.com/docker/login-action/releases) - [Commits](https://github.com/docker/login-action/compare/v4...v4.5.2) --- updated-dependencies: - dependency-name: docker/login-action dependency-version: 4.5.2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
build(deps): bump actions/setup-python from 6 to 7
2026-08-06 07:51:47 +00:00 · 2026-07-31 22:44:19 +00:00 · 2026-07-31 13:57:34 +02:00 · 2026-07-31 12:49:37 +02:00 · 2026-07-31 10:38:58 +02:00 · 2026-07-31 10:36:39 +02:00
35 changed files with 80 additions and 41 deletions
@@ -20,7 +20,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7 # https://github.com/actions/checkout
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # 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
@@ -66,7 +66,7 @@ jobs:
              short_name: macos
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
        with:
          architecture: 'x64' # optional x64 or x86. Defaults to x64 if not specified
          python-version: '3.11'
@@ -49,7 +49,7 @@ jobs:
              short_name: macos
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
        with:
          architecture: 'x64' # optional x64 or x86. Defaults to x64 if not specified
          python-version: '3.11'
@@ -19,7 +19,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # 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
@@ -19,7 +19,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # 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
@@ -19,7 +19,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # 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
@@ -19,7 +19,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # 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
@@ -19,7 +19,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # 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
@@ -38,7 +38,7 @@ jobs:
          }
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # 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
@@ -19,7 +19,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # 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
@@ -19,7 +19,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # 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
@@ -19,7 +19,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6
+      - uses: actions/setup-python@v7
        with:
          python-version: '3.11'
      - name: Compile
@@ -19,7 +19,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # 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
@@ -19,7 +19,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6
+      - uses: actions/setup-python@v7
        with:
          python-version: '3.11'
      - name: Compile
@@ -104,7 +104,7 @@ jobs:
      uses: docker/setup-buildx-action@v4        
    - 
      name: Login to Dockerhub
-      uses: docker/login-action@v4 
+      uses: docker/login-action@v4.5.2 
      with:
        username: aecgeeks
        password: ${{ secrets.DOCKER_HUB_TOKEN }}
@@ -50,7 +50,7 @@ jobs:
      - uses: actions/checkout@v7
        with:
          submodules: recursive
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # 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
@@ -41,7 +41,7 @@ jobs:
      - uses: actions/checkout@v7
        with:
          submodules: recursive
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # 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
@@ -19,7 +19,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # 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
@@ -19,7 +19,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6
+      - uses: actions/setup-python@v7
        with:
          python-version: '3.11'
      - name: Compile
@@ -26,7 +26,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
        with:
          architecture: 'x64' # optional x64 or x86. Defaults to x64 if not specified
          python-version: '3.11'
@@ -20,7 +20,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # https://github.com/actions/setup-python
        with:
          architecture: 'x64' # optional x64 or x86. Defaults to x64 if not specified
          python-version: '3.11'
@@ -19,7 +19,7 @@ jobs:
      fail-fast: false
    steps:
      - uses: actions/checkout@v7
-      - uses: actions/setup-python@v6 # https://github.com/actions/setup-python
+      - uses: actions/setup-python@v7 # 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
@@ -85,7 +85,7 @@ jobs:
          cmake --build build-ifcopenshell --target install -j "$(nproc)"

      - name: Set up Python 3.11
-        uses: actions/setup-python@v6
+        uses: actions/setup-python@v7
        with:
          python-version: 3.11

@@ -120,7 +120,7 @@ jobs:
          PY

      - name: Set up Python 3.12
-        uses: actions/setup-python@v6
+        uses: actions/setup-python@v7
        with:
          python-version: 3.12

@@ -15,12 +15,12 @@ jobs:
        uses: actions/checkout@v7

      - name: Action - install python
-        uses: actions/setup-python@v6
+        uses: actions/setup-python@v7
        with:
          python-version: ${{ env.MIN_IOS_PY_VERSION }}

      - name: Action - install python
-        uses: actions/setup-python@v6
+        uses: actions/setup-python@v7
        with:
          python-version: ${{ env.MIN_BLENDER_PY_VERSION }}

@@ -48,7 +48,7 @@ jobs:
            submodules: recursive

      - name: Set up Python
-        uses: actions/setup-python@v6
+        uses: actions/setup-python@v7
        with:
          python-version: 3.11

@@ -263,7 +263,9 @@ jobs:
          cd ../ifcquery && make test || ERROR=1
          pip install -e ../ifcedit --no-deps
          cd ../ifcedit && make test || ERROR=1
-          pip install mcp
+          # 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 -e ../ifcmcp --no-deps
          cd ../ifcmcp && make test || ERROR=1
          pip install -e ../ifctester --no-deps
@@ -14,7 +14,7 @@ jobs:
        uses: actions/checkout@v7

      - name: Set up Python
-        uses: actions/setup-python@v6
+        uses: actions/setup-python@v7
        with:
          python-version: '3.x'

@@ -42,7 +42,7 @@ jobs:
        run: |
          rsync -av --delete --exclude='.git/' src/ifcchat/ output/
      - name: Setup Python
-        uses: actions/setup-python@v6
+        uses: actions/setup-python@v7
        with:
          python-version: "3.x"
      - name: Download wheels
@@ -13,6 +13,14 @@ namespace ifcopenshell {
 			profile_point(const std::array<double, 2>& p, const boost::optional<double>& r = boost::none)
 				: xy(p), radius(r) {
 			}
+
+			// Recent Boost makes optional's converting constructor explicit,
+			// and an explicit constructor cannot be used in copy-initialization
+			// - which is what `{{x, y}, {radius}}` in the profile mappings is.
+			// Taking the double directly keeps every call site working.
+			profile_point(const std::array<double, 2>& p, double r)
+				: xy(p), radius(r) {
+			}
 		};

 		struct profile_point_with_edges {
@@ -9,7 +9,7 @@ from ifcmcp.core import IfcSession
 try:
    from mcp.server.fastmcp import FastMCP  # type: ignore
    from mcp.types import ImageContent  # type: ignore
-except Exception:  # pragma: no cover
+except ImportError:  # pragma: no cover
    FastMCP = None  # type: ignore
    ImageContent = None  # type: ignore

@@ -18,7 +18,9 @@ classifiers = [
 dependencies = ["ifcopenshell", "ifcquery", "ifcedit"]

 [project.optional-dependencies]
-mcp = ["mcp"]
+# Pinned <2: mcp 2.0.0 renamed mcp.server.fastmcp.FastMCP to
+# mcp.server.mcpserver.MCPServer, which this package doesn't support yet.
+mcp = ["mcp>=1.0,<2"]

 [project.scripts]
 ifcmcp = "ifcmcp.__main__:main"
@@ -60,7 +60,7 @@ operating systems. GCC (4.7 or newer) or Clang (any version) is required.

   .. code-block:: bash

-       sudo apt-get install git cmake gcc g++ libboost-all-dev libcgal-dev
+       sudo apt-get install git cmake gcc g++ libboost-all-dev libcgal-dev libeigen3-dev

   The CGAL version that ships with Ubuntu 20.04 is too old. Users on Ubuntu 20.04 are advised to manually install CGAL 5.3.

@@ -64,7 +64,10 @@ DEFAULTS = {
    "project_globalid": lambda d: compress(uuid.uuid4().hex),
    "schema_identifier": lambda d: "IFC4",
    "timestamp": lambda d: int(time.time()),
-    "timestring": lambda d: time.strftime("%Y-%m-%dT%H:%M:%S", time.gmtime(d.get("timestamp") or time.time())),
+    "timestring": lambda d: time.strftime(
+        "%Y-%m-%dT%H:%M:%S",
+        time.gmtime(d["timestamp"] if d.get("timestamp") is not None else time.time()),
+    ),
    "mvd": lambda d: (
        "ReferenceView_V1.2"
        if d.get("schema_identifier") == "IFC4"
@@ -0,0 +1,7 @@
+def pytest_addoption(parser):
+    parser.addoption(
+        "--rule",
+        action="store",
+        default=None,
+        help="Only run test_rules.py fixtures whose filename contains this substring.",
+    )
@@ -1,6 +1,5 @@
 import glob
 import os
-import sys

 import pytest
 import tabulate
@@ -9,14 +8,18 @@ import ifcopenshell.express.rule_executor
 import ifcopenshell.validate


-@pytest.mark.parametrize(
-    "filename",
-    [
+def pytest_generate_tests(metafunc):
+    if "filename" not in metafunc.fixturenames:
+        return
+    rule = metafunc.config.getoption("--rule")
+    filenames = [
        fn
        for fn in glob.glob(os.path.join(os.path.dirname(__file__), "fixtures/rules/*.ifc"))
-        if len(sys.argv) < 2 or sys.argv[1] in os.path.basename(fn)
-    ],
-)
+        if not rule or rule in os.path.basename(fn)
+    ]
+    metafunc.parametrize("filename", filenames, ids=[os.path.basename(fn) for fn in filenames])
+
+
 def test_file(filename):
    base = os.path.basename(filename)
    file = ifcopenshell.open(filename)
@@ -46,6 +46,13 @@ struct FullBufferImpl final : FileReader::Impl {
 #else
        auto stream = fopen(fn.c_str(), "rb");
 #endif
+        if (stream == nullptr) {
+            // Missing or unreadable file. Leave the buffer empty so the
+            // caller sees a zero-length input and reports a read error;
+            // handing a null FILE* to the CRT below terminates the whole
+            // process instead of failing the parse.
+            return;
+        }
        fseek(stream, 0, SEEK_END);
        buf_.resize((size_t)ftell(stream));
        rewind(stream);
@@ -84,6 +91,13 @@ struct PagedFileImpl final : FileReader::Impl {
 #else
        fp_ = fopen(fn.c_str(), "rb");
 #endif
+        if (fp_ == nullptr) {
+            // As above: behave like an empty file rather than passing a
+            // null FILE* to fseek. get() then throws out_of_range for any
+            // position and fetchPage_() is never reached.
+            file_size_ = 0;
+            return;
+        }
        fseek(fp_, 0, SEEK_END);
        file_size_ = (size_t)ftell(fp_);
        rewind(fp_);
Author	SHA1	Message	Date
dependabot[bot]	09ceb64fff	build(deps): bump docker/login-action from 4 to 4.5.2 Bumps [docker/login-action](https://github.com/docker/login-action) from 4 to 4.5.2. - [Release notes](https://github.com/docker/login-action/releases) - [Commits](https://github.com/docker/login-action/compare/v4...v4.5.2) --- updated-dependencies: - dependency-name: docker/login-action dependency-version: 4.5.2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>	2026-07-31 22:44:19 +00:00
dependabot[bot]	a11ebdf8c4	build(deps): bump actions/setup-python from 6 to 7 Bumps [actions/setup-python](https://github.com/actions/setup-python) from 6 to 7. - [Release notes](https://github.com/actions/setup-python/releases) - [Commits](https://github.com/actions/setup-python/compare/v6...v7) --- updated-dependencies: - dependency-name: actions/setup-python dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>	2026-07-31 13:57:34 +02:00
Bartok	b997726564	docs: add Eigen to Linux install deps Linux "basic dependencies" omitted libeigen3-dev even though ifcgeom requires Eigen3 (find_package Eigen3 REQUIRED) and the cmake snippet already passes -DEIGEN_DIR=/usr/include/eigen3. macOS Homebrew line already installs eigen. Closes #6903 Generated with the assistance of an AI coding tool.	2026-07-31 12:49:37 +02:00
Petru Conduraru	25713a486a	Fix test_rules.py filtering by sys.argv, which empties the corpus under pytest test_file's parametrize list was filtered with `sys.argv[1] in os.path.basename(fn)`, reading the raw process argv instead of a pytest-native option. Under a bare `pytest` invocation sys.argv[1] is pytest's own first CLI token, never a match, so the 138-fixture EXPRESS rule corpus in test/fixtures/rules collapses to an empty parametrize and pytest reports it as a single skipped test rather than an error. Under CI's actual invocation (pytest -p no:pytest-blender -n $NPROCS test ...) sys.argv[1] is "-p", which happens to substring-match 47 of the 138 fixtures, so CI has been silently running a coincidental 34% slice of the corpus with no signal anything was wrong. Replaced the module-level list comprehension with a pytest_generate_tests hook plus a --rule CLI option (added via a new test/conftest.py). This runs the full corpus by default under any pytest invocation, still allows filtering to one rule for local debugging via --rule, and no longer collides with pytest's own argv. Verified all 138 fixtures collect and pass under the fixed harness (63 fail- fixtures each raise a violation, 75 pass- fixtures raise none). Generated with the assistance of an AI coding tool.	2026-07-31 10:38:58 +02:00
Petru Conduraru	d3b6b82151	ifcmcp: pin mcp below 2.0 to fix broken FastMCP import mcp 2.0.0 (unpinned in CI and in the ifcmcp[mcp] extra) renamed mcp.server.fastmcp.FastMCP to mcp.server.mcpserver.MCPServer, which ifcmcp does not support yet. server.py caught the resulting ModuleNotFoundError with a bare except Exception and silently reported it as FastMCP not installed, masking the real breakage until the ifcmcp test suite failed in CI. Pinned mcp to >=1.0,<2 in both ci.yml and ifcmcp's pyproject.toml mcp extra, confirmed the full ifcmcp test suite (70 tests) passes against mcp 1.29.0, and confirmed the genuinely-not-installed path still raises the expected ImportError. Also narrowed the except clause to ImportError only so an unrelated future bug in that import block surfaces instead of being swallowed as "not installed". Generated with the assistance of an AI coding tool.	2026-07-31 10:36:39 +02:00
yekose	9e6797e172	ifcparse: check the result of fopen before using the FILE* FullBufferImpl and PagedFileImpl both open the file and then use the handle without ever testing it: auto stream = _wfopen(fn_wide, L"rb"); // null when the file is missing fseek(stream, 0, SEEK_END); // null goes straight to the CRT buf_.resize((size_t)ftell(stream)); Opening a path that does not exist therefore hands a null FILE* to the CRT. On MSVC that does not return an error: the runtime terminates the process immediately (fastfail, exit code 0xC0000409). No exception is thrown, no stack unwinding starts, so a caller cannot defend with try/catch — the host application simply dies. On glibc it is undefined behaviour as well. This is reachable through the ordinary entry point, because guess_file_type() answers FT_IFCSPF for a path that does not exist (its own comment calls this "just weird, but for consistency with earlier behaviour"), so a missing path flows into the reader rather than being reported. The fix is to leave the reader empty when the open fails. Both implementations then behave like a zero-length file: size() is 0 and get() throws out_of_range for any position, so the parse fails and IfcFile::good() reports it, which is what a caller can actually handle. PagedFileImpl's destructor already tested fp_ for null, so the possibility was known — only the constructor did not check. Verified by reading a non-existent path through IfcParse::IfcFile: the constructor returns and good() reports the failure, where before the process died with 0xC0000409 and no output.	2026-07-31 10:33:04 +02:00
Petru Conduraru	1f9a0a53bb	ifcopenshell.template: fix timestring ignoring an explicit timestamp of 0 create(timestamp=0) computed the FILE_NAME timestring with `d.get("timestamp") or time.time()`, which treats 0 (a legitimate epoch timestamp) as unset because 0 is falsy. The header ended up with the current wall-clock time in FILE_NAME while IFCOWNERHISTORY correctly stored CreationDate=0, an inconsistent pair of dates in the same file. Switched to an explicit None check so an explicit timestamp of 0 is honoured the same way any other explicit timestamp is. Generated with the assistance of an AI coding tool.	2026-07-31 10:19:33 +02:00
yekose	b82c4c53fe	ifcgeom: add a profile_point overload taking a plain double The profile mapping builds its points as profile_helper(m4, { {{-x, -y}, {f2}}, ... where `f2` is a `double` and profile_point's second member is a `boost::optional<double>`. In recent Boost (somewhere between 1.85 and 1.91) optional's converting constructor became explicit, and an explicit constructor cannot be used in copy-initialization — which is what a braced element is. So every one of these call sites stops compiling: MSVC 19.4x: error C2664: cannot convert argument 2 from 'initializer list' to 'const std::vector<profile_point>&' clang-cl 22: error: chosen constructor is explicit in copy-initialization Twelve translation units are affected (IfcCShapeProfileDef, IfcIShapeProfileDef, IfcLShapeProfileDef, IfcTShapeProfileDef, IfcUShapeProfileDef, IfcZShapeProfileDef, IfcAsymmetricIShapeProfileDef, IfcCraneRailAShapeProfileDef, IfcRectangleProfileDef, IfcRectangleHollowProfileDef, IfcRoundedRectangleProfileDef, IfcTrapeziumProfileDef), roughly 100 call sites in total. Adding one overload that takes the double directly fixes all of them without touching a single call site, and changes nothing for existing code: the optional overload still wins wherever an optional is passed. Verified by building schemas 2x3;4;4x3_add2 with MSVC 2022 against Boost 1.91 and OCCT 7.9.3 — IfcParse, IfcGeom, the schema mappings and geometry_kernel_opencascade all archive cleanly. Without this, the same build against Boost 1.85 succeeds, which is what identified Boost as the variable.	2026-07-31 10:14:16 +02:00