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

Support multicall3 in status endpoint #151

Merged
merged 2 commits into from
Sep 6, 2024
Merged

Support multicall3 in status endpoint #151

merged 2 commits into from
Sep 6, 2024

Conversation

jbrower95
Copy link
Contributor

@jbrower95 jbrower95 commented Sep 6, 2024

  • Support multicall3 (https://www.multicall3.com/) in our status endpoint.
  • You can totally see how some of these calls should be codegenned by abigen, which we can iterate on.

Before: ./cli status --podAddress 0x2641C2ded63a0C640629F5eDF1189e0f53C06561 3.60s user 4.62s system 5% cpu 2:38.98 total
After: ./cli status --podAddress 0x2641C2ded63a0C640629F5eDF1189e0f53C06561 1.71s user 1.00s system 25% cpu 10.646 total

2:38m -> 10.6s

Tested by running on master/branch for a particular pod and asserting the JSON output. Used a graphical visualizer here so if any keys mismatched, I could figure out which ones. None did.
image

wooo

@jbrower95 jbrower95 marked this pull request as ready for review September 6, 2024 21:58
@jbrower95 jbrower95 merged commit c0f3579 into master Sep 6, 2024
4 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.

2 participants