flamegraph #2

Open
artemis wants to merge 2 commits from flamegraph into main
Owner

add flamegraph profiling to the tests and switch the tests to time minecraft/Block instead of minecraft/Frog

you need to

raco pkg install profile-flame-graph

for this to work. i wasnt really sure if there was a way to add that as a test-dependency, or make it only import/run if profile-flame-graph is actually installed. you would know better than me.

you also need flamegraph.pl installed which may or may not be available from your package manager. on gentoo it's dev-util/FlameGraph. I think https://github.com/brendangregg/FlameGraph is the source

anyways it will generate the flamegraphs for u automatically after that in /storage

also you need to use raco test --errortrace so that the errortrace library is loaded up for the errortrace version of the flamegraph

add flamegraph profiling to the tests and switch the tests to time minecraft/Block instead of minecraft/Frog you need to ``` raco pkg install profile-flame-graph ``` for this to work. i wasnt really sure if there was a way to add that as a test-dependency, or make it only import/run if profile-flame-graph is actually installed. you would know better than me. you also need `flamegraph.pl` installed which may or may not be available from your package manager. on gentoo it's `dev-util/FlameGraph`. I think https://github.com/brendangregg/FlameGraph is the source anyways it will generate the flamegraphs for u automatically after that in /storage also you need to use `raco test --errortrace` so that the errortrace library is loaded up for the errortrace version of the flamegraph
artemis added 2 commits 2023-03-23 05:49:37 +00:00
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin flamegraph:flamegraph
git checkout flamegraph

Merge

Merge the changes and update on Forgejo.
git checkout main
git merge --no-ff flamegraph
git checkout main
git merge --ff-only flamegraph
git checkout flamegraph
git rebase main
git checkout main
git merge --no-ff flamegraph
git checkout main
git merge --squash flamegraph
git checkout main
git merge --ff-only flamegraph
git checkout main
git merge flamegraph
git push origin main
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: artemis/breezewiki#2
No description provided.