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

Ec pairing no hints #1020

Open
wants to merge 36 commits into
base: main
Choose a base branch
from
Open

Conversation

feltroidprime
Copy link
Contributor

@feltroidprime feltroidprime commented Nov 7, 2024

This change is Reviewable

manlikeHB and others added 30 commits September 30, 2024 09:59
* add validate_eth_tx test

* polish: fmt + docstrings
* First batch of replacements

* Second batch of replacements

* Clean commented code

* scarb fmt

* Apply code review recommendations
* ceil to bytes32

* overflow and nits
* load word to bytes

* unwrap and nits
* implement eth_get_balance

* add missing files

* made requested changes

* fmt

---------

Co-authored-by: enitrat <[email protected]>
* dev: use native in ci

* remove outdated gas reports

* fix fmt

* fix runtime location

* avoid clearing the ssj checkout

* use correct working dir

* update stack size when running rust ef-tests

* update workflows

* simplify workflows by avoiding re-building runtimes

* rework workflow structure
* dev: implement compile-time filtering

* fmt
* Implementation of eth_get_transaction_count function

* Refactoring validate nonce in validation.cairo

* Changing the deprecated function

* Validating nonce without new function

* Adding method to interface and test it

* fix tests

* fmt

---------

Co-authored-by: enitrat <[email protected]>
* fix: mulmod

* fmt
* fix: overflow in message_call_gas

* fmt
* fix typos

* fix typo

* fix typos

* fix typo

* fix typo
* fix: saturate jumpi index taken on stack

* scout: remove print
dev: optimized bitshifts by using a lookup table for powers of two

tmp
* fix: conversion of stack values in opcode execution

* fix test
* test_kakarot_core_get_starknet_address

* ci: downgrade cairo native (kkrt-labs#1008)

* dev: use checked math (kkrt-labs#1009)

* fix get_starknet_address test

---------

Co-authored-by: Mathieu <[email protected]>
* creating test case for emit events

* fixing format

* side stuff

* Update crates/evm/src/model.cairo

Co-authored-by: Mathieu <[email protected]>

* Update crates/evm/src/backend/starknet_backend.cairo

Co-authored-by: Mathieu <[email protected]>

* changes for implementation of  into trait

---------

Co-authored-by: enitrat <[email protected]>
Co-authored-by: Mathieu <[email protected]>
* dev: optimize to u64 words

* opti from_le_bytes

* dev: optimize ToBytes

* fmt

* fix bug
…krt-labs#1012)

* refactor: decode_legacy_tx for consistency with typed transactions

* fmt

---------

Co-authored-by: enitrat <[email protected]>
* refactor: eth_send_raw_transaction

* fix tests

* fix test

* revert unwanted change
* feat: safe get evm address

* ignore untestable tests
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.

10 participants