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

perf: add Krausest inspired tests as default test suite #5841

Closed
Tracked by #5845
prudepixie opened this issue Apr 14, 2022 · 1 comment · Fixed by #5921
Closed
Tracked by #5845

perf: add Krausest inspired tests as default test suite #5841

prudepixie opened this issue Apr 14, 2022 · 1 comment · Fixed by #5921
Assignees

Comments

@prudepixie
Copy link
Contributor

prudepixie commented Apr 14, 2022

This would be an important addition, as I'm adding tests to FAST libraries, there are a lot of repeated code that could be condensed. Most of the tests are run by creating # items and run # times through tachometer (with different implementation for benchmarking).
But essentially, we'd be interested in collecting similar data- where we define performance.mark and performance.measure- including start up metrics, CPU, memory (as Krausest framework).
Thus, it would benefit from having some templates that does this.

@prudepixie prudepixie self-assigned this Apr 14, 2022
@EisenbergEffect EisenbergEffect added this to the FASTElement 2.0 milestone Apr 18, 2022
@prudepixie prudepixie moved this from Todo to In Progress in FAST Architecture Roadmap Apr 25, 2022
@prudepixie
Copy link
Contributor Author

As I progressed in this ticket, I realized on top of having a default set of tests or operations to run for a benchmark, there should probably be an option to allow users to write their own 'operation' to test their benchmark. So in conjuncture with this work, I wanted to restructure the layout to make this possible.

Repository owner moved this from In Progress to Done in FAST Architecture Roadmap Apr 29, 2022
@prudepixie prudepixie linked a pull request May 24, 2022 that will close this issue
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment