Skip to content

Move dpctl_ext.tensor to dpnp.tensor#2829

Merged
vlad-perevezentsev merged 324 commits intoinclude-dpctl-tensorfrom
move_dpctl_ext_to_dpnp_tensor
Apr 8, 2026
Merged

Move dpctl_ext.tensor to dpnp.tensor#2829
vlad-perevezentsev merged 324 commits intoinclude-dpctl-tensorfrom
move_dpctl_ext_to_dpnp_tensor

Conversation

@vlad-perevezentsev
Copy link
Copy Markdown
Contributor

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

This PR proposes a refactoring that migrates dpctl_ext.tensor module into dpnp package as dpnp.tensor

Changes:

  1. Moved dpctl_ext/tensor/ directory to dpnp/tensor/
  2. Updated all imports from dpctl_ext.tensor to dpnp.tensor across the codebase
  3. Consolidated build: removed dpctl_ext/CMakeLists.txt, added build_dpnp_tensor_ext() to dpnp/CMakeLists.txt
  4. Added DPNP_BUILD_COMPONENTS CMake option (ALL/TENSOR_ONLY/SKIP_TENSOR) for staged builds
  5. Split coverage workflow into two steps to avoid memory issues
  6. Updated include paths in all backend extension CMake files
  7. Removed dpctl_ext/ directory and cleaned up .gitignore
  • 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?

@vlad-perevezentsev vlad-perevezentsev marked this pull request as ready for review March 31, 2026 14:07
@vlad-perevezentsev vlad-perevezentsev force-pushed the move_dpctl_ext_to_dpnp_tensor branch from a621aaa to d819ae3 Compare April 1, 2026 12:35
@vlad-perevezentsev vlad-perevezentsev force-pushed the move_dpctl_ext_to_dpnp_tensor branch from 4497e9e to 1d9cabd Compare April 1, 2026 14:11
@vlad-perevezentsev vlad-perevezentsev force-pushed the move_dpctl_ext_to_dpnp_tensor branch from 1d9cabd to e023f2f Compare April 1, 2026 14:14
ndgrigorian
ndgrigorian previously approved these changes Apr 2, 2026
Copy link
Copy Markdown
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.

other than that minor curiosity and Anton's comments, LGTM

Base automatically changed from remove_c_api_tensor to include-dpctl-tensor April 7, 2026 12:58
@vlad-perevezentsev vlad-perevezentsev dismissed ndgrigorian’s stale review April 7, 2026 12:58

The base branch was changed.

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.

LGTM

@vlad-perevezentsev vlad-perevezentsev merged commit ed3476f into include-dpctl-tensor Apr 8, 2026
63 of 70 checks passed
@vlad-perevezentsev vlad-perevezentsev deleted the move_dpctl_ext_to_dpnp_tensor branch April 8, 2026 14:57
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