Skip to content

Add share_template_references! hook to the template deep copy - #164

Merged
jd-lara merged 1 commit into
mainfrom
jd/share-template-references
Sep 9, 2026
Merged

jd-lara merged 1 commit into
mainfrom
jd/share-template-references

Conversation

@jd-lara

@jd-lara jd-lara commented Sep 9, 2026

Copy link
Copy Markdown
Member

Template types can share objects with the caller's original by reference after the build-time copy: objects a build mutates that callers then inspect on the instance they passed in. The default shares nothing. The network-data sharing that must happen before deepcopy is unchanged, now under _deepcopy_sharing_network_data.

Template types can share objects with the caller's original by reference
after the build-time copy: objects a build mutates that callers then
inspect on the instance they passed in. The default shares nothing. The
network-data sharing that must happen before deepcopy is unchanged, now
under _deepcopy_sharing_network_data.
@jd-lara
jd-lara requested a review from luke-kiernan September 9, 2026 18:32
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Performance Results
Main


This branch


@codecov

codecov Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@luke-kiernan luke-kiernan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks simple enough.

@jd-lara
jd-lara merged commit a237c18 into main Sep 9, 2026
8 checks passed
rodrigomha added a commit that referenced this pull request Sep 14, 2026
Brings #164 (the share_template_references! hook), #165 (the ComponentBaseUnit rename) and
#166 (cost conversions derive their ratio to the system base; the coverage tooling leaves
the test environment).
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