Skip to content

add zero dim matmat/matvec-mul tests#646

Draft
tam724 wants to merge 2 commits into
JuliaGPU:mainfrom
tam724:test_zero_dim_matmat_matvec_edgecases
Draft

add zero dim matmat/matvec-mul tests#646
tam724 wants to merge 2 commits into
JuliaGPU:mainfrom
tam724:test_zero_dim_matmat_matvec_edgecases

Conversation

@tam724

@tam724 tam724 commented Nov 4, 2025

Copy link
Copy Markdown
Contributor

@tam724

tam724 commented Nov 4, 2025

Copy link
Copy Markdown
Contributor Author

Test failures are expected:
CUDA.jl requires JuliaGPU/CUDA.jl#2958 to pass. Metal.jl and oneAPI.jl need similar changes.

@kshyatt kshyatt force-pushed the test_zero_dim_matmat_matvec_edgecases branch from 44c8bdd to 7dcff50 Compare November 14, 2025 08:50
@kshyatt

kshyatt commented Nov 14, 2025

Copy link
Copy Markdown
Member

@tam724 now with the CUDA changes merged, is this ready to go?

@tam724

tam724 commented Nov 14, 2025

Copy link
Copy Markdown
Contributor Author

No, because it adds tests into the testsuite, merging this PR would break the CI for Metal.jl and oneAPI.jl.

@kshyatt

kshyatt commented Nov 14, 2025

Copy link
Copy Markdown
Member

Canceled the macOS build for this due to queue pressure and it's not ready anyway

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