Skip to content
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

Let's create an option to stop or pause profiling. #4

Closed
GoogleCodeExporter opened this issue May 12, 2015 · 7 comments
Closed

Let's create an option to stop or pause profiling. #4

GoogleCodeExporter opened this issue May 12, 2015 · 7 comments

Comments

@GoogleCodeExporter
Copy link

It is very useful sometimes to have an option to start/stop profiling by 
demand. It allows us to look at results, investigate it etc., especially if we 
did profiling run for specific time of app work.

The best solution would be to implement kind of 'project' / 'workspace' 
objects, but it would require too much development for now I guess.

It also makes sense to allow to start profiler being turned OFF initially 
somehow (as optional flag).

Original issue reported on code.google.com by 4ekme...@gmail.com on 18 Oct 2011 at 2:56

@GoogleCodeExporter
Copy link
Author

Original comment by artem.ge...@gmail.com on 20 Oct 2011 at 6:47

  • Added labels: Type-Enhancement
  • Removed labels: Type-Defect

@pk1954
Copy link

pk1954 commented Jan 16, 2020

I support this feature request. Sometimes it ist necessary to do some preparations before the application comes into a state where the time critical action happens. If there was a way to turn profiling on only for the time critical phase, the profiling output would be "cleaner" and more significant.
I don't quite understand, what the 'project' / 'workspace' objects in the original request means. I could imagine either an API call to microprofiler, or an option in the debugger interface.

@SergeLalonde
Copy link

If you're going to have Start and Stop, you should have a Clear operation so that the profiling data collected so far can be cleared to start over.
These features would make this a killer extension (and it's already really good).

@tyoma
Copy link
Owner

tyoma commented Feb 4, 2020

@SergeLalonde the 'Clear' command is already there - it's the cross in the tiny toolbar on top of the statistics lists ('Clear' button in the standalone version).

@tyoma
Copy link
Owner

tyoma commented Feb 4, 2020

@pk1954 The author of that suggestion originally meant both - API and the toggle button. I'm going to implement the toggle button sooner. The API puts some implications on how to make it available with the minimal hassle, especially considering that I plan to eliminate the necessity of micro-profiler.initializer.cpp inclusion (yes, the only thing to include to the project will be the .lib for the collector's dll/so).

@tyoma
Copy link
Owner

tyoma commented Feb 4, 2020

@SergeLalonde, @pk1954 Guys, please check out this version - is that what you wanted?
https://github.com/tyoma/micro-profiler/releases/download/v1.5.613.0/micro-profiler.v1.5.613.0.vsix

The change is here: b88ebb7

@pk1954
Copy link

pk1954 commented Feb 10, 2020

Yes, that's fine as for me.

@tyoma tyoma closed this as completed Feb 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants