Skip to content
This repository has been archived by the owner on Feb 3, 2023. It is now read-only.

Profiling tools #2033

Open
wants to merge 13 commits into
base: develop
Choose a base branch
from
Open

Profiling tools #2033

wants to merge 13 commits into from

Conversation

StaticallyTypedAnxiety
Copy link
Contributor

PR summary

-Added profiling tools for running perf under conductor and sim2h_server
-Added tools for generating flame graph

testing/benchmarking notes

( if any manual testing or benchmarking was/should be done, add notes and/or screenshots here )

followups

( any new tickets/concerns that were discovered or created during this work but aren't in scope for review here )

changelog

  • if this is a code change that effects some consumer (e.g. zome developers) of holochain core, then it has been added to our between-release changelog with the format
- summary of change [PR#1234](https://github.com/holochain/holochain-rust/pull/1234)

documentation

@thedavidmeister
Copy link
Contributor

this doesn't work until the next version of holonix is cut and included

Copy link
Contributor

@thedavidmeister thedavidmeister left a comment

Choose a reason for hiding this comment

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

newest holonix includes perf so this should work

Copy link
Member

@zippy zippy left a comment

Choose a reason for hiding this comment

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

small changes needed

profiling/default.nix Outdated Show resolved Hide resolved
profiling/default.nix Outdated Show resolved Hide resolved
Copy link
Contributor

@thedavidmeister thedavidmeister left a comment

Choose a reason for hiding this comment

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

@AshantiMutinta actually i had a look and flamegraph exists in nix pkgs we should use it

image

@StaticallyTypedAnxiety
Copy link
Contributor Author

Wont work without this release

https://github.com/holochain/holonix/pull/131

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants