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

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

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

holochain/holonix#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.

5 participants