Skip to content

Trivial tensors fast path into TensorOperations machinery#463

Merged
lkdvos merged 2 commits into
mainfrom
lkdvos/trivial-tensorop-fastpath
Jun 29, 2026
Merged

Trivial tensors fast path into TensorOperations machinery#463
lkdvos merged 2 commits into
mainfrom
lkdvos/trivial-tensorop-fastpath

Conversation

@lkdvos

@lkdvos lkdvos commented Jun 25, 2026

Copy link
Copy Markdown
Member

This bypasses some of the unnecessary machinery for dense trivial tensors to hopefully improve compilation times by being able to re-use some of the precompiled things in TensorOperations.
I also found an allocator argument not being passed on to the permute call, so that should now also be fixed.

@lkdvos
lkdvos requested a review from Jutho June 25, 2026 14:21
@codecov

codecov Bot commented Jun 25, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.90909% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/tensors/tensoroperations.jl 90.00% 1 Missing ⚠️
Files with missing lines Coverage Δ
src/tensors/indexmanipulations.jl 88.06% <100.00%> (-0.05%) ⬇️
src/tensors/tensoroperations.jl 97.29% <90.00%> (-0.38%) ⬇️

... and 3 files with indirect coverage changes

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

@lkdvos
lkdvos requested a review from kshyatt June 25, 2026 18:21
Comment thread src/tensors/tensoroperations.jl Outdated
@lkdvos
lkdvos force-pushed the lkdvos/trivial-tensorop-fastpath branch from 04e417d to 0c21fe2 Compare June 28, 2026 11:10
@lkdvos
lkdvos force-pushed the lkdvos/trivial-tensorop-fastpath branch from 0c21fe2 to bb80f9d Compare June 28, 2026 11:10
@lkdvos
lkdvos requested a review from Jutho June 28, 2026 11:10
@lkdvos
lkdvos enabled auto-merge (squash) June 28, 2026 14:39
@lkdvos
lkdvos merged commit a50858d into main Jun 29, 2026
55 checks passed
@lkdvos
lkdvos deleted the lkdvos/trivial-tensorop-fastpath branch June 29, 2026 14:10
@lkdvos lkdvos mentioned this pull request Jul 13, 2026
lkdvos referenced this pull request Jul 15, 2026
* Update changelog for v0.17.1

* Bump version to v0.17.1

* Update CITATION.cff for v0.17.1

* Document #485 and MatrixAlgebraKit 0.6.9 bump in changelog

* Address review comments: clarify Enzyme index-manip mode, note cuTENSOR
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