Skip to content

cv::TickMeter class addition#6821

Merged
opencv-pushbot merged 1 commit intoopencv:masterfrom
sturkmen72:TickMeter_class_addition
Jul 14, 2016
Merged

cv::TickMeter class addition#6821
opencv-pushbot merged 1 commit intoopencv:masterfrom
sturkmen72:TickMeter_class_addition

Conversation

@vpisarev
Copy link
Copy Markdown
Contributor

@vpisarev vpisarev commented Jul 8, 2016

Former #6584

resolves #6582

What does this PR change?

moves cv::TickMeter class in the core library

initial commit

@apavlenko
Copy link
Copy Markdown
Contributor

apavlenko commented Jul 8, 2016

Summarizing the discussion inside #6584 the cv::TickMeter class should use the std::steady_clock if C++11 is available and fall-back to the current implementation otherwise.
@sturkmen72 , do you plan to add the C++11 branch?

@alalek
Copy link
Copy Markdown
Member

alalek commented Jul 8, 2016

cv::TickMeter class should use the std::steady_clock

I believe cv::getTickCount should think about "steady_clock" or other implementations. TickMeter is just a wrapper over getTickCount and should be simple as much as possible.

@apavlenko
Copy link
Copy Markdown
Contributor

@alalek , sounds reasonable!
@sturkmen72 ?

@apavlenko apavlenko self-assigned this Jul 8, 2016
@apavlenko
Copy link
Copy Markdown
Contributor

OK, if so - let's merge this one and wait for a fix for the feature request #6902.
👍

@opencv-pushbot opencv-pushbot merged commit d2bad6f into opencv:master Jul 14, 2016
@sturkmen72 sturkmen72 deleted the TickMeter_class_addition branch July 14, 2016 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature Request cv::TickMeter

5 participants