-
-
Notifications
You must be signed in to change notification settings - Fork 778
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
add kaiser{ord,_beta,_atten} functions #7635
Conversation
/test mini |
Rebased on main. |
/test full |
/test full |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Waiting for CI finishes.
/test full |
This pull request is now in conflicts. Could you fix it @ev-br? 🙏 |
/test full |
This pull request is now in conflicts. Could you fix it @ev-br? 🙏 |
/test full |
/test mini |
For record: These functions take Python scalars and return them as well, where computing on CPU is enough. An option is depending on SciPy then having aliases to it, but here we're independent of SciPy and have the same ones on CuPy. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
CI failures are not related. |
Thanks, @ev-br! |
cross-ref #7403
Add
on top of gh-7537