artemis/breezewiki
1
0
Fork 0
forked from cadence/breezewiki
Code Issues Pull requests 2 Projects Releases Packages Wiki Activity

flamegraph #2

Open
artemis wants to merge 2 commits from flamegraph into main
pull from: flamegraph
merge into: artemis:main
Conversation 0 Commits 2 Files changed 1 +30 -3
artemis commented 2023-03-23 05:49:36 +00:00
Owner
Copy link

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
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 switch flamegraph

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff flamegraph
git switch flamegraph
git rebase main
git switch main
git merge --ff-only flamegraph
git switch flamegraph
git rebase main
git switch main
git merge --no-ff flamegraph
git switch main
git merge --squash flamegraph
git switch main
git merge --ff-only flamegraph
git switch main
git merge flamegraph
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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.