[BLAS] Wrong namespace usage in BLAS Documentation about onemkl

[hjabird]

It looks to me like you're right in that the specification has errors.

The transpose should be in the omeapi::mkl namespace as documented here:

oneMKL BLAS and LAPACK for Data Parallel C++ (DPC++) introduces several new enumeration data types, which are type-safe versions of the traditional Fortran characters in BLAS and LAPACK. They are declared in types.hpp, which is included automatically when you include mkl.hpp. Like all oneMKL DPC++ functionality, they belong to the namespace oneapi::mkl.

However, I think this is a specification issue, not a oneMKL interface library issue. It might be better to open an issue here.

EDIT: I got confused between the oneAPI specification and the documentation for this project. It looks like the oneAPI specification has the same issue here. However, it still looks like everything should be in the oneapi::mkl namespace (here).

[AD2605]

Thanks for the quick response !

So from what I understand, it's both a spec as well as a documentation issue correct, and a similar Issue must be raised on the oneapi-spec Repo as well ?

[hjabird]

It looks like this already has an issue for the oneAPI spec uxlfoundation/oneAPI-spec#534 and a PR uxlfoundation/oneAPI-spec#535.

[hjabird]

@mmeterel, @andrewtbarker we think it makes more sense to direct users to the oneAPI/oneMKL specification instead of duplicating the documentation here. I think the solution to this is to:

• delete the BLAS domain documentation in this repo.
• Replace with a link the the oneAPI/oneMKL specification.

What do you think?

[andrewtbarker]

@hjabird I tend to agree with you. The only concern I have is that the implementation here might not be perfectly in line with the oneAPI specification at all times, and ideally we would have a way to document discrepancies, but I agree that maintaining two sets of docs does not really make sense.