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

Test new and refactor step #66

Merged
merged 11 commits into from
May 31, 2022
Merged

Conversation

Juan-M-V
Copy link
Contributor

No description provided.

Removed test due to unknown details of the original representation. Mainly how and why are some big numbers (e.g. 106710729501573572985208420194530329073740042555888586719647) added to the memory.
@Juan-M-V Juan-M-V marked this pull request as ready for review May 31, 2022 15:21
@unbalancedparentheses unbalancedparentheses merged commit 99bb19d into main May 31, 2022
@unbalancedparentheses unbalancedparentheses deleted the test-new-and-refactor-step branch May 31, 2022 19:35
azteca1998 pushed a commit that referenced this pull request Feb 10, 2023
* Implement gas usage estimation

* Fix clippy

* Add tests

* Move value in test

* Change bigint to felt

* Delete macro bigint

* Add test for core function calculate_tx_gas_usage

* Fix error in multiplicaction
Oppen added a commit that referenced this pull request Apr 24, 2023
Version from whitelist `starknet/security/whitelists/cairo_keccak.json`,
differs from the current one in bounds for `_block_size`, `1000` instead
of `10`.
Oppen added a commit that referenced this pull request Apr 24, 2023
Version from whitelist `starknet/security/whitelists/cairo_keccak.json`,
differs from the current one in bounds for `_block_size`, `1000` instead
of `10`.
kariy pushed a commit to dojoengine/cairo-rs that referenced this pull request Jun 23, 2023
…ass#1041)

Version from whitelist `starknet/security/whitelists/cairo_keccak.json`,
differs from the current one in bounds for `_block_size`, `1000` instead
of `10`.
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.

None yet

2 participants