Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removes the deprecated feature of providing kernel launch params as lists and tuple #1251

Merged
merged 3 commits into from
Dec 26, 2023

Conversation

diptorupd
Copy link
Contributor

  • Have you provided a meaningful PR description?

The existing numba_dpex.kernel functions can be launched by providing either a Range or NdRange object to numba_dpex.core.kernel_interface.dispatcher.JitKernel.__getitem__, or the global and local sizes as lists or tuples. The use of list and tuples was deprecated a while back. This PR removes the deprecated support and now only Range or NdRange arguments are supported as kernel launch parameters.

  • Have you added a test, reproducer or referred to an issue with a reproducer?
    No, but existing tests have been fixed and updated.
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • If this PR is a work in progress, are you filing the PR as a draft?

Fixes #1040

@diptorupd diptorupd requested a review from ZzEeKkAa December 18, 2023 20:22
@diptorupd diptorupd force-pushed the remove/deprecated_launch_params branch 2 times, most recently from 134d0e5 to e016de9 Compare December 19, 2023 00:23
Copy link
Contributor

@ZzEeKkAa ZzEeKkAa left a comment

Choose a reason for hiding this comment

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

LGTM! Just need to update dpbench

@ZzEeKkAa ZzEeKkAa force-pushed the remove/deprecated_launch_params branch from e016de9 to b539970 Compare December 26, 2023 21:26
@ZzEeKkAa
Copy link
Contributor

Tested with latest dpbench - works OK.

@ZzEeKkAa ZzEeKkAa merged commit 1f0e316 into main Dec 26, 2023
34 of 42 checks passed
@ZzEeKkAa ZzEeKkAa deleted the remove/deprecated_launch_params branch December 26, 2023 21:54
github-actions bot added a commit that referenced this pull request Dec 26, 2023
…rams

Removes the deprecated feature of providing kernel launch params as lists and tuple 1f0e316
github-actions bot added a commit to chudur-budur/numba-dpex that referenced this pull request Dec 26, 2023
…d_launch_params

Removes the deprecated feature of providing kernel launch params as lists and tuple 1f0e316
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.

Add compute follows data unit tests for kernel API
2 participants