Skip to content

Use a fixed revision of OpenCOLLADA...#39

Merged
aothms merged 3 commits into
IfcOpenShell:masterfrom
Tridify:fixed_opencollada_revision
Mar 5, 2016
Merged

Use a fixed revision of OpenCOLLADA...#39
aothms merged 3 commits into
IfcOpenShell:masterfrom
Tridify:fixed_opencollada_revision

Conversation

@Stinkfist0

Copy link
Copy Markdown
Contributor

in order to prevent introducing breaking changes. Closes #36, related to #38

@Stinkfist0

Copy link
Copy Markdown
Contributor Author

The *nix build script seems to use pretty old revision - maybe good to update it to the same revision if possible?

@aothms

aothms commented Feb 27, 2016

Copy link
Copy Markdown
Member

This didn't update the repo on my local machine because it was already cloned. Can you do a git checkout in the ELSE branch as well?

@Stinkfist0

Copy link
Copy Markdown
Contributor Author

Ah, yes this assumed nuking the old clone completely.

…ision, so that users don't have the nuke their existing clones before running the script.
@Stinkfist0

Copy link
Copy Markdown
Contributor Author

Does this look OK now?

@aothms

aothms commented Mar 5, 2016

Copy link
Copy Markdown
Member

Sorry for the slow replies these days. Thanks again!

aothms added a commit that referenced this pull request Mar 5, 2016
Use a fixed revision of OpenCOLLADA...
@aothms aothms merged commit 82a1e33 into IfcOpenShell:master Mar 5, 2016
@Stinkfist0 Stinkfist0 deleted the fixed_opencollada_revision branch March 7, 2016 19:43
theoryshaw added a commit that referenced this pull request Jun 7, 2026
SetDimensionAnchor — hover-select-then-confirm:
- Cursor highlights candidate IFC elements (orange Blender selection outline)
  before committing; Tab cycles through overlapping/coplanar candidates
- _compute_candidates: ray-cast all IFC mesh objects; falls back to 2D
  bounding-box proximity (5 cm tolerance) for plan-view picks where the
  ray misses the mesh by sub-mm amounts
- _write_anchor: after anchoring a face, immediately calls
  regenerate_dimension with placement_override (Blender matrix_world)
  and _update_blender_curve so the curve vertex moves to the resolved point

DrawParametricDimension — ForcePerpendicularToFace live snap constraint:
- Reads force_perpendicular_to_face toggle from annotation props on invoke
- After anchor[0] is placed on a FACE, _update_perp_constraint extracts
  the face normal and stores it as the constraint axis
- _apply_perp_constraint runs every modal tick after handle_snap_selection,
  projecting the current snap point onto pt[0] + t*normal
- On finalize, _create_dimension_from_polyline writes ForcePerpendicularToFace
  to the BBIM_Dimension pset and calls regenerate_dimension to snap the
  stored curve to the constraint before the operator exits

regenerate_dimension.py:
- ForcePerpendicularToFace block: after resolving all anchors, projects
  vertices 1…n onto the line through pt[0] along anchor[0]'s face normal
- _get_anchor_face_normal_world: reads normal_local from anchor fingerprint,
  calls _rotate_local_to_world with placement_override; falls back to stored
  world-space normal

resolve_anchor.py:
- _rotate_local_to_world: transforms an element-local direction vector to
  world space using the element's placement or placement_override matrix

pset/operator.py:
- EditPset._execute: after editing a BBIM_Dimension pset on an IfcAnnotation,
  auto-calls regenerate_dimension + _update_blender_curve so changes to
  anchors/ForcePerpendicularToFace are reflected immediately in the viewport

prop.py / workspace.py:
- Added force_perpendicular_to_face BoolProperty to BIMAnnotationProperties
- UI toggle shown in annotation tool header for DIMENSION/RADIUS/DIAMETER/
  ANGLE/PLAN_LEVEL/SECTION_LEVEL types

Psets_BBIM_Annotation.ifc:
- Added ForcePerpendicularToFace property template (#39) to BBIM_Dimension
- Extended BBIM_Dimension applicability to ANGLE, PLAN_LEVEL, SECTION_LEVEL

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants