-
-
Notifications
You must be signed in to change notification settings - Fork 9.5k
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
ENH: Add SIMD operations for min and max value comparision #18889
Conversation
ebbc5d0
to
71644f0
Compare
0f0bbf9
to
b3ea1c0
Compare
b3ea1c0
to
111f35d
Compare
@howjmay, Could you please add a testing case for integer min/max? |
111f35d
to
da0bb26
Compare
@seiko2plus Is there anything I should fix or add? |
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.
The current testing case only tests f32/f64, you have to add a new test case for integer max/min at the tail of the following class
numpy/numpy/core/tests/test_simd.py
Lines 158 to 161 in c753b23
class _SIMD_INT(_Test_Utility): | |
""" | |
To test all integer vector types at once | |
""" |
22b1331
to
d8ae6e0
Compare
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.
fix unit test cases
d8ae6e0
to
2618ff5
Compare
2618ff5
to
e755377
Compare
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, Thank you.
Thank you so much for helping |
Thanks @howjmay . |
Add SIMD operations for comparing maximum and minimum values.
npyv_[max|min]_u64
is not implemented in this PR, since AVX2 has some problems on implementing them.