Skip to content

Add GPU-safe index selection for eig/eigh pullbacks - #273

Merged
kshyatt merged 1 commit into
mainfrom
ksh/select-indices
Aug 22, 2026
Merged

Add GPU-safe index selection for eig/eigh pullbacks#273
kshyatt merged 1 commit into
mainfrom
ksh/select-indices

Conversation

@kshyatt

@kshyatt kshyatt commented Aug 18, 2026

Copy link
Copy Markdown
Member

Let's see if this removes the need to copy back for the PB steps

@kshyatt
kshyatt force-pushed the ksh/select-indices branch from 29dc1a7 to b3a1b30 Compare August 18, 2026 11:25
@kshyatt

kshyatt commented Aug 18, 2026

Copy link
Copy Markdown
Member Author

Blocked on #272 but this seems to work!!!

@kshyatt
kshyatt marked this pull request as ready for review August 18, 2026 12:44
@kshyatt
kshyatt requested a review from lkdvos August 18, 2026 12:44
@kshyatt
kshyatt force-pushed the ksh/select-indices branch from b3a1b30 to 0b13690 Compare August 19, 2026 13:06
@kshyatt
kshyatt force-pushed the ksh/select-indices branch from 0b13690 to e48ab96 Compare August 20, 2026 12:18
@Jutho

Jutho commented Aug 20, 2026

Copy link
Copy Markdown
Member

This looks like a nice solution.

@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
...MatrixAlgebraKitCUDAExt/MatrixAlgebraKitCUDAExt.jl 80.95% <ø> (-1.14%) ⬇️
src/common/pullbacks.jl 100.00% <100.00%> (ø)
src/pullbacks/eig.jl 92.72% <100.00%> (ø)
src/pullbacks/eigh.jl 86.13% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kshyatt
kshyatt merged commit 1544237 into main Aug 22, 2026
89 of 92 checks passed
@kshyatt
kshyatt deleted the ksh/select-indices branch August 22, 2026 06:30
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