Skip to content

Move all logic with cyl_bessel_i0 support in i0.hpp #2440

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

Merged
merged 4 commits into from
May 12, 2025

Conversation

antonwolfy
Copy link
Contributor

@antonwolfy antonwolfy commented May 8, 2025

This PR proposes to make cyl_bessel_i0 function fully exposed by dpnp/backend/kernels/elementwise_functions/i0.hpp header to hide all logic validating if sycl::ext::intel::math::cyl_bessel_i0 function from sycl/ext/intel/math.hpp header can be used or native implementation is required.

  • 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?

@antonwolfy antonwolfy self-assigned this May 8, 2025
@antonwolfy antonwolfy force-pushed the expose-cyl_bessel_i0-as-part-of-i0-header branch 2 times, most recently from bda573e to b204f05 Compare May 8, 2025 14:39
Copy link
Contributor

github-actions bot commented May 8, 2025

View rendered docs @ https://intelpython.github.io/dpnp/index.html

Copy link
Contributor

github-actions bot commented May 8, 2025

Array API standard conformance tests for dpnp=0.18.0dev2=py312he4f9c94_8 ran successfully.
Passed: 1228
Failed: 1
Skipped: 9

Base automatically changed from update_kaiser_i0_cuda to master May 8, 2025 16:58
@antonwolfy antonwolfy force-pushed the expose-cyl_bessel_i0-as-part-of-i0-header branch from b204f05 to 5411d71 Compare May 8, 2025 17:18
@antonwolfy antonwolfy force-pushed the expose-cyl_bessel_i0-as-part-of-i0-header branch from 5411d71 to e1ad68c Compare May 8, 2025 17:21
@coveralls
Copy link
Collaborator

coveralls commented May 8, 2025

Coverage Status

coverage: 71.781% (+0.007%) from 71.774%
when pulling 9ae87c9 on expose-cyl_bessel_i0-as-part-of-i0-header
into a37fadb on master.

@antonwolfy antonwolfy added this to the 0.18.0 release milestone May 8, 2025
Copy link
Collaborator

@vlad-perevezentsev vlad-perevezentsev left a comment

Choose a reason for hiding this comment

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

LGTM!
Thank you @antonwolfy

@antonwolfy antonwolfy merged commit 9792685 into master May 12, 2025
63 of 68 checks passed
@antonwolfy antonwolfy deleted the expose-cyl_bessel_i0-as-part-of-i0-header branch May 12, 2025 18:37
github-actions bot added a commit that referenced this pull request May 12, 2025
This PR proposes to make `cyl_bessel_i0` function fully exposed by
`dpnp/backend/kernels/elementwise_functions/i0.hpp` header to hide all
logic validating if `sycl::ext::intel::math::cyl_bessel_i0` function
from `sycl/ext/intel/math.hpp` header can be used or native
implementation is required. 9792685
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.

4 participants