Closed willow-ahrens closed 4 months ago
Attention: Patch coverage is 0%
with 28 lines
in your changes are missing coverage. Please review.
Project coverage is 75.94%. Comparing base (
e500926
) to head (1a74a3b
).:exclamation: Current head 1a74a3b differs from pull request most recent head 3c0bfa9. Consider uploading reports for the commit 3c0bfa9 to get more accurate results
Files | Patch % | Lines |
---|---|---|
src/interface/einsum.jl | 0.00% | 28 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@kylebd99 it looks like this uncovered a few bugs, but otherwise what do you think of my current approach?
Draft einsum proposal. Note, the current implementation does not support in-place operations because Finch High-level AST does not support in-place operations.