mention in the function description if it's TBB-accelerated (Feature #421)
Description
As written in the summary, I think it would be important to have in the documentation of each function an indication that such function uses or does not use OpenMP for its computations.
[other things useful to know for each function, at least for the C/C++ interface, would be the header file and the library where such function is declared/implemented]
Associated revisions
Mentioned in doc if a function is parallelized with the TBB library (issue #421)
Merge pull request #421 from asmaloney:check-mem-alloc2
History
Updated by Sebastian Krämer over 14 years ago
You can assume that no functions are OpenMP-accelerated any more. That support was stripped from OpenCV some time after 2.0 release I think. Now Intel's TBB library is used for some parallelization.
Updated by Francesco Montorsi over 14 years ago
Replying to [comment:1 decision]:
You can assume that no functions are OpenMP-accelerated any more. That support was stripped from OpenCV some time after 2.0 release I think. Now Intel's TBB library is used for some parallelization.
ok, let me update this ticket then: I think it would be important to have in the documentation of each function an indication that such function uses or does not use TBB for its computations, i.e. if it works on parallel cores when available or not.
AFAICS functions TBB-accelerated are those using one of the template constructs:
void parallel_for( const [[BlockedRange]]& range, const Body& body ) void parallel_do( Iterator first, Iterator last, const Body& body ) void parallel_reduce( const [[BlockedRange]]& range, Body& body )
right?
Grepping OpenCV files it looks like not so many functions are using them... at least this means that documenting such functions as TBB-accelerated should be easier :)
Updated by Alexander Shishkov about 13 years ago
- Description changed from As written in the summary, I think it would be important to have in the docum... to As written in the summary, I think it would be important to have in the docum... More
Updated by Ilya Lysenkov almost 13 years ago
- Target version set to 2.4.0
- Assignee set to Ilya Lysenkov
Updated by Ilya Lysenkov almost 13 years ago
Thank you for the report. It was documented in r8038.
- Status changed from Open to Done