Plot improvements#57
Merged
fabianp merged 5 commits intopythonprofilers:masterfrom Aug 27, 2013
Merged
Conversation
This option allows to set the title of figures created by 'mprof plot'.
Numpy is still required for plotting, but only when pylab is used.
When interrupting "mprof run", an empty file can be generated, which caused problem with "mprof plot". A clear error message is now displayed to the user.
The full command line of what was run during the profiling is now saved into the profile file. It is used to set the graph title with 'mprof plot'.
Memory usage when entering and leaving a function is now included with the other measurements. This fixes some discrepancies between brackets and curves when sampling frequency was low. Add an option to prevent displaying of timestamps in plots (mprof plot -n)
Collaborator
|
This is great Philippe, I'll have to upgrade you to co-author of the package :-) (to be fair by now you probably have more code than me). |
Contributor
|
I think this could be better documented. I did not know this was an option until I came here looking for similar issues. Anyway, these are great features. |
Collaborator
|
Pull requests welcomed :-)
…On Tue, Aug 28, 2018, 16:48 Alexandre Pinto ***@***.***> wrote:
I think this could be better documented. I did not know this was an option
until I came here looking for similar issues. Anyway, these are great
features.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#57 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAQ8h0jFSXzFRPziU8jg4gVsBe-0PB_oks5uVVgbgaJpZM4A8GiW>
.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I added a couple of improvements, mainly related to plotting:
mprof plotto manually set the figure titlemprof plotto prevent display of function timestampsPlus some code cleanup.