Skip to content

Conversation

@diptorupd
Copy link
Contributor

  • Have you provided a meaningful PR description?
    • Removes superfluous copy inside JitKernel.getitem
    • Improves the JitKernel._check_range and JitKernel._check_ranges
      methods. We now check if a range is empty and if a well-formed
      global range was provided.
    • Minor fixes to error messages (typos, etc.)
    • Add missing docstrings to numba_dpex.core.exceptions classes.
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?

    - Removes superfluous copy inside JitKernel.__getitem__
    - Improves the JitKernel._check_range and JitKernel._check_ranges
      methods. We now check if a range is empty and if a well-formed
      global range was provided.
    - Minor fixes to error messages (typos, etc.)
    - Add missing docstrings to numba_dpex.core.exceptions classes.
@diptorupd diptorupd merged commit 9809c95 into main Jan 19, 2023
@diptorupd diptorupd deleted the fix/check_range branch January 19, 2023 04:32
github-actions bot added a commit that referenced this pull request Jan 19, 2023
Fixes to checking validity of a range. 9809c95
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.

2 participants