Skip to content

Commit 7a4a2d3

Browse files
authored
Merge branch 'master' into specify-array-api-version
2 parents 55dd09c + e0b7932 commit 7a4a2d3

22 files changed

+879
-919
lines changed

.github/workflows/build-sphinx.yml

Lines changed: 19 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -135,11 +135,26 @@ jobs:
135135
environment-file: ${{ env.environment-file }}
136136
activate-environment: 'docs'
137137

138-
- name: Conda info
139-
run: mamba info
138+
# We can't install dpctl as a conda package when the environment is created through
139+
# installing of Intel OneAPI packages because the dpctl conda package has a runtime
140+
# dependency on DPC++ RT one. Whereas the DPC++ RT package has been already installed
141+
# by the apt command above and its version has been matched with the DPC++ compiler.
142+
# In case where we install the DPC++ compiler with the apt (including DPC++ RT) and
143+
# install the DPC++ RT conda package while resolving dependencies, this can lead
144+
# to a versioning error, i.e. compatibility issue as the DPC++ compiler only guarantees
145+
# backwards compatibility, not forward compatibility (DPC++ RT may not run a binary built
146+
# with a newer version of the DPC++ compiler).
147+
# Installing dpctl via the pip manager has no such limitation, as the package has no
148+
# run dependency on the DPC++ RT pip package, so this is why the step is necessary here.
149+
- name: Install dpctl
150+
if: env.oneapi-pkgs-env == ''
151+
run: |
152+
pip install -i https://pypi.anaconda.org/dppy/label/dev/simple dpctl==0.20.0dev0
140153
141-
- name: Conda list
142-
run: mamba list
154+
- name: Conda info
155+
run: |
156+
mamba info
157+
mamba list
143158
144159
- name: Build library
145160
run: |

.github/workflows/check-mkl-interfaces.yaml

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ env:
1313
environment-file-name: 'environment.yml'
1414
environment-file-loc: '${{ github.workspace }}/environments'
1515
build-with-oneapi-env: 'environments/build_with_oneapi.yml'
16+
dpctl-pkg-env: 'environments/dpctl_pkg.yml'
1617
oneapi-pkgs-env: 'environments/oneapi_pkgs.yml'
1718
test-env-name: 'test_onemkl_interfaces'
1819
rerun-tests-on-failure: 'true'
@@ -47,10 +48,11 @@ jobs:
4748

4849
- name: Merge conda env files
4950
run: |
50-
conda-merge ${{ env.build-with-oneapi-env }} ${{ env.oneapi-pkgs-env }} > ${{ env.environment-file }}
51+
conda-merge ${{ env.build-with-oneapi-env }} ${{ env.dpctl-pkg-env }} ${{ env.oneapi-pkgs-env }} > ${{ env.environment-file }}
52+
cat ${{ env.environment-file }}
5153
5254
- name: Upload artifact
53-
uses: actions/upload-artifact@4cec3d8aa04e39d1a68397de0c4cd6fb9dce8ec1 # v4.6.1
55+
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
5456
with:
5557
name: ${{ env.environment-file-name }}
5658
path: ${{ env.environment-file }}
@@ -81,7 +83,7 @@ jobs:
8183
fetch-depth: 0
8284

8385
- name: Download artifact
84-
uses: actions/download-artifact@cc203385981b70ca67e1cc392babf9cc229d5806 # v4.1.9
86+
uses: actions/download-artifact@95815c38cf2ff2164869cbab79da8d1f422bc89e # v4.2.1
8587
with:
8688
name: ${{ env.environment-file-name }}
8789
path: ${{ env.environment-file-loc }}
@@ -174,7 +176,7 @@ jobs:
174176
fetch-depth: 0
175177

176178
- name: Download artifact
177-
uses: actions/download-artifact@cc203385981b70ca67e1cc392babf9cc229d5806 # v4.1.9
179+
uses: actions/download-artifact@95815c38cf2ff2164869cbab79da8d1f422bc89e # v4.2.1
178180
with:
179181
name: ${{ env.environment-file-name }}
180182
path: ${{ env.environment-file-loc }}

.github/workflows/conda-package.yml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -102,13 +102,13 @@ jobs:
102102
MAX_BUILD_CMPL_MKL_VERSION: '2025.1a0'
103103

104104
- name: Upload artifact
105-
uses: actions/upload-artifact@4cec3d8aa04e39d1a68397de0c4cd6fb9dce8ec1 # v4.6.1
105+
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
106106
with:
107107
name: ${{ env.package-name }} ${{ runner.os }} Python ${{ matrix.python }}
108108
path: ${{ env.CONDA_BLD }}${{ env.package-name }}-*.conda
109109

110110
- name: Upload wheels artifact
111-
uses: actions/upload-artifact@4cec3d8aa04e39d1a68397de0c4cd6fb9dce8ec1 # v4.6.1
111+
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
112112
with:
113113
name: ${{ env.package-name }} ${{ runner.os }} Wheels Python ${{ matrix.python }}
114114
path: ${{ env.WHEELS_OUTPUT_FOLDER }}${{ env.package-name }}-*.whl
@@ -146,7 +146,7 @@ jobs:
146146
path: ${{ env.dpnp-repo-path }}
147147

148148
- name: Download artifact
149-
uses: actions/download-artifact@cc203385981b70ca67e1cc392babf9cc229d5806 # v4.1.9
149+
uses: actions/download-artifact@95815c38cf2ff2164869cbab79da8d1f422bc89e # v4.2.1
150150
with:
151151
name: ${{ env.package-name }} ${{ runner.os }} Python ${{ matrix.python }}
152152
path: ${{ env.pkg-path-in-channel }}
@@ -278,7 +278,7 @@ jobs:
278278
path: ${{ env.dpnp-repo-path }}
279279

280280
- name: Download artifact
281-
uses: actions/download-artifact@cc203385981b70ca67e1cc392babf9cc229d5806 # v4.1.9
281+
uses: actions/download-artifact@95815c38cf2ff2164869cbab79da8d1f422bc89e # v4.2.1
282282
with:
283283
name: ${{ env.package-name }} ${{ runner.os }} Python ${{ matrix.python }}
284284
path: ${{ env.pkg-path-in-channel }}
@@ -442,12 +442,12 @@ jobs:
442442
fetch-depth: ${{ env.fetch-depth }}
443443

444444
- name: Download artifact
445-
uses: actions/download-artifact@cc203385981b70ca67e1cc392babf9cc229d5806 # v4.1.9
445+
uses: actions/download-artifact@95815c38cf2ff2164869cbab79da8d1f422bc89e # v4.2.1
446446
with:
447447
name: ${{ env.package-name }} ${{ runner.os }} Python ${{ matrix.python }}
448448

449449
- name: Download wheels artifact
450-
uses: actions/download-artifact@cc203385981b70ca67e1cc392babf9cc229d5806 # v4.1.9
450+
uses: actions/download-artifact@95815c38cf2ff2164869cbab79da8d1f422bc89e # v4.2.1
451451
with:
452452
name: ${{ env.package-name }} ${{ runner.os }} Wheels Python ${{ matrix.python }}
453453

@@ -530,7 +530,7 @@ jobs:
530530
path: ${{ env.dpnp-repo-path }}
531531

532532
- name: Download artifact
533-
uses: actions/download-artifact@cc203385981b70ca67e1cc392babf9cc229d5806 # v4.1.9
533+
uses: actions/download-artifact@95815c38cf2ff2164869cbab79da8d1f422bc89e # v4.2.1
534534
with:
535535
name: ${{ env.package-name }} ${{ runner.os }} Python ${{ env.python-ver }}
536536
path: ${{ env.pkg-path-in-channel }}

.github/workflows/generate_coverage.yaml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,22 @@ jobs:
9494
environment-file: ${{ env.environment-file }}
9595
activate-environment: 'coverage'
9696

97+
# We can't install dpctl as a conda package when the environment is created through
98+
# installing of Intel OneAPI packages because the dpctl conda package has a runtime
99+
# dependency on DPC++ RT one. Whereas the DPC++ RT package has beedn already installed
100+
# by the apt command above and its version has been matched with the DPC++ compiler.
101+
# In case where we install the DPC++ compiler with the apt (including DPC++ RT) and
102+
# install the DPC++ RT conda package while resolving dependencies, this can lead
103+
# to a versioning error, i.e. compatibility issue as the DPC++ compiler only guarantees
104+
# backwards compatibility, not forward compatibility (DPC++ RT may not run a binary built
105+
# with a newer version of the DPC++ compiler).
106+
# Installing dpctl via the pip manager has no such limitation, as the package has no
107+
# run dependency on the DPC++ RT pip package, so this is why the step is necessary here.
108+
- name: Install dpctl
109+
if: env.oneapi-pkgs-env == ''
110+
run: |
111+
pip install -i https://pypi.anaconda.org/dppy/label/dev/simple dpctl==0.20.0dev0
112+
97113
- name: Conda info
98114
run: |
99115
mamba info

.github/workflows/openssf-scorecard.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,14 +60,14 @@ jobs:
6060
# Upload the results as artifacts (optional). Commenting out will disable uploads of run results in SARIF
6161
# format to the repository Actions tab.
6262
- name: "Upload artifact"
63-
uses: actions/upload-artifact@4cec3d8aa04e39d1a68397de0c4cd6fb9dce8ec1 # v4.6.1
63+
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
6464
with:
6565
name: SARIF file
6666
path: results.sarif
6767
retention-days: 14
6868

6969
# Upload the results to GitHub's code scanning dashboard.
7070
- name: "Upload to code-scanning"
71-
uses: github/codeql-action/upload-sarif@6bb031afdd8eb862ea3fc1848194185e076637e5 # v3.28.11
71+
uses: github/codeql-action/upload-sarif@5f8171a638ada777af81d42b55959a643bb29017 # v3.28.12
7272
with:
7373
sarif_file: results.sarif

CHANGELOG.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
1515

1616
### Changed
1717

18+
* Extended `dpnp.fft.fftfreq` and `dpnp.fft.rfftfreq` functions to support `dtype` keyword per Python Array API spec 2024.12 [#2384](https://github.com/IntelPython/dpnp/pull/2384)
1819
* Allowed input array of `uint64` dtype in `dpnp.bincount` [#2361](https://github.com/IntelPython/dpnp/pull/2361)
20+
* The vector norms `ord={None, 1, 2, inf}` and the matrix norms `ord={None, 1, 2, inf, "fro", "nuc"}` now consistently return zero for empty arrays, which are arrays with at least one axis of size zero. This change affects `dpnp.linalg.norm`, `dpnp.linalg.vector_norm`, and `dpnp.linalg.matrix_norm`. Previously, dpnp would either raise errors or return zero depending on the parameters provided [#2371](https://github.com/IntelPython/dpnp/pull/2371)
21+
* Improved performance of `dpnp.nansum`, `dpnp.nanprod`, `dpnp.nancumsum`, and `dpnp.nancumprod` by reusing `dpnp.nan_to_num` function in implementation of the functions [#2339](https://github.com/IntelPython/dpnp/pull/2339)
1922

2023
### Fixed
2124

25+
* Resolved an issue with an incorrect result returned due to missing dependency from the strided kernel on a copy event in `dpnp.erf` [#2378](https://github.com/IntelPython/dpnp/pull/2378)
26+
2227

2328
## [0.17.0] - 02/26/2025
2429

dpnp/backend/extensions/fft/CMakeLists.txt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,6 @@
2727
set(python_module_name _fft_impl)
2828
set(_module_src
2929
${CMAKE_CURRENT_SOURCE_DIR}/fft_py.cpp
30-
${CMAKE_CURRENT_SOURCE_DIR}/in_place.cpp
31-
${CMAKE_CURRENT_SOURCE_DIR}/out_of_place.cpp
3230
)
3331

3432
pybind11_add_module(${python_module_name} MODULE ${_module_src})

dpnp/backend/extensions/fft/in_place.hpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,3 +42,5 @@ std::pair<sycl::event, sycl::event>
4242
const std::vector<sycl::event> &depends);
4343

4444
} // namespace dpnp::extensions::fft
45+
46+
#include "in_place.tpp" // Include template definition

dpnp/backend/extensions/fft/in_place.cpp renamed to dpnp/backend/extensions/fft/in_place.tpp

Lines changed: 1 addition & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
// THE POSSIBILITY OF SUCH DAMAGE.
2424
//*****************************************************************************
2525

26+
#pragma once
2627
#include <stdexcept>
2728

2829
#include <oneapi/mkl.hpp>
@@ -32,7 +33,6 @@
3233

3334
#include "common.hpp"
3435
#include "fft_utils.hpp"
35-
#include "in_place.hpp"
3636
// dpctl tensor headers
3737
#include "utils/output_validation.hpp"
3838

@@ -107,21 +107,4 @@ std::pair<sycl::event, sycl::event>
107107
return std::make_pair(fft_event, args_ev);
108108
}
109109

110-
// Explicit instantiations
111-
// single precision c2c FFT
112-
template std::pair<sycl::event, sycl::event> compute_fft_in_place(
113-
DescriptorWrapper<mkl_dft::precision::SINGLE, mkl_dft::domain::COMPLEX>
114-
&descr,
115-
const dpctl::tensor::usm_ndarray &in_out,
116-
const bool is_forward,
117-
const std::vector<sycl::event> &depends);
118-
119-
// double precision c2c FFT
120-
template std::pair<sycl::event, sycl::event> compute_fft_in_place(
121-
DescriptorWrapper<mkl_dft::precision::DOUBLE, mkl_dft::domain::COMPLEX>
122-
&descr,
123-
const dpctl::tensor::usm_ndarray &in_out,
124-
const bool is_forward,
125-
const std::vector<sycl::event> &depends);
126-
127110
} // namespace dpnp::extensions::fft

dpnp/backend/extensions/fft/out_of_place.hpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,3 +43,5 @@ std::pair<sycl::event, sycl::event>
4343
const std::vector<sycl::event> &depends);
4444

4545
} // namespace dpnp::extensions::fft
46+
47+
#include "out_of_place.tpp" // Include template definition

0 commit comments

Comments
 (0)