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 #245

Merged
merged 248 commits into from
Feb 7, 2024
Merged

Develop #245

merged 248 commits into from
Feb 7, 2024

Conversation

krlosMata
Copy link
Contributor

from develop to main

ignasirv and others added 30 commits October 14, 2023 15:26
@ignasirv @laisolizq
Fork id 6 implementation
6f87ae0
@ignasirv @laisolizq
Get GER from free input
fb43daf
@ignasirv @laisolizq
First refactor iteration -> full tracer
f5ff8ad
@zkronos73 @laisolizq
arith with complex numbers multiplication
3f7135b
@zkronos73 @laisolizq
fix bug on complex multiplication
cee5635
@zkronos73 @laisolizq
adding more tests and fix a bug
ba0e7f2
@zkronos73 @laisolizq
add custom helpers
bee8fb5
@zkronos73 @laisolizq
update helper method call
3724007
@zkronos73 @laisolizq
update field element name
b8cb2c1
@zkronos73 @laisolizq
add fullFe examples, pending sm_arith and pils
984e577
@zkronos73 @laisolizq
rename new arith instruction
fa6eac1
@zkronos73 @laisolizq
improvements on zkasmtest for big number of steps
bd39ede
@zkronos73 @laisolizq
add test for ADDPF2
ed7054c
@zkronos73 @laisolizq
Fix bug on arith helpers + improve tests
4920b71
@zkronos73 @laisolizq
fix bug on witness generation
896e322
@hecmas @laisolizq
Added the SUBFP2 instruction
915c484
@hecmas @laisolizq
Minor fix in arith equation selectors
32461e1
@hecmas @laisolizq
Bug found by Laia fixed
9d06226
@laisolizq
fix run-inputs custom helpers
9a52f21
@laisolizq
update package
2d3554c
@ignasirv @laisolizq
Fork id 6 implementation
b21727d
@laisolizq
update package & fix run-inputs
f7245eb
@laisolizq
update helpers
0685004
@laisolizq
fix helpers
eb47b72
@laisolizq
fix helpers
33b49d9
@krlosMata
WIP fork-etrog incliding l1InfoTree
f5970b6
@ignasirv
Unify full tracer
f2b089e
@krlosMata
Merge branch 'feature/forkid-6' of github.com:0xPolygonHermez/zkevm-p…
f28384c 
…roverjs into feature/forkid-6
@krlosMata
Merge pull request #222 from 0xPolygonHermez/feature/forkid-6-ft
ebff0ad 
Unify full tracer
@ignasirv @hecmas
Fork id 6 implementation
4292c8e
krlosMata and others added 23 commits January 25, 2024 23:23
@krlosMata
Merge pull request #70 from 0xPolygonHermez/feature/update-packages-e…
8d49d59 
…trog

update packages to last versions
@krlosMata
Merge pull request #12 from 0xPolygonHermez/feature/fork-etrog
0605eb3 
Feature/fork etrog
@krlosMata
remove internal dependencies
2e62bab
@ignasirv
Fix gha
0cb03b9
@ignasirv
Add geth oputput tests
07056d3
@ignasirv
Refactor ft tests prover for etrog
ffdb2b3
@ignasirv
Add geth oputput tests
44da371
@ignasirv
Refactor ft tests prover for etrog
3d8fb0f
@ignasirv
Check block info root at full tracer tests
0b6bf57
@ignasirv
Check block info root at prover ft tests
2a1504e
@ignasirv
Add new tests
464e840
@ignasirv
Remove internal references
c23e230
@krlosMata
Merge pull request #234 from 0xPolygonHermez/feature/block-info-root-2
d68fac3 
Feature/block info root 2
@krlosMata
Merge branch 'develop' into develop-etrog
e58a270
@zkronos73
update package-lock.json
ede5454
@krlosMata
set timestamp reading from state when skipping changeL2Block
bfcf7bc
@krlosMata
comments remove setGlobalVar
15d52c2
@krlosMata
Merge pull request #236 from 0xPolygonHermez/fix/load-timestamp
c3cab39 
set timestamp reading from state when skipping changeL2Block
@krlosMata
update some packages
4a809a2
@ignasirv
Fix block hash at full tracer
bb549d9
@krlosMata
Merge pull request #243 from 0xPolygonHermez/feature/fix-block-hash
f0bae64 
Fix block hash at full tracer
@krlosMata
update to last versions
71dcdf3 
finish update packages
@krlosMata
Merge pull request #244 from 0xPolygonHermez/feature/final-versions
f52d312 
Feature/final versions
@cla-bot cla-bot bot added the cla-signed label Feb 7, 2024
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Feb 7, 2024

Quality Gate Failed Quality Gate failed

Failed conditions

E Security Rating on New Code (required ≥ A)
D Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

idea Catch issues before they fail your Quality Gate with our IDE extension SonarLint SonarLint

ignasirv
ignasirv approved these changes Feb 7, 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.

💯

zkronos73
zkronos73 approved these changes Feb 7, 2024
View reviewed changes
Copy link
Contributor

@zkronos73 zkronos73 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 f5922cc into main Feb 7, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zkronos73 zkronos73 zkronos73 approved these changes

@ignasirv ignasirv ignasirv approved these changes

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

Successfully merging this pull request may close these issues.
7 participants
@krlosMata @ignasirv @zkronos73 @hecmas @laisolizq @jbaylina @rickb80