From fb7aa40bf867975ba94ec07735ae465299fbb0c7 Mon Sep 17 00:00:00 2001 From: Petru Conduraru Date: Sat, 25 Jul 2026 09:35:53 +0300 Subject: [PATCH] ifcgeom: ignore BboxSubstitutionThreshold in the SWIG wrapper The new bbox-substitution-threshold setting struct was added to ConversionSettings.h without the matching %ignore entry that every other SettingBase derived setting struct has in IfcPython.i. Without it, SWIG generated a stray BboxSubstitutionThreshold class exposing just its name/description constants, which is not how any of the other ~50 setting structs are surfaced (they are all read and written through the generic Settings.get_/set_ string API instead) and which made ifcopenshell_wrapper.pyi drift from the compiled wrapper, failing test_validate_stub.py in CI. Adding the ignore entry, in the same position as the struct itself was inserted in ConversionSettings.h, restores parity with its siblings and needs no stub changes. Generated with the assistance of an AI coding tool. --- src/ifcwrap/IfcPython.i | 1 + 1 file changed, 1 insertion(+) diff --git a/src/ifcwrap/IfcPython.i b/src/ifcwrap/IfcPython.i index 8020ecf882..cbf95af0a9 100644 --- a/src/ifcwrap/IfcPython.i +++ b/src/ifcwrap/IfcPython.i @@ -148,6 +148,7 @@ %ignore CircleSegments; %ignore CgalSmoothAngleDegrees; %ignore KeepBoundingBoxes; +%ignore BboxSubstitutionThreshold; %ignore SurfaceColour; %ignore ComputeCurvature; %ignore FunctionStepType;