-
Notifications
You must be signed in to change notification settings - Fork 22
Implement syevd_batch
and heevd_batch
#1936
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
Merged
Merged
Changes from 12 commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
b94fcb4
Implement syevd_batch and heevd_batch
vlad-perevezentsev 082d53f
Merge master into impl_syevd_heevd_batch
vlad-perevezentsev f4dd124
Merge master into impl_syevd_heevd_batch
vlad-perevezentsev 2da4e59
Move include dpctl type_utils header to sourse files
vlad-perevezentsev 88249e4
Add memory alocation check for scratchpad
vlad-perevezentsev d222704
Add more checks for scratchpad_size
vlad-perevezentsev 73265c5
Move includes
vlad-perevezentsev 5eaaba4
Allocate memory for w with expected shape
vlad-perevezentsev fca9470
Applied review comments
vlad-perevezentsev e523056
Add common_evd_checks to reduce dublicate code
vlad-perevezentsev dba395a
Merge master into impl_syevd_heevd_batch
vlad-perevezentsev a080369
Remove host_task_events from syevd and heevd
vlad-perevezentsev cca72b8
Applied review comments
vlad-perevezentsev 19c82fb
Use init_evd_dispatch_table instead of init_evd_batch_dispatch_table
vlad-perevezentsev d4ce515
Move init_evd_dispatch_table to evd_common_utils.hpp
vlad-perevezentsev 5932918
Add helper function check_zeros_shape
vlad-perevezentsev 48b2d63
Implement alloc_scratchpad function to evd_batch_common.hpp
vlad-perevezentsev 7339c69
Merge master into impl_syevd_heevd_batch
vlad-perevezentsev e974eff
Make round_up_mult as inline
vlad-perevezentsev 21dc0b0
Add comment for check_zeros_shape
vlad-perevezentsev db45662
Merge master into impl_syevd_heevd_batch
vlad-perevezentsev 894ac54
Make alloc_scratchpad as inline
vlad-perevezentsev File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,138 @@ | ||
//***************************************************************************** | ||
// Copyright (c) 2024, Intel Corporation | ||
// All rights reserved. | ||
// | ||
// Redistribution and use in source and binary forms, with or without | ||
// modification, are permitted provided that the following conditions are met: | ||
// - Redistributions of source code must retain the above copyright notice, | ||
// this list of conditions and the following disclaimer. | ||
// - Redistributions in binary form must reproduce the above copyright notice, | ||
// this list of conditions and the following disclaimer in the documentation | ||
// and/or other materials provided with the distribution. | ||
// | ||
// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" | ||
// AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE | ||
// IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE | ||
// ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE | ||
// LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR | ||
// CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF | ||
// SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS | ||
// INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN | ||
// CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) | ||
// ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF | ||
// THE POSSIBILITY OF SUCH DAMAGE. | ||
//***************************************************************************** | ||
|
||
#pragma once | ||
|
||
#include <oneapi/mkl.hpp> | ||
#include <pybind11/pybind11.h> | ||
|
||
// dpctl tensor headers | ||
#include "utils/type_dispatch.hpp" | ||
|
||
#include "evd_common_utils.hpp" | ||
#include "types_matrix.hpp" | ||
|
||
namespace dpnp::extensions::lapack::evd | ||
{ | ||
typedef sycl::event (*evd_batch_impl_fn_ptr_t)( | ||
sycl::queue &, | ||
const oneapi::mkl::job, | ||
const oneapi::mkl::uplo, | ||
const std::int64_t, | ||
const std::int64_t, | ||
char *, | ||
char *, | ||
const std::vector<sycl::event> &); | ||
|
||
namespace dpctl_td_ns = dpctl::tensor::type_dispatch; | ||
namespace py = pybind11; | ||
|
||
template <typename dispatchT> | ||
std::pair<sycl::event, sycl::event> | ||
evd_batch_func(sycl::queue &exec_q, | ||
const std::int8_t jobz, | ||
const std::int8_t upper_lower, | ||
dpctl::tensor::usm_ndarray &eig_vecs, | ||
dpctl::tensor::usm_ndarray &eig_vals, | ||
const std::vector<sycl::event> &depends, | ||
const dispatchT &evd_batch_dispatch_table) | ||
{ | ||
const int eig_vecs_nd = eig_vecs.get_ndim(); | ||
|
||
const py::ssize_t *eig_vecs_shape = eig_vecs.get_shape_raw(); | ||
const py::ssize_t *eig_vals_shape = eig_vals.get_shape_raw(); | ||
|
||
const int expected_eig_vecs_nd = 3; | ||
const int expected_eig_vals_nd = 2; | ||
|
||
common_evd_checks(exec_q, eig_vecs, eig_vals, eig_vecs_shape, | ||
expected_eig_vecs_nd, expected_eig_vals_nd); | ||
|
||
if (eig_vecs_shape[2] != eig_vals_shape[0] || | ||
eig_vecs_shape[0] != eig_vals_shape[1]) | ||
{ | ||
throw py::value_error( | ||
"The shape of 'eig_vals' must be (batch_size, n), " | ||
"where batch_size = " + | ||
std::to_string(eig_vecs_shape[0]) + | ||
" and n = " + std::to_string(eig_vecs_shape[1])); | ||
} | ||
|
||
size_t src_nelems(1); | ||
|
||
for (int i = 0; i < eig_vecs_nd; ++i) { | ||
src_nelems *= static_cast<size_t>(eig_vecs_shape[i]); | ||
} | ||
|
||
if (src_nelems == 0) { | ||
// nothing to do | ||
return std::make_pair(sycl::event(), sycl::event()); | ||
} | ||
vlad-perevezentsev marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
auto array_types = dpctl_td_ns::usm_ndarray_types(); | ||
const int eig_vecs_type_id = | ||
array_types.typenum_to_lookup_id(eig_vecs.get_typenum()); | ||
const int eig_vals_type_id = | ||
array_types.typenum_to_lookup_id(eig_vals.get_typenum()); | ||
|
||
evd_batch_impl_fn_ptr_t evd_batch_fn = | ||
evd_batch_dispatch_table[eig_vecs_type_id][eig_vals_type_id]; | ||
if (evd_batch_fn == nullptr) { | ||
throw py::value_error( | ||
"Types of input vectors and result array are mismatched."); | ||
} | ||
|
||
char *eig_vecs_data = eig_vecs.get_data(); | ||
char *eig_vals_data = eig_vals.get_data(); | ||
|
||
const std::int64_t batch_size = eig_vecs_shape[2]; | ||
const std::int64_t n = eig_vecs_shape[1]; | ||
|
||
const oneapi::mkl::job jobz_val = static_cast<oneapi::mkl::job>(jobz); | ||
const oneapi::mkl::uplo uplo_val = | ||
static_cast<oneapi::mkl::uplo>(upper_lower); | ||
|
||
sycl::event evd_batch_ev = | ||
evd_batch_fn(exec_q, jobz_val, uplo_val, batch_size, n, eig_vecs_data, | ||
eig_vals_data, depends); | ||
|
||
sycl::event ht_ev = dpctl::utils::keep_args_alive( | ||
exec_q, {eig_vecs, eig_vals}, {evd_batch_ev}); | ||
|
||
return std::make_pair(ht_ev, evd_batch_ev); | ||
vlad-perevezentsev marked this conversation as resolved.
Show resolved
Hide resolved
|
||
} | ||
|
||
template <typename dispatchT, | ||
template <typename fnT, typename T, typename RealT> | ||
typename factoryT> | ||
void init_evd_batch_dispatch_table( | ||
vlad-perevezentsev marked this conversation as resolved.
Show resolved
Hide resolved
|
||
dispatchT evd_batch_dispatch_table[][dpctl_td_ns::num_types]) | ||
{ | ||
dpctl_td_ns::DispatchTableBuilder<dispatchT, factoryT, | ||
dpctl_td_ns::num_types> | ||
contig; | ||
contig.populate_dispatch_table(evd_batch_dispatch_table); | ||
} | ||
} // namespace dpnp::extensions::lapack::evd |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,92 @@ | ||
//***************************************************************************** | ||
// Copyright (c) 2024, Intel Corporation | ||
// All rights reserved. | ||
// | ||
// Redistribution and use in source and binary forms, with or without | ||
// modification, are permitted provided that the following conditions are met: | ||
// - Redistributions of source code must retain the above copyright notice, | ||
// this list of conditions and the following disclaimer. | ||
// - Redistributions in binary form must reproduce the above copyright notice, | ||
// this list of conditions and the following disclaimer in the documentation | ||
// and/or other materials provided with the distribution. | ||
// | ||
// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" | ||
// AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE | ||
// IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE | ||
// ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE | ||
// LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR | ||
// CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF | ||
// SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS | ||
// INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN | ||
// CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) | ||
// ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF | ||
// THE POSSIBILITY OF SUCH DAMAGE. | ||
//***************************************************************************** | ||
|
||
#pragma once | ||
|
||
#include <pybind11/pybind11.h> | ||
|
||
// dpctl tensor headers | ||
#include "utils/memory_overlap.hpp" | ||
#include "utils/output_validation.hpp" | ||
|
||
namespace dpnp::extensions::lapack::evd | ||
{ | ||
namespace py = pybind11; | ||
|
||
inline void common_evd_checks(sycl::queue &exec_q, | ||
dpctl::tensor::usm_ndarray &eig_vecs, | ||
dpctl::tensor::usm_ndarray &eig_vals, | ||
const py::ssize_t *eig_vecs_shape, | ||
const int expected_eig_vecs_nd, | ||
const int expected_eig_vals_nd) | ||
{ | ||
const int eig_vecs_nd = eig_vecs.get_ndim(); | ||
const int eig_vals_nd = eig_vals.get_ndim(); | ||
|
||
if (eig_vecs_nd != expected_eig_vecs_nd) { | ||
throw py::value_error("The output eigenvectors array has ndim=" + | ||
std::to_string(eig_vecs_nd) + ", but a " + | ||
std::to_string(expected_eig_vecs_nd) + | ||
"-dimensional array is expected."); | ||
} | ||
else if (eig_vals_nd != expected_eig_vals_nd) { | ||
throw py::value_error("The output eigenvalues array has ndim=" + | ||
std::to_string(eig_vals_nd) + ", but a " + | ||
std::to_string(expected_eig_vals_nd) + | ||
"-dimensional array is expected."); | ||
} | ||
|
||
if (eig_vecs_shape[0] != eig_vecs_shape[1]) { | ||
throw py::value_error("Output array with eigenvectors must be square"); | ||
} | ||
|
||
dpctl::tensor::validation::CheckWritable::throw_if_not_writable(eig_vecs); | ||
dpctl::tensor::validation::CheckWritable::throw_if_not_writable(eig_vals); | ||
|
||
// check compatibility of execution queue and allocation queue | ||
if (!dpctl::utils::queues_are_compatible(exec_q, {eig_vecs, eig_vals})) { | ||
throw py::value_error( | ||
"Execution queue is not compatible with allocation queues"); | ||
} | ||
|
||
auto const &overlap = dpctl::tensor::overlap::MemoryOverlap(); | ||
if (overlap(eig_vecs, eig_vals)) { | ||
throw py::value_error("Arrays with eigenvectors and eigenvalues are " | ||
"overlapping segments of memory"); | ||
} | ||
|
||
const bool is_eig_vecs_f_contig = eig_vecs.is_f_contiguous(); | ||
const bool is_eig_vals_c_contig = eig_vals.is_c_contiguous(); | ||
if (!is_eig_vecs_f_contig) { | ||
throw py::value_error( | ||
"An array with input matrix / output eigenvectors " | ||
"must be F-contiguous"); | ||
} | ||
else if (!is_eig_vals_c_contig) { | ||
throw py::value_error( | ||
"An array with output eigenvalues must be C-contiguous"); | ||
} | ||
} | ||
} // namespace dpnp::extensions::lapack::evd |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.