Skip to content

Use maximum rather than norm - #280

Merged
kshyatt merged 2 commits into
mainfrom
ksh/gpu_norm
Sep 16, 2026
Merged

kshyatt merged 2 commits into
mainfrom
ksh/gpu_norm

Conversation

@kshyatt

@kshyatt kshyatt commented Sep 16, 2026

Copy link
Copy Markdown
Member

To bypass norm_recursive_check which is getting called because these are SubArray{T, <:GPUArray}

To bypass `norm_recursive_check` which is getting called because these are `SubArray{T, <:GPUArray}`
@kshyatt
kshyatt requested a review from lkdvos September 16, 2026 13:40
@kshyatt
kshyatt enabled auto-merge (squash) September 16, 2026 13:55
@kshyatt
kshyatt merged commit 9cd1f86 into main Sep 16, 2026
89 of 91 checks passed
@kshyatt
kshyatt deleted the ksh/gpu_norm branch September 16, 2026 21:39
@codecov

codecov Bot commented Sep 16, 2026

Copy link
Copy Markdown

Codecov Report

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

Files with missing lines Patch % Lines
src/pullbacks/svd.jl 0.00% 2 Missing ⚠️
Files with missing lines Coverage Δ
src/pullbacks/svd.jl 93.36% <0.00%> (ø)

... and 2 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.

3 participants