Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## latest #3293 +/- ##
==========================================
+ Coverage 73.31% 73.56% +0.24%
==========================================
Files 446 446
Lines 108397 109543 +1146
Branches 17360 17547 +187
==========================================
+ Hits 79476 80588 +1112
- Misses 28645 28678 +33
- Partials 276 277 +1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Quick question: are you wanting to use openblas_get_num_threads for anything else?
An alternate implementation would be to modify the set_num_threads to also return the number of threads:
int openblas_set_num_threads(int num_threads) {
#if defined(HIPO_USES_OPENBLAS)
openblas_set_num_threads(num_threads);
return openblas_get_num_threads();
#else
return num_threads;
#endif
}This would make most of the other changes unnecessary.
|
|
Description
openblas_get_num_threads, to query the number of threads used by OpenBLAS.HIPO_USES_OPENBLASis passed to HiPO.BLAS_LIBRARIESis used and the provided library contains the stringopenblas, set the BLAS metadata so that HiPO knows that OpenBLAS is being used.Checklist
latestbranch