Repository navigation
Implement .prof output - #13
Merged
sunhailin-Leo merged 1 commit intoMay 5, 2023
Merged
Conversation
msnidal
commented
Apr 30, 2023
msnidal
commented
Apr 30, 2023
Owner
|
LGTM~ |
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
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.
Nice profiler! I used it on a project and got some good usage out of it, as #9 raised though there is a lot of Python tooling out there to support
.proffiles such as snakeviz, etc. so I thought I'd add support for cProfile.In addition to the new code I added an example and test for it - although the test is rudimentary (it just checks whether the
.proffile exists and is non-empty) I had success on my machine getting the.profoutput and loading it into snakeviz! Thanks again for the profiler.