Skip to content

Add dpctl.SyclQueue.fill() method - #2365

Merged
vlad-perevezentsev merged 21 commits into
masterfrom
add_sycl_queue_fill
Sep 1, 2026
Merged

Add dpctl.SyclQueue.fill() method#2365
vlad-perevezentsev merged 21 commits into
masterfrom
add_sycl_queue_fill

Conversation

@vlad-perevezentsev

@vlad-perevezentsev vlad-perevezentsev commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

This PR proposes adding dpctl.SyclQueue.fill and dpctl.SyclQueue.fill_async methods as a Python wrapper over sycl::queue::fill backed by the existing DPCTLQueue_Fill8/16/32/64/128 and new DPCTLQueue_Fill8/16/32/64/128WithEvents C-API functions and adding a new test_sycl_queue_fill.py

  • 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?
  • If this PR is a work in progress, are you opening the PR as a draft?

@vlad-perevezentsev vlad-perevezentsev self-assigned this Aug 18, 2026
@github-actions

github-actions Bot commented Aug 18, 2026

Copy link
Copy Markdown

Deleted rendered PR docs from intelpython.github.com/dpctl, latest should be updated shortly. 🤞

@coveralls

coveralls commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

Coverage Status

coverage: 74.797% (+0.3%) from 74.51% — add_sycl_queue_fill into master

@vlad-perevezentsev
vlad-perevezentsev marked this pull request as ready for review August 18, 2026 15:20
Comment thread dpctl/_sycl_queue.pyx
Comment thread dpctl/_sycl_queue.pyx Outdated
Comment thread libsyclinterface/source/dpctl_sycl_queue_interface.cpp
Comment thread libsyclinterface/tests/test_sycl_queue_interface.cpp

@ndgrigorian ndgrigorian left a comment

Copy link
Copy Markdown
Collaborator

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 0bae6a8 into master Sep 1, 2026
125 checks passed
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