brunopostle/simple-ifc
1
1
Fork 0
mirror of https://github.com/brunopostle/simple-ifc.git synced 2026-03-29 22:43:17 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix half-space clipping normal convention in docs

Browse source 
The clip normal must point toward the REMOVED material (not the kept
region). Correct the CLAUDE.md examples and note. Update workflow to
use new geometry.clip_solid API.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Bruno Postle 2026-03-12 21:27:57 +00:00
parent fe04095207
commit 7b15ba18b6
1 changed files with 12 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
CLAUDE.md
View file

@ -242,7 +242,7 @@ ifc_edit("geometry.regenerate_wall_representation", '{"wall": "<wall_id>", "leng
**CRITICAL: `regenerate_wall_representation` removes slope clippings.** Do not use it on walls with top-clipped (sloped) geometry. **CRITICAL: `regenerate_wall_representation` removes slope clippings.** Do not use it on walls with top-clipped (sloped) geometry.
**For sloped walls needing a mitre**, use `ifc_shape` to bake the 45° cut into the 2D profile as a trapezoid, then apply the slope as a single `IfcHalfSpaceSolid` via `geometry.add_boolean`: **For sloped walls needing a mitre**, use `ifc_shape` to bake the 45° cut into the 2D profile as a trapezoid, then apply the slope via `geometry.clip_solid`:
``` ```
# South porch wall: mitre cuts from (1.9,0) to (1.57,0.33) at east end # South porch wall: mitre cuts from (1.9,0) to (1.57,0.33) at east end
@ -254,12 +254,9 @@ ifc_shape("extrude", {"profile_or_curve": <profile_id>, "magnitude": 3.5, "posit
# → extrusion_id # → extrusion_id
ifc_shape("get_representation", {"context": 11, "items": [<extrusion_id>]}) # → rep_id (SweptSolid) ifc_shape("get_representation", {"context": 11, "items": [<extrusion_id>]}) # → rep_id (SweptSolid)
# Create the slope half-space (normal points INTO the kept region below the slope) # Clip to slope — normal points toward the REMOVED material (above slope = positive Z)
ifc_shape("plane", {"location": [0,0,3.26], "normal": [-0.419, 0, -0.908]}) # → plane_id ifc_edit("geometry.clip_solid", '{"item": "<extrusion_id>", "location": [0,0,3.26], "normal": [0.419,0,0.908]}')
ifc_shape("half_space_solid", {"plane": <plane_id>, "agreement_flag": false}) # → hss_id # → BCR id; extrusion_id is now part of the BCR, not the rep top-level
# Apply slope as boolean — moves extrusion out of rep top-level, adds BCR in its place
ifc_edit("geometry.add_boolean", '{"first_item": "<extrusion_id>", "second_items": "<hss_id>"}')
ifc_edit("attribute.edit_attributes", '{"product": "<rep_id>", "attributes": "{\"RepresentationType\": \"Clipping\"}"}') ifc_edit("attribute.edit_attributes", '{"product": "<rep_id>", "attributes": "{\"RepresentationType\": \"Clipping\"}"}')
# Swap body rep # Swap body rep
@ -274,9 +271,14 @@ ifc_edit("geometry.assign_representation", '{"product": "<wall_id>", "representa
- West-end mitre: profile points `(0,0)→(L,0)→(L,t)→(t,t)` (removes NW inner corner, closing from `(t,t)` to `(0,0)`) - West-end mitre: profile points `(0,0)→(L,0)→(L,t)→(t,t)` (removes NW inner corner, closing from `(t,t)` to `(0,0)`)
- For walls with origin at the mitre end: profile points `(0,0)→(L,0)→(L,t)→(t,t)` where x=0 is the mitre end - For walls with origin at the mitre end: profile points `(0,0)→(L,0)→(L,t)→(t,t)` where x=0 is the mitre end
**Slope normal** must point INTO the kept region (below slope = negative Z component): **Slope normal for `geometry.clip_solid`** must point INTO the REMOVED region (above the slope = positive Z component). This is the same convention as the `clippings` parameter of `add_wall_representation`.
- Slope rising west→east: normal `[+0.419, 0, −0.908]` at location `[0, 0, z_at_x0]`
- Slope falling west→east: normal `[−0.419, 0, −0.908]` at location `[0, 0, z_at_x0]` > ⚠️ This convention is counter-intuitive: **normal → removed material** (not toward the kept material).
- Slope falling west→east (high at x=0): normal `[+0.419, 0, +0.908]` at location `[0, 0, z_high]`
- Slope rising west→east (low at x=0): normal `[−0.419, 0, +0.908]` at location `[0, 0, z_low]`
The Z component is always **positive** (pointing upward, toward the removed material above the slope).
**Do NOT use dual `add_wall_representation` clippings** (two half-spaces: slope + mitre) — this produces incorrect geometry in viewers even though the IFC math is sound. **Do NOT use dual `add_wall_representation` clippings** (two half-spaces: slope + mitre) — this produces incorrect geometry in viewers even though the IFC math is sound.