Skip to content

feat: fee algo performance metrics #215

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

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

segfault-magnet
Copy link
Contributor

Related to: #214

Before updating the price algo I decided to update the simulator so we can get actual numbers of saved eth against which to compare the new algo.

You can now configure how big a bundle is (eg 3600 fuel blocks), how many blobs that is (eg 3 eth blob) and how long does it take for one eth tx to finalize (eg 5 mins). When ran on the last month of eth pricing data:
fees png

This is good enough for now, but ideally we'd want a tool that can connect to the prod db and give insight into expected vs realized fees, as these can differ depending on time to inclusion.

@segfault-magnet segfault-magnet added the enhancement New feature or request label Mar 20, 2025
@segfault-magnet segfault-magnet self-assigned this Mar 20, 2025
@@ -1,267 +1,5 @@
use std::time::Duration;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly moved into a separate file.

@@ -0,0 +1,200 @@
use std::num::NonZeroU64;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nothing much changed, mostly just moved from the parent mod.

Copy link

@digorithm digorithm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is very neat!

@hal3e hal3e requested a review from a team as a code owner April 3, 2025 15:04
Copy link

@AurelienFT AurelienFT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nothing shocked me and I was able to understand easily the high level design

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants