-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
EVM blockchain metrics #6502
EVM blockchain metrics #6502
Conversation
On issue I see here is that you can't properly aggregate the address and contract metrics (if you want to show daily or weekly stats in the dashboard, there's no way to correctly calculate these from the hourly stats as they are dependent on the time window) |
@0xRobin That's a good point. And even if the timeframe was daily, it would push the problem to broader timeframes like weekly or monthly. The new_addresses and new_contracts are ok at least, because they only ever get counted once. But we should remove active_addresses, returning addresses, and active_contracts. |
0306e22
to
2ee3fd7
Compare
expected schema, per requirements doc:
columns removed, per feedback:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, waiting for final approval from @hannahblue before merge
This looks good. Thank you @jeff-dude and @0xRobin |
EVM blockchain metrics