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

Adding Read Benches, P50/90, UOPS, SPS, and additional refactors #46

Merged
merged 51 commits into from
Jun 12, 2024

Conversation

Angel-Petrov
Copy link
Contributor

Changelog

  • Refactored codebase to create Shooter trait for dev: enhance extensibility #43
  • Changed Mint Shooter to use multiple contracts as a solution around dev: ERC721 multi-account #29
  • Improved error handling so it is likely to panic in CI and instead produce report with less data
  • Updated other dependancies and removed unused colored crate
  • Verification times are now gotten from goose request data, improving accuracy of times
  • Adding read benchmarks which can be modified in gomu configuration files
  • Added P90/50 times for reads and write submissions/verifications
  • Added UOPS and SPS for write benchmarks
    • Made block info collection code multithreaded to counteract the extra requests needed for these metrics
  • Updated starknet dependancy to the one used in madara
    • Required to be able to get SPS metrics

src/actions/goose.rs Outdated Show resolved Hide resolved
src/actions/goose.rs Outdated Show resolved Hide resolved
src/actions/goose.rs Outdated Show resolved Hide resolved
src/actions/mod.rs Outdated Show resolved Hide resolved
src/actions/mod.rs Outdated Show resolved Hide resolved
src/actions/mod.rs Outdated Show resolved Hide resolved
src/actions/shooter.rs Outdated Show resolved Hide resolved
src/actions/shooter.rs Outdated Show resolved Hide resolved
src/metrics.rs Outdated Show resolved Hide resolved
src/metrics.rs Show resolved Hide resolved
src/actions/goose.rs Outdated Show resolved Hide resolved
@EvolveArt EvolveArt merged commit 99a8e4a into keep-starknet-strange:main Jun 12, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants