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.

Sign up for GitHub

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

Jump to bottom

Easy profiling option #1375

Draft

jcheng5 wants to merge 3 commits into main from profile-option

Collaborator

jcheng5 commented May 14, 2024

The idea is to add --profile to your shiny run command, and that will run Shiny under py-spy, then launch speedscope with the resulting trace.

TODO

On Windows, sys.orig_argv has two executables in the argv array (python.exe and shiny.exe), we just need the second. Why?
Currently pass the profiling data to speedscope.app via URL hash, need to work around the limits of URL hash sizes.

jcheng5 and others added 3 commits

May 13, 2024 21:41


          First pass at --profile switch

b8cc428


          Use pyinstrument instead of py-spy

ea1e082

I couldn't get py-spy working on Python 3.11
or later on Windows


          Remove py-spy code

309f650

jcheng5 force-pushed the profile-option branch from c3d6a35 to 309f650 Compare

May 15, 2024 06:06

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment