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

Add option to output intermediate stacks #74

Closed
wants to merge 7 commits into from

Conversation

aparamon
Copy link

benfred and others added 5 commits January 15, 2019 20:42
Rather than shorten python filenames by removing the python install prefix,
only include directories that are python packages or subpackages
@aparamon
Copy link
Author

aparamon commented Feb 18, 2019

@benfred Anything from my side needed to get this (and #76) merged?

@okapies
Copy link

okapies commented Feb 21, 2019

I really need this feature to integrate with other visualizers like speedscope.

@aparamon aparamon mentioned this pull request Apr 11, 2019
@sophiebits-humu sophiebits-humu mentioned this pull request Apr 17, 2019
@Norbo11
Copy link

Norbo11 commented May 23, 2019

Will this be merged at any point?

@dongx3
Copy link

dongx3 commented Jul 23, 2019

@benfred any reason this PR not to be merged?

@benfred
Copy link
Owner

benfred commented Sep 23, 2019

Thanks for the PR !

Sorry about the delay on getting back to you on this - I wasn't sure about customizing the flamegraph code more since it was taken from rbspy, and there was some potential licensing headaches w/ py-spy being GPL and rbspy being MIT (like I wanted to merge this code and some others into a common library and would be complicated to do that with changes from other people in the py-spy variant). I've relicensed this entire project to MIT so get around this with 0.2.0 - and I've also added this functionality there.

@benfred benfred closed this Sep 23, 2019
@xmo-odoo
Copy link

@benfred maybe #45 could be closed then?

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.

6 participants