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

Develop #134

Merged
merged 53 commits into from
Feb 1, 2024
Merged

Develop #134

merged 53 commits into from
Feb 1, 2024

Conversation

krlosMata
Copy link
Contributor

  • etrog version from develop to main

ignasirv and others added 30 commits September 12, 2023 17:00
@ignasirv
Fork id 6 implementation
882d322 
Move isForced to batch level

Update tests to fork id 6

fix header timestamp

Implement forced batches logic

Remove GER from batchData

Set block gas limit
@ignasirv
Secured block hash implementation
ad6c4c5 
Store block info tree root at the end of tx processing
@ignasirv
Fix log index
e966198
@ignasirv
Comments apply
d0bc991
@laisolizq
fix TX_GAS_LIMIT
dbbe44b
@laisolizq
update check tx_gas_limit
2a19edd
@krlosMata
Merge pull request #113 from 0xPolygonHermez/feature/forkid-6-fix-gas
687b197 
fix TX_GAS_LIMIT
@krlosMata
add l1InfoTree
60f62e6
@krlosMata
Merge pull request #118 from 0xPolygonHermez/feature/l1-info-tree-final
3e71479 
add l1InfoTree
@krlosMata
add compute global index
ab71b56
@krlosMata
Merge pull request #119 from 0xPolygonHermez/feature/add-compute-gi
13a353c 
add compute global index
@zkronos73
update with new storage requerimients, add temporal logs
cbb1b1c
@krlosMata
add flags to skipchangeL2Block & skipWriteBlockInfoRoot
5c19cd8
@zkronos73
update siblingChild as single element
cd0f7fe
@ignasirv
Merge pull request #120 from 0xPolygonHermez/feature/skip-first-chang…
3c5dee6 
…e-l2-block

add flags to skipChangeL2Block & skipWriteBlockInfoRoot
@ignasirv
Add effectiver percentage at l2 tx hash
2e66bc2
@ignasirv
Unit testing for computeL2TxHash function
c233000
@ignasirv
Refactor secure effective percentage
e7b0f39
@krlosMata
Merge pull request #122 from 0xPolygonHermez/feature/ep-l2txhash
71f51d2 
Feature/ep l2txhash
@ignasirv
PR review resolution
7015711
@zkronos73
rename some variables
836dac3
@ignasirv
Unit testing blockInfoTree computation
a1675af
@ignasirv
Review
dbb2da6
@krlosMata
Merge pull request #123 from 0xPolygonHermez/feature/PR-review
57fa389 
PR review resolution
@krlosMata
add tests different batches
3033c81
@krlosMata
Merge pull request #124 from 0xPolygonHermez/feature/tests-skip
402901f 
add tests different batches
@zkronos73
remove devel logs
b63ff87
@zkronos73
fix lint errors
ac7e37e
@krlosMata
Merge pull request #121 from 0xPolygonHermez/feature/climb-key-sm
3c80751 
update smt class with etrog storage changes
@ignasirv
Update monorepo package
c0d872b
krlosMata and others added 23 commits December 11, 2023 12:09
@krlosMata
Merge pull request #125 from 0xPolygonHermez/feature/update-monorepor
a142793 
Update monorepo package
@ignasirv
Update vm version
0115219
@krlosMata
Merge pull request #127 from 0xPolygonHermez/feature/update-vm
d0bb993 
Update vm version
@ignasirv
Increase block gas limit to 2**64
48ffaac
@ignasirv
Update tests
fa5cef8
@krlosMata
Merge pull request #128 from 0xPolygonHermez/feature/cumulative-gas-fix
9da5cd1 
Increase block gas limit to 2**64
@ignasirv
New l2 tx hash computation
a61bc3a
@ignasirv
Fix typo chainId -> chainID
0d70c21
@krlosMata
Merge pull request #130 from 0xPolygonHermez/feature/new-l2hash
cc59e7d 
Feature/new l2hash
@krlosMata
Merge branch 'develop' into feature/fork-etrog
227349c
@krlosMata
Merge pull request #114 from 0xPolygonHermez/feature/fork-etrog
2aa8a9a 
Feature/fork etrog
@ignasirv
Add block utils at index
fa7bb0d
@krlosMata
Merge pull request #131 from 0xPolygonHermez/feature/expose-block-utils
97d546f 
Add block utils at index
@ignasirv
Virtual counters manager implementation
ec30358
@ignasirv
ecrecover counter compute improvement
32fce30
@ignasirv
Add new l2tx hash counters
9463551
@ignasirv
Improve virtual counters
8a7b3cd
@krlosMata
fix utils processor
5389d24
@ignasirv
Update initial mcpl
b802e70
@krlosMata
Merge pull request #132 from 0xPolygonHermez/feature/fix-utils-processor
dd488e9 
fix utils processor
@ignasirv
PR review
53ca9b0
@ignasirv
Fix poseidon exceed
3897398
@krlosMata
Merge pull request #133 from 0xPolygonHermez/feature/vc-improvement
8e09d92 
Feature/vc improvement
@cla-bot cla-bot bot added the cla-signed label Feb 1, 2024
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Feb 1, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

47 New issues
0 Security Hotspots
No data about Coverage
1.8% Duplication on New Code

See analysis details on SonarCloud

ignasirv
ignasirv approved these changes Feb 1, 2024
View reviewed changes
Copy link
Contributor

@ignasirv ignasirv left a comment

Choose a reason for hiding this comment

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

💯

@krlosMata krlosMata merged commit fefa98e into main Feb 1, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ignasirv ignasirv ignasirv approved these changes

@laisolizq laisolizq Awaiting requested review from laisolizq laisolizq is a code owner

@invocamanman invocamanman Awaiting requested review from invocamanman invocamanman is a code owner

Assignees
No one assigned
Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@krlosMata @ignasirv @laisolizq @zkronos73