Update the error message for cublas version check#2843
Conversation
Signed-off-by: Xin Yao <xiny@nvidia.com>
Greptile SummaryThis PR corrects five error messages in the Confidence Score: 5/5Safe to merge — pure error message correction with no logic or behavioral changes. All changes are string literals inside compile-time fallback stubs. No logic, API, or behavior is altered. The updated wording is factually accurate per cuBLAS's independent versioning scheme. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Compile-time check] --> B{CUBLAS_VERSION >= 130300?}
B -- Yes --> C[Full grouped GEMM implementation compiled in]
B -- No --> D[Stub functions compiled in]
D --> E[NVTE_ERROR called at runtime]
E --> F["Error: requires cuBLAS 13.3+, current version is CUBLAS_VERSION
Please upgrade to cuBLAS 13.3 (shipped with CUDA 13.2) or newer."]
Reviews (1): Last reviewed commit: "update the error message for cublas vers..." | Re-trigger Greptile |
Description
Now that cuBLAS has its own semantic versioning, update the error message to avoid confusion.
cc @ksivaman @ptrendx Please review.
Type of change
Changes
Please list the changes introduced in this PR:
Checklist: