Skip to content

Put the host's elements in FromRevit/HostModel when the model has links - #35

Merged
robDL89 merged 1 commit into
mainfrom
feature/host-model-collection
Nov 11, 2026
Merged

robDL89 merged 1 commit into
mainfrom
feature/host-model-collection

Conversation

@robDL89

@robDL89 robDL89 commented Nov 11, 2026

Copy link
Copy Markdown
Owner

In a model with linked models, Send Selection now puts the host's own elements in FromRevit/HostModel - <file name>, next to the LinkModel - <link name> collections.

  • Revit: revit_batch_begin carries an optional host_name (the document title) when the document contains at least one RevitLinkInstance. Decided on the model, not the selection.
  • Blender: new host elements are created in the HostModel sub-collection; existing ones move there only from the FromRevit root, so a collection the user chose is kept. Without host_name (no links, or an older add-in) nothing changes.
  • DESIGN.md, README and CHANGELOG (new Unreleased section, also listing the fixes from Fixes from the code review: connection state, bake, hardening #33).

Tests: pytest, dotnet test (net48 + net8.0), add-in build for Revit 2024 and 2025, Blender 5.2 smoke_pull with the four placement cases. Tried in Revit 2025 + Blender with a real linked model.

🤖 Generated with Claude Code

revit_batch_begin carries host_name, the document title, when the host
document contains a RevitLinkInstance. Blender then creates the host's
new elements in FromRevit/HostModel - <name>, next to the LinkModel
collections, and moves there the ones still in the FromRevit root; an
element the user moved elsewhere stays. Without links nothing changes.

CHANGELOG gets an Unreleased section with the fixes from #33 too.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@robDL89
robDL89 merged commit df5929d into main Nov 11, 2026
7 checks passed
@robDL89
robDL89 deleted the feature/host-model-collection branch November 11, 2026 17:15
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