-
Notifications
You must be signed in to change notification settings - Fork 39
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
MIVisionX compatibility fix - Resample and preemphasis filter #442
MIVisionX compatibility fix - Resample and preemphasis filter #442
Conversation
r-abishek
commented
Aug 27, 2024
- Move out the windowed_sinc function from rppdefs.h and readded vector usage for lookup table for compatibility
- Disable the numDims validation for preemphasis HOST kernel temporarily
…ith older rpp builds
disabled numDims validation check for preemphasis HOST kernel
…anges Resample and preemphasis filter compatibility changes for MIVisionX
@@ -316,4 +316,46 @@ void verify_non_silent_region_detection(int *detectedIndex, int *detectionLength | |||
qaResults << status << std::endl; | |||
qaResults.close(); | |||
} | |||
} | |||
|
|||
inline Rpp32f sinc(Rpp32f x) |
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.
when we enable RPP HIP audio, we need to remove these from here and MIVisionX. To do that you can use a new RPP version and check that
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.
Added some comment for future improvement
* modified windowed_sinc function to resolve the compatibility issues with older rpp builds * incremented RPP_VERSION_PATCH value disabled numDims validation check for preemphasis HOST kernel --------- Co-authored-by: sampath1117 <sampath.rachumallu@multicorewareinc.com>