Skip to content

Support for Diagonal in orthnull#144

Open
kshyatt wants to merge 1 commit intomainfrom
orthnull-diag
Open

Support for Diagonal in orthnull#144
kshyatt wants to merge 1 commit intomainfrom
orthnull-diag

Conversation

@kshyatt
Copy link
Member

@kshyatt kshyatt commented Jan 6, 2026

Fixes up some missing methods. Needs #128 first.

@kshyatt kshyatt requested a review from lkdvos January 6, 2026 10:16
@kshyatt kshyatt force-pushed the orthnull-diag branch 3 times, most recently from 866b350 to d07f57d Compare January 8, 2026 12:33
@kshyatt
Copy link
Member Author

kshyatt commented Mar 19, 2026

OK, I think we finally got it lol. Passed locally on an AMDGPU for me.

@kshyatt kshyatt marked this pull request as ready for review March 19, 2026 14:12
@codecov
Copy link

codecov bot commented Mar 19, 2026

Codecov Report

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

Files with missing lines Patch % Lines
src/interface/orthnull.jl 50.00% 2 Missing ⚠️
Files with missing lines Coverage Δ
src/interface/orthnull.jl 69.44% <50.00%> (-1.15%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kshyatt
Copy link
Member Author

kshyatt commented Mar 19, 2026

GPU test fail here is the CUDA mempool thing unrelated to us

@kshyatt kshyatt requested a review from Jutho March 19, 2026 18:39
@Jutho
Copy link
Member

Jutho commented Mar 20, 2026

Can you rebase; it now shows all the changes from #128 as well, which makes it hard to see if anything was changed there.

@kshyatt
Copy link
Member Author

kshyatt commented Mar 20, 2026

All rebased!

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