Repository navigation
Added option for MPO-MPS zipup - #97
Conversation
|
Sorry for the delay to move it here. |
|
@mtfishman If I resolve the merge conflicts can this be merged? |
Sure, though I have some review comments. Go ahead and fix the merge conflicts and then I'll do a review. |
|
Your PR no longer requires formatting changes. Thank you for your contribution! |
|
@mtfishman up to date and the tests are passing locally so I think it's good for your review. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #97 +/- ##
==========================================
+ Coverage 90.31% 90.33% +0.01%
==========================================
Files 54 54
Lines 3605 3610 +5
==========================================
+ Hits 3256 3261 +5
Misses 349 349
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Hi @corbett5 @mtfishman, is there any update on when this PR might be merged? |
Sorry this slipped through the cracks, I'll make a few tweaks to this and then merge it (today or tomorrow). |
|
Your PR no longer requires formatting changes. Thank you for your contribution! |
|
Sorry for the slow response on this @corbett5, thanks for the PR. I'll merge once tests finish running. |
|
@mtfishman, no worries. Thanks for finishing it off. |
Moved from ITensor/ITensors.jl#1536