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

feat: Use l1 real gas #59

Closed
wants to merge 30 commits into from
Closed

feat: Use l1 real gas #59

wants to merge 30 commits into from

Conversation

toni-calvin
Copy link

What ❔

Why ❔

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted via zk fmt and zk lint.
  • Spellcheck has been run via zk spellcheck.

toni-calvin and others added 30 commits January 5, 2024 15:47
@toni-calvin
set pubdata constants to 0
c944c92
Set to 0
837cf6f
Merge branch 'validium_mode_pubdata_price' into validium_mode_pubdata_0
181d80c
@jorbush
Create contract for writing bytes
cb7edd8
@jorbush
Deploy bytes contract
714bd35
@toni-calvin
update full node to use rpc providers
3f91e20
@toni-calvin
fmt
c3373d2
@toni-calvin
add colored prints
ce4de82
@toni-calvin
add better debug logs
cc32b58
@toni-calvin
add better logs
f8edfa4
@toni-calvin
Merge branch 'validium_mode_pubdata_0' of github.com:lambdaclass/zksy…
27ab211 
…nc-era into test-erc20-pubdata-0
@toni-calvin
add erc20 contract and update test
240c0a8
@toni-calvin
update logs
c683f9a
@toni-calvin
print gas used
4699ae9
@Oppen
Use ETH_SENDER_SENDER_VALIDIUM_MODE to check whether we need to cha…
215b9d9 
…rge the pubdata
@Oppen
wip
5d3d96c
@Oppen
wip 2: just change runtime behavior
c71ff9c
@Oppen
Restore original values
183957d
@jordibonet-lambdaclass @toni-calvin
Add validium example readme (#34)
cff8bfd 
* Add validium example readme

* update validium.md and change test name

* fmt

---------

Co-authored-by: toni-calvin <[email protected]>
@Oppen
Only keep changes for vm_latest
5bbeb3c
@Oppen
Fix submodule
aad41bb
@Oppen
Fix integration test license
64e639a
@mationorato
change contracts branch
557ec52
Remove l1 gas data
b9ceb92
Update submodule branch
3aa9f50
@jordibonet-lambdaclass
fix: update output readme (#39)
479061d 
* Update output readme

* Add a note for transactions

* Fix note
@toni-calvin
feat: remove logs pubdata (#42)
ef48b51 
* remove logs from pubdata

* update comment
@jordibonet-lambdaclass
feat: refactor readme example (#44)
5c3dfc5 
* Refactor readme example

* Fix some comments

* Remove validium.md
@toni-calvin
set real l1 gas
051b983
@toni-calvin
fmt
8cd527f
@toni-calvin toni-calvin linked an issue Jan 18, 2024 that may be closed by this pull request
Generate gasUsed results for real L1 gas prices #54
Closed
Base automatically changed from validium_mode_pubdata_0_env to validium_mode January 19, 2024 14:22
@toni-calvin toni-calvin changed the base branch from validium_mode to validium_mode_new_fee_model_final January 28, 2024 21:19
@toni-calvin toni-calvin changed the base branch from validium_mode_new_fee_model_final to validium_mode January 28, 2024 21:19
@toni-calvin toni-calvin closed this Feb 1, 2024
@ilitteri ilitteri deleted the use-l1-real-gas branch February 9, 2024 15:58
Oppen pushed a commit that referenced this pull request Feb 9, 2024
@StanislavBreadless
Allow ts-ignore (#59)
97208b7 
allow ts ignore
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
external-contribution
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Generate gasUsed results for real L1 gas prices
5 participants
@toni-calvin @jorbush @Oppen @jordibonet-lambdaclass @mationorato