Closed Jutho closed 1 year ago
Ok, I am merging it like this. The contract!
implementation needs a revision (listed as TODO), in order to be able to use different backends. The plan would be:
sectortype isa Trivial
, simply pas tensors .data
to TO.tensorcontract!
mul!
implementation, but call TO.tensorcontract!
on the matrix blocks instead of LinearAlgebra.mul!
. TO.tensorcontract!
has a dedicated implementation for the case where everything are matrices, but this way one could still have the backend select different matrix multiplication algorithms (e.g. BLAS vs Octavian vs ...)
Codecov Report
91.42% <ø> (ø)
95.67% <0.00%> (ø)
75.72% <0.00%> (ø)
19.88% <36.36%> (+19.88%)
40.00% <40.00%> (ø)
71.66% <66.66%> (+71.66%)
72.09% <72.54%> (+6.37%)
72.72% <77.77%> (+72.72%)
61.03% <90.00%> (+61.03%)
89.15% <90.90%> (+89.15%)
... and 5 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more