-
-
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
cupyx/scipy.signal: port *ord filter design functions from scipy.signal #7632
Conversation
These are fixed up and included in cupy#7632
0db80b4
to
db46a78
Compare
w, h = scp.signal.sosfreqz(sos) | ||
return w, h | ||
|
||
@testing.with_requires("mpmath > 0.10") |
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.
I'll make another PR to make CI install mpmath.
/test mini |
Rebased on main, now that gh-7537 is in. |
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!
I'll kick the CI after #7653 is merged. |
/test full |
Just copy-pasted from scipy. This implementation is pure python and thus horribly inefficient.
/test full |
/test full |
This pull request is now in conflicts. Could you fix it @ev-br? 🙏 |
/test full |
CI failures are not related. |
Thanks, @ev-br! |
cross-ref gh-7403
Port order selection functions for several types of filters
.
cupyx.scipy.signal._optimize
moduleThis PR is on top of gh-7537