Skip to content

Optional MetadataStore deep copy when cloning#354

Open
AyoubMDL wants to merge 5 commits intoonnx:mainfrom
AyoubMDL:deep_copy
Open

Optional MetadataStore deep copy when cloning#354
AyoubMDL wants to merge 5 commits intoonnx:mainfrom
AyoubMDL:deep_copy

Conversation

@AyoubMDL
Copy link
Copy Markdown

This PR introduces an optional deep copy when cloning for objects stored in the metadata store (addresses #333).

Signed-off-by: AyoubMDL <ayoubbih119@gmail.com>
Signed-off-by: AyoubMDL <ayoubbih119@gmail.com>
@justinchuby justinchuby added this to the 0.2.1 milestone Apr 2, 2026
@justinchuby justinchuby self-assigned this Apr 3, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 8, 2026

Codecov Report

❌ Patch coverage is 96.77419% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 80.09%. Comparing base (8b4620c) to head (ba6b068).

Files with missing lines Patch % Lines
src/onnx_ir/_cloner.py 95.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #354      +/-   ##
==========================================
+ Coverage   80.06%   80.09%   +0.02%     
==========================================
  Files          52       52              
  Lines        6396     6405       +9     
  Branches     1294     1296       +2     
==========================================
+ Hits         5121     5130       +9     
  Misses        912      912              
  Partials      363      363              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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