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

Continous Benchmarking #117

Closed
zbraniecki opened this issue Jun 6, 2020 · 2 comments · Fixed by #197
Closed

Continous Benchmarking #117

zbraniecki opened this issue Jun 6, 2020 · 2 comments · Fixed by #197
Assignees
Labels
A-performance Area: Performance (CPU, Memory) C-test-infra Component: Integration test infrastructure T-enhancement Type: Nice-to-have but not required
Milestone

Comments

@zbraniecki
Copy link
Member

With https://github.com/rhysd/github-action-benchmark we can get https://rhysd.github.io/github-action-benchmark/dev/bench/ for our components.

Seems like a good investment to monitor for performance per-component.

@sffc sffc added A-performance Area: Performance (CPU, Memory) C-meta Component: Relating to ICU4X as a whole T-enhancement Type: Nice-to-have but not required C-test-infra Component: Integration test infrastructure and removed C-meta Component: Relating to ICU4X as a whole labels Jun 8, 2020
@sffc sffc added this to the 2020 Q3 milestone Jun 17, 2020
@echeran
Copy link
Contributor

echeran commented Jun 17, 2020

This could also include some of the tools mentioned in https://blog.tonari.no/why-we-love-rust, like cargo flamegraph

@zbraniecki
Copy link
Member Author

I started using flamegraph over the last two days and it's pure magic and unicorns. It works great.

I don't know if we'll be able to hook it into continuous visualization but I am going to use it in my work and recommend it to every one :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-performance Area: Performance (CPU, Memory) C-test-infra Component: Integration test infrastructure T-enhancement Type: Nice-to-have but not required
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants