-
Notifications
You must be signed in to change notification settings - Fork 5
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
Changes for setting number of threads used in OpenMP for HOST functions #120
Changes for setting number of threads used in OpenMP for HOST functions #120
Conversation
sampath1117
commented
Mar 31, 2023
- Added another parameter numThreads in handle structure
- Updated batchsize to numThreads for OpenMP to all tensor HOST functions and batchpd HOST functions
- Updated test suite with latest function used for handle creation
…d crop and cmn augmentations made changes to set numthreads as batchsize if numthreads value is 0
@@ -39,6 +39,7 @@ extern "C" { | |||
// *param[in] kernelSize kernel size for erode (a single Rpp32u odd number with kernelSize = 3/5/7/9 that applies to all images in the batch) | |||
// *param[in] roiTensorSrc ROI data for each image in source tensor (2D tensor of size batchSize * 4, in either format - XYWH(xy.x, xy.y, roiWidth, roiHeight) or LTRB(lt.x, lt.y, rb.x, rb.y)) | |||
// *param[in] roiType ROI type used (RpptRoiType::XYWH or RpptRoiType::LTRB) | |||
// *param[in] rppHandle HOST-handle/HIP-handle for "_gpu" variants and Host-handle for "_host" variants |
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.
HOST-handle/HIP-handle for "_gpu" variants and Host-handle for "_host" variants
Why is it "HOST-handle/HIP-handle for "_gpu" variants"?
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.
Does batchPD code work with any handle type?
@sampath1117 Hope you've verified working of previous/new test suites with this set of common changes |