Skip to content

Add some rules for a tensorfree which also takes the tensor - #321

Merged
kshyatt merged 2 commits into
mainfrom
ksh/free
Oct 30, 2026
Merged

kshyatt merged 2 commits into
mainfrom
ksh/free

Conversation

@kshyatt

@kshyatt kshyatt commented Oct 30, 2026

Copy link
Copy Markdown
Member

We need this for the GPU allocators, otherwise Zygote tries to differentiate through them 😬

@kshyatt
kshyatt requested a review from lkdvos October 30, 2026 17:18
lkdvos
lkdvos previously approved these changes Oct 30, 2026
@kshyatt

kshyatt commented Oct 30, 2026

Copy link
Copy Markdown
Member Author

Going to force merge to get past the deep ROCm queue (what's the point of having the tests then...)

@kshyatt
kshyatt merged commit bac9993 into main Oct 30, 2026
13 of 14 checks passed
@kshyatt
kshyatt deleted the ksh/free branch October 30, 2026 21:14
@codecov

codecov Bot commented Oct 31, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
ext/TensorOperationsChainRulesCoreExt.jl 0.00% 4 Missing ⚠️
...orOperationsEnzymeExt/TensorOperationsEnzymeExt.jl 0.00% 1 Missing ⚠️
Files with missing lines Coverage Δ
...erationsMooncakeExt/TensorOperationsMooncakeExt.jl 96.63% <ø> (ø)
...orOperationsEnzymeExt/TensorOperationsEnzymeExt.jl 74.12% <0.00%> (-0.38%) ⬇️
ext/TensorOperationsChainRulesCoreExt.jl 80.68% <0.00%> (-2.85%) ⬇️

... and 5 files with indirect coverage changes

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

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