Skip to content

Conversation

@mtfishman
Copy link
Member

@mtfishman mtfishman commented Dec 4, 2025

This upgrades to TensorAlgebra v0.6.2 (ITensor/TensorAlgebra.jl#101, ITensor/TensorAlgebra.jl#103), which made breaking changes to the matricize interface.

@codecov
Copy link

codecov bot commented Dec 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.62%. Comparing base (8173240) to head (453a4c0).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #83      +/-   ##
==========================================
+ Coverage   93.55%   93.62%   +0.07%     
==========================================
  Files          12       12              
  Lines         915      926      +11     
==========================================
+ Hits          856      867      +11     
  Misses         59       59              
Flag Coverage Δ
docs 0.00% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mtfishman mtfishman closed this Dec 5, 2025
@mtfishman mtfishman reopened this Dec 5, 2025
@mtfishman
Copy link
Member Author

@lkdvos is it alright with you if I merge this? It will probably cause conflicts for #79 but I think they shouldn't be too hard to deal with and I need to merge this in order to update other packages to TensorAlgebra v0.6.

@lkdvos
Copy link
Contributor

lkdvos commented Dec 5, 2025

Yes, no problem! apologies for the slow reply, I got slightly sidetracked on this. I'll deal with the merge, I agree it shouldn't be too hard.

@mtfishman
Copy link
Member Author

mtfishman commented Dec 5, 2025

Yes, no problem! apologies for the slow reply, I got slightly sidetracked on this. I'll deal with the merge, I agree it shouldn't be too hard.

Thanks. While upgrading some other packages like KroneckerArrays.jl I realized there were a few things off with the new matricization interface so I'm tweaking it now, which might affect this PR (but after that is fixed I'll merge this).

@mtfishman mtfishman closed this Dec 5, 2025
@mtfishman mtfishman reopened this Dec 5, 2025
@mtfishman mtfishman merged commit 722a24a into main Dec 5, 2025
32 checks passed
@mtfishman mtfishman deleted the mf/tensoralgebra-v0.6 branch December 5, 2025 21:56
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.

3 participants