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.

Sign up for GitHub

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

Jump to bottom

fix(explorer): show correct cost per proof and remove inspects #830

Merged
merged 287 commits into from
Aug 27, 2024
Merged

fix(explorer): show correct cost per proof and remove inspects #830

merged 287 commits into from
Aug 27, 2024

Conversation

glpecile
Copy link
Contributor

@glpecile glpecile commented Aug 21, 2024 •
edited
Loading

Important

We should test in stage first and compare results.
This also needs a redeploy of the contracts.
This PR could have conflicts w/ #843.

Changes

Test

  • Start aligned locally
  • Run explorer and verify everything works, prices should now be higher than the deployed version

glpecile and others added 30 commits July 15, 2024 15:54
@glpecile
Merge branch 'main' into 605-feat-show-proof-hashes-in-each-batch
4505643
@uri-99
feat: fix update of batches
aa6bc3d
@uri-99
Merge branches '605-feat-show-proof-hashes-in-each-batch' and '605-fe…
e26ce19 
…at-show-proof-hashes-in-each-batch' of https://github.com/yetanotherco/aligned_layer into 605-feat-show-proof-hashes-in-each-batch
@uri-99 @glpecile
Update explorer/lib/explorer_web/live/pages/batch/index.html.heex
28fcd6c 
Co-authored-by: Gian <58370608+glpecile@users.noreply.github.com>
@uri-99
chore: detail on front
ba2d5cd
@glpecile
Merge branch '605-feat-show-proof-hashes-in-each-batch' into explorer…
14d4000 
…/598-feat-add-cost-per-proof-of-batch
@uri-99
fix: {,} on process_unverified_batches
88eb429
@uri-99
chore: remove commented code
f90fc30
@uri-99
chore: delete comment
f3b0db9
@glpecile
Merge branch 'main' into 605-feat-show-proof-hashes-in-each-batch
7b9bbaf
@glpecile
Merge branch 'main' into 605-feat-show-proof-hashes-in-each-batch
38a69da
@glpecile
Merge branch '605-feat-show-proof-hashes-in-each-batch' into explorer…
e62d060 
…/598-feat-add-cost-per-proof-of-batch
@glpecile
fix: merge error
62e0c07
@glpecile
feat: add usd prices
1d0a95c
@glpecile
build(deps): bump with mix deps.update --all
1419198
@glpecile
fix: add missing _ to variable
0f9f6e8
@glpecile
Update explorer/lib/explorer_web/live/utils.ex
90f312a
@glpecile
Merge branch 'main' into 605-feat-show-proof-hashes-in-each-batch
97299bb
@glpecile
Merge branch '605-feat-show-proof-hashes-in-each-batch' into explorer…
e9e7bc8 
…/598-feat-add-cost-per-proof-of-batch
@glpecile
refactor: move count operators to avs_directory instead of index
5d6eb5a
@glpecile
Merge branch 'main' into explorer/598-feat-add-cost-per-proof-of-batch
83da205
@uri-99
feat(wip): creating tables
4b4201d
@glpecile
feat(wip): add views for operator and assets
76ed35f
@uri-99
fix: ecto migrate of create restakings table
a352af6
@glpecile
fix: add missing attribute def
0606dd7
@uri-99
feat: make operators model
e9af4ee
@uri-99
Merge branch 'explorer/597-feat-add-validator-screen' of https://gith…
b475309 
…ub.com/yetanotherco/aligned_layer into explorer/597-feat-add-validator-screen
@uri-99
feat: tokens and restakings models
5cd7d90
@uri-99
fix: models
2db552c
@glpecile
refactor: move table to core components
d1cfdbc
glpecile added 17 commits August 23, 2024 19:28
@glpecile
chore(periodically): replace io inspect for logger debug
2c7dede
@glpecile
chore(avs-directory): replace io inspect for logger debug
61c4d7f
@glpecile
chore(delegation): replace io inspect for logger debug
d897769
@glpecile
chore(batches): replace io for logger
0623f86
@glpecile
chore(utils): replace io for logger
20440dd
@glpecile
chore(scripts-batches): replace io for logger
9aeabfe
@glpecile
chore: replace io for logger
b778b9e
@glpecile
chore(script-operators): replace dbg for logger
c2b72c1
@glpecile
chore(strategies): replace dbg for logger
824b4a7
@glpecile
chore(stake-registry): replace dbg for logger
4a07c69
@glpecile
chore(strategy-manager): replace dbg for logger
eb7b68b
@glpecile
chore(eigen-struct): replace dbg for logger
b3463d9
@glpecile
chore(operators): replace dbg for logger
2b7fd42
@glpecile
chore(quorums): replace dbg for logger
1a66571
@glpecile
chore(restakings): replace dbg for logger
6ab736e
@glpecile
chore: anvil deploy
439ae97
@glpecile
fix: use warning instead of warn
465ef7c
@glpecile glpecile changed the title fix(explorer): show correct cost per proof fix(explorer): show correct cost per proof and remove inspects Aug 23, 2024
@glpecile glpecile requested a review from uri-99 August 23, 2024 23:36
@glpecile glpecile added the frontend Frontend related issue. label Aug 23, 2024
@glpecile
Merge branch 'refs/heads/main' into explorer/742-fix-show-correct-cos…
ec8e795 
…t-per-proof

# Conflicts:
#	contracts/scripts/anvil/state/alignedlayer-deployed-anvil-state.json
#	contracts/src/core/BatcherPaymentService.sol
@glpecile
fix: update contract abi after merge
18c7b2e
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 27, 2024 •
edited
Loading

Changes to gas cost

Generated at commit: 8a7578659fb3741cb85d29bf1796f17bf4be8d65, compared to commit: 288db9e6c1b2f239411a2efa025551e96866097f

🧾 Summary (10% most significant diffs)

Contract Method Avg (+/-) %
AlignedLayerServiceManager createNewTask -4 ✅ -0.01%
Full diff report 👇
Contract Deployment Cost (+/-) Method Min (+/-) % Avg (+/-) % Median (+/-) % Max (+/-) % # Calls (+/-)
AlignedLayerServiceManager 4,359,036 (0) createNewTask 51,138 (0) 0.00% 51,323 (-4) -0.01% 51,240 (+6) +0.01% 51,876 (+24) +0.05% 256 (0)

@glpecile glpecile merged commit 61624d9 into main Aug 27, 2024
3 checks passed
@glpecile glpecile deleted the explorer/742-fix-show-correct-cost-per-proof branch August 27, 2024 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@uri-99 uri-99 uri-99 approved these changes

@entropidelic entropidelic entropidelic approved these changes

Assignees

@glpecile glpecile

Labels
frontend Frontend related issue.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

chore(explorer): replace inspect for logger fix(explorer): show correct cost per proof
3 participants
@glpecile @uri-99 @entropidelic