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

Tiny structural improvement of README. #199

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

apotonick
Copy link

Add some headers an example for mode: :object in the very top.

Thanks for this tool, I am in love with it! 💚

Add an example for `mode: :object` in the very top.
Copy link
Collaborator

@tenderlove tenderlove left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update the docs to differentiate between cpu and object profiling.

Thanks!

StackProf.run(mode: :object, out: "tmp/stackprof-cpu-myapp.dump") do
#...
end
```
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this is accurate. You should only switch from :cpu to :object when you want to change from profiling time to profiling object allocations. If you run with :cpu doesn't produce any samples it means that your sampling frequency might not be high enough. Measuring :cpu vs :object are completely different things (some code could take lots of time but allocate no objects, some code could run fast but allocate many objects).

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.

2 participants