Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CLIENTS]fix cuda_runtime not found and assert len(MARKERS) >= len(y_keys[index]) #535

Merged
merged 1 commit into from
Mar 13, 2024

Conversation

engineer1109
Copy link
Contributor

When compile with

cmake .. -DCLIENTS=ON -DCUBLAS=ON
make clblast_client_xgemm

And Run

python3 ../scripts/benchmark/benchmark.py  -p 0 -d 0 -b gemm -c clBLAS CPU-BLAS cuBLAS

Will meet 2 bugs:

  1. Header cuda_runtime.h not found
    add ${CUBLAS_INCLUDE_DIRS} to REF_INCLUDES to fix this.

  2. assert len(MARKERS) >= len(y_keys[index])

python3 ../scripts/benchmark/benchmark.py  -p 0 -d 0 -b gemm -c clBLAS CPU-BLAS cuBLAS

The cuBLAS lacks the MARKERS.
Changed:
MARKERS = ["o-", "x-", ".-", "--"]
to give cuBLAS the markers.

@engineer1109
Copy link
Contributor Author

@CNugteren BUGS with CLIENTS benchmark small fix.

@CNugteren CNugteren self-assigned this Mar 13, 2024
@CNugteren CNugteren enabled auto-merge (squash) March 13, 2024 19:11
@CNugteren CNugteren merged commit e320c20 into CNugteren:master Mar 13, 2024
5 checks passed
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