Skip to content

Move tensor tests from dpctl #2817

Merged
vlad-perevezentsev merged 363 commits intoinclude-dpctl-tensorfrom
move_tensor_tests
Apr 10, 2026
Merged

Move tensor tests from dpctl #2817
vlad-perevezentsev merged 363 commits intoinclude-dpctl-tensorfrom
move_tensor_tests

Conversation

@vlad-perevezentsev
Copy link
Copy Markdown
Contributor

@vlad-perevezentsev vlad-perevezentsev commented Mar 12, 2026

This PR moves all tensor-related tests to dpnp/tests/tensor as part of the ongoing migration of tensor functionality from dpctl to dpnp.tensor

Key changes:

  • Relocated 89 tensor tests (elementwise functions, usm_ndarray, and tensor utilities)
  • Updated imports to use dpnp.tensor
  • Included tests in packaging configuration
  • Integrated tensor tests into CI
  • Fixed several issues discovered during migration (dtype expectations, boolean reductions, etc.)
  • Fixed a circular import in _usmarray.py
  • Added SKIP_TENSOR_TESTS env variable to manage the launch of the test scope

In a follow-up PR:

  • Conditional logic will be added to run dpctl_ext/tests only when changes affect the tensor code.
  • Array API tests for tensor will be introduced and executed as a separate CI job.
  • 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?

Base automatically changed from move_dpctl_ext_to_dpnp_tensor to include-dpctl-tensor April 8, 2026 14:57
antonwolfy
antonwolfy previously approved these changes Apr 8, 2026
Copy link
Copy Markdown
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.

In general, LGTM

Copy link
Copy Markdown
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.

Thank you @vlad-perevezentsev, no more comments from me

@vlad-perevezentsev vlad-perevezentsev merged commit e1aad07 into include-dpctl-tensor Apr 10, 2026
20 of 36 checks passed
@vlad-perevezentsev vlad-perevezentsev deleted the move_tensor_tests branch April 10, 2026 11:42
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