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

[oneMKL][spblas] Add optimize_symv() and optimize_gemm() routines in oneMKL specification. #499

Merged

Conversation

baeseung-intel
Copy link
Contributor

Goal:
In this PR, I'd like to add two optimization routines, optimize_symv() and optimize_gemm() into oneMKL specification.

Details:

  • Add optimize_symv() API into oneMKL specification, which prepares optimization for oneapi::mkl::sparse::symv() API.
  • Add optimize_gemm() API into oneMKL specification, which prepares optimization for oneapi::mkl::sparse::gemm() API.
  • For optimize_gemm() API, we propose two different routines; the first API will prepare optimization based on sparse input matrix only, and the other API will prepare optimization based on both sparse and dense input matrices.
  • Add those two APIs in the Sparse BLAS main page.

@rscohn2
Copy link
Member

rscohn2 commented Sep 11, 2023

I would like to wrap up the 1.3-provisional release. Is this ready for merging.

@spencerpatty
Copy link
Contributor

@rscohn2 let me take one more pass at cleaning up language a little bit, it is 95% there in my opinion... :)

Copy link
Contributor

@spencerpatty spencerpatty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I approve of this addition to the Sparse BLAS spec :)

@spencerpatty spencerpatty self-assigned this Sep 11, 2023
@spencerpatty spencerpatty added enhancement New feature or request MKL labels Sep 11, 2023
@rscohn2 rscohn2 merged commit b260b06 into uxlfoundation:main Sep 13, 2023
3 checks passed
aepanchi pushed a commit to aepanchi/oneAPI-spec that referenced this pull request Oct 31, 2023
… in oneMKL specification. (uxlfoundation#499)

* [oneMKL][spblas] add spec for oneapi::mkl::sparse::optimize_gemm() routine.

* [oneMKL][spblas] add spec for oneapi::mkl::sparse::optimize_symv() routine.

* minor fix.

* [oneMKL][spblas] add explanation of why we propose two versions of optimize_gemm().

* Update source/elements/oneMKL/source/domains/spblas/gemmoptimize.rst

* Apply suggestions from code review

* fix whitespace issues

---------

Co-authored-by: Spencer Patty <spencer.patty@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request MKL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants