Skip to content

Make svd_trunc pullback GPU friendly - #271

Merged
kshyatt merged 4 commits into
mainfrom
ksh/svd_trunc_gpu
Aug 20, 2026
Merged

Make svd_trunc pullback GPU friendly#271
kshyatt merged 4 commits into
mainfrom
ksh/svd_trunc_gpu

Conversation

@kshyatt

@kshyatt kshyatt commented Aug 17, 2026

Copy link
Copy Markdown
Member

Otherwise this hits scalar indexing

@kshyatt
kshyatt requested a review from lkdvos August 17, 2026 13:20
Comment thread src/pullbacks/svd.jl Outdated
@kshyatt
kshyatt force-pushed the ksh/svd_trunc_gpu branch from 78ba59a to 702bccc Compare August 19, 2026 13:07
Comment thread src/pullbacks/svd.jl Outdated
kshyatt and others added 2 commits August 19, 2026 15:10
Co-authored-by: Lukas Devos <ldevos98@gmail.com>
@kshyatt

kshyatt commented Aug 20, 2026

Copy link
Copy Markdown
Member Author

GPUArrays released a new branch, let me quickly rerun the failing tests and see if they resolve?

@lkdvos

lkdvos commented Aug 20, 2026

Copy link
Copy Markdown
Member

Am I reading this correctly that the failing tests are not related to this PR, and everything works, or should we still wait for a rerun of these tests?

@kshyatt

kshyatt commented Aug 20, 2026

Copy link
Copy Markdown
Member Author

I think you're reading it correctly indeed, sorry, this fell off my radar

@kshyatt
kshyatt merged commit 9d021d2 into main Aug 20, 2026
46 of 47 checks passed
@kshyatt
kshyatt deleted the ksh/svd_trunc_gpu branch August 20, 2026 12:18
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