Skip to content

Extend ._tensor_impl with advanced indexing functions#2777

Merged
vlad-perevezentsev merged 64 commits intoinclude-dpctl-tensorfrom
move_tensor_impl_accum_bool_ind_eye
Mar 3, 2026
Merged

Extend ._tensor_impl with advanced indexing functions#2777
vlad-perevezentsev merged 64 commits intoinclude-dpctl-tensorfrom
move_tensor_impl_accum_bool_ind_eye

Conversation

@vlad-perevezentsev
Copy link
Contributor

This PR extends _tensor_impl in dpctl_ext.tensor with theadvanced indexing (_extract, _place, _nonzero, mask_positions, ), repeat (_cumsum_1d) and _eye functions

It also adds eye(), extract(), nonzero(), place(), put_along_axis(), take_along_axis() to dpctl_ext.tensor and updates the corresponding dpnp functions to use these implementations internally

  • Have you provided a meaningful PR description?
  • Have you added a test, reproducer or referred to an issue with a reproducer?
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • Have you checked performance impact of proposed changes?
  • Have you added documentation for your changes, if necessary?
  • Have you added your changes to the changelog?

@github-actions
Copy link
Contributor

github-actions bot commented Feb 19, 2026

Array API standard conformance tests for dpnp=0.20.0dev3=py313h509198e_74 ran successfully.
Passed: 1355
Failed: 2
Skipped: 16

@github-actions
Copy link
Contributor

github-actions bot commented Feb 19, 2026

View rendered docs @ https://intelpython.github.io/dpnp/index.html

@vlad-perevezentsev vlad-perevezentsev marked this pull request as ready for review February 20, 2026 11:08
ndgrigorian
ndgrigorian previously approved these changes Mar 2, 2026
Copy link
Collaborator

@ndgrigorian ndgrigorian left a comment

Choose a reason for hiding this comment

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

LGTM

Base automatically changed from move_tensor_impl_copy to include-dpctl-tensor March 3, 2026 10:25
@vlad-perevezentsev vlad-perevezentsev dismissed ndgrigorian’s stale review March 3, 2026 10:25

The base branch was changed.

ndgrigorian
ndgrigorian previously approved these changes Mar 3, 2026
@vlad-perevezentsev vlad-perevezentsev merged commit 192bd93 into include-dpctl-tensor Mar 3, 2026
91 of 93 checks passed
@vlad-perevezentsev vlad-perevezentsev deleted the move_tensor_impl_accum_bool_ind_eye branch March 3, 2026 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants