Skip to content

Extend ._tensor_impl with where(), clip() and type utils functions#2778

Merged
vlad-perevezentsev merged 91 commits intoinclude-dpctl-tensorfrom
move_tensor_impl_repeat_where_clip
Mar 5, 2026
Merged

Extend ._tensor_impl with where(), clip() and type utils functions#2778
vlad-perevezentsev merged 91 commits intoinclude-dpctl-tensorfrom
move_tensor_impl_repeat_where_clip

Conversation

@vlad-perevezentsev
Copy link
Contributor

@vlad-perevezentsev vlad-perevezentsev commented Feb 20, 2026

This PR extends _tensor_impl in dpctl_ext.tensor with the _where, _clip and repeat functions
(_repeat_by_sequence, _repeat_by_scalar)

It also adds repeat(), where(), clip() and can_cast, finfo, iinfo, isdtype, result_type from _type_utils.py `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 20, 2026

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

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_accum_bool_ind_eye to include-dpctl-tensor March 3, 2026 13:39
@vlad-perevezentsev vlad-perevezentsev dismissed ndgrigorian’s stale review March 3, 2026 13:39

The base branch was changed.

antonwolfy
antonwolfy previously approved these changes Mar 4, 2026
Copy link
Contributor

@antonwolfy antonwolfy left a comment

Choose a reason for hiding this comment

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

All other LGTM!

@vlad-perevezentsev vlad-perevezentsev merged commit 0edd3b1 into include-dpctl-tensor Mar 5, 2026
70 of 77 checks passed
@vlad-perevezentsev vlad-perevezentsev deleted the move_tensor_impl_repeat_where_clip branch March 5, 2026 12:27
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.

3 participants