-
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
RPP OpenMP threads update #143
Conversation
…d crop and cmn augmentations made changes to set numthreads as batchsize if numthreads value is 0
Changes for setting number of threads used in OpenMP for HOST functions
@r-abishek does this PR break functionality/build with the current version of MIVisionX? if yes, can we update the RPP version to |
@kiritigowda Yes, the numThreads needs to be passed to RPP functions from MIVisionX, and that will be a separate PR in MIVisionX. We'll run tests and confirm compatibility next week before merging either PRs. |
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.
please address review comments
added description for numThreadsin readme and test suite files
Default value for numThreads used for OpenMp
Added default value for numthreads for internal handle creation call in HIP
@kiritigowda This PR can be merged along with PR1060 in MIVisionX |
RPP Version update for backward compatibility failure
@kiritigowda @rrawther This PR needs to be merged together with an upcoming PR in MIVisionX for ToT to be in sync.
The MIVisionX PR will be issued in the next few days. The RPP#143 and MIVisionX PRs can be tested merged after that.
Adds numThreads as a parameter in RPP handle for HOST backend functionalities.
Modifies all BatchPD and Tensor functionalities with a common change to use this numThreads parameter