Draft
Conversation
kariy
force-pushed
the
feat/update-gen-and-statefull-compr
branch
2 times, most recently
from
13f2ef1 to
6862b2f
Compare
Bumps `blockifier` to upstream tag [`v0.15.0-rc.2`]. This is necessary to make Katana works with the new [SNOS] implementation as it includes some changes in how the transaction fee is computed. SNOS is not compatible with the current version of `blockifier` due to a fee mismatch error. ## Database compatibility The tag introduces a change in the memory representation of `TransactionExecutionInfo` struct. This breaks compatibility with the current database format. --- We previously did a [bump] to v0.15.0-rc.3 but it was reverted due to the breaking database change. [`v0.15.0-rc.2`]: https://github.com/dojoengine/sequencer/commits/blockifier/v0.15.0-rc.2 [SNOS]: https://github.com/chudkowsky/snos-poc [bump]: #248
kariy
force-pushed
the
feat/update-gen-and-statefull-compr
branch
from
6862b2f to
dc11414
Compare
kariy
force-pushed
the
feat/update-gen-and-statefull-compr
branch
from
4a61e2d to
79a7d26
Compare
…tlement by default
kariy
force-pushed
the
main
branch
2 times, most recently
from
e3bd68b to
e897bbb
Compare
kariy
added a commit
that referenced
this pull request
Mar 19, 2026
Rebased version of #238 onto the latest main. Keeping #238 unchanged; this PR is the updated review target. ## Latest Changes ### fix: increase deploy_account test funding for Starknet v0.14.0 The Starknet version bump from 0.13.4 to 0.14.0 increased the OS resource overhead for `DeployAccount` transactions in the blockifier's versioned constants. Key changes in v0.14.0: - `execute_txs_inner.DeployAccount.constant.n_steps`: 4,161 → 4,583 (+422 steps) - `range_check_builtin`: 72 → 93 - `poseidon_builtin`: 0 → 11 (new) - Multiple syscall costs also increased (`StorageRead`, `StorageWrite`, `GetExecutionInfo`, `CallContract`, etc.) This made the previously hardcoded funding amount (`0x5ea0fb889c9400`) insufficient for the `deploy_account` RPC test. Updated the funding to 1 STRK (`0xde0b6b3a7640000`) for comfortable headroom. --------- Co-authored-by: Mateusz Chudkowski <mateuszchudy03@gmail.com> Co-authored-by: glihm <dev@glihm.net> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.