Sync main #50
Open
Sync main #50
Conversation
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
… storage (starkware-libs#5011) For forward-compatibility.
Signed-off-by: Dori Medini <[email protected]>
…are-libs#4967) Most of this is boilerplate, major changes are: 1. catch up height is now lazily initialized by the spawned sync task which continously asks the sync state for its height. This can be None if the sync is starting up. 2. Until catch up height is ready, the rest of the startup flow is unchanged, with the addition that before the spawned task starts generating commit-blocks to sync up the provider, it'll first update the catch up height, which is synced with the bootstrapper. Co-authored-by: Gilad Chase <[email protected]>
…-libs#5013) * chore(starknet_sequencer_node): format deployment variants * chore(starknet_sequencer_node): service name to service fn
* feat(starknet_os): copy store_da_segment implementation Signed-off-by: Dori Medini <[email protected]> * feat(starknet_os): fix store_da_segment implementation * refactor(starknet_os): polynomial_coefficients_to_kzg_commitment directly outputs felts Signed-off-by: Dori Medini <[email protected]> --------- Signed-off-by: Dori Medini <[email protected]>
* feat(starknet_os): copy store_da_segment implementation Signed-off-by: Dori Medini <[email protected]> * feat(starknet_os): fix store_da_segment implementation * refactor(starknet_os): polynomial_coefficients_to_kzg_commitment directly outputs felts Signed-off-by: Dori Medini <[email protected]> * test(starknet_os): add small fft regression test --------- Signed-off-by: Dori Medini <[email protected]>
…tarkware-libs#5030) * chore(starknet_sequencer_node): use service enum instead of const * chore(starknet_sequencer_node): bundle all deployments in an array
…ages to test setup (starkware-libs#4784)
…ared_at without impl in server (starkware-libs#5053)
…libs#5063) As prep. for addition of a class object label, to partition by Sierra, Casm, deprecated Casm.
…tarkware-libs#5031) * chore(starknet_sequencer_node): use service enum instead of const * chore(starknet_sequencer_node): remove redundant lifetime generics
Use the client, rather than direct l1_provider calls. This is relevant here because the async task is already using the (mock) client, and making the test use the provider directly while the sync task is using the client will be too different from prod code. Also fixed an incorrect copy-paste code remark. Co-authored-by: Gilad Chase <[email protected]>
…s#5188) Also add util for flushing the fake l1 client: its currently just buffering the calls it gets, and needs a manual flush. Co-authored-by: Gilad Chase <[email protected]>
* feat(starknet_os): integrate the split function * refactor(starknet_os): split to bigint3 Signed-off-by: Dori Medini <[email protected]> --------- Signed-off-by: Dori Medini <[email protected]>
* feat(starknet_os): integrate the split function * refactor(starknet_os): split to bigint3 Signed-off-by: Dori Medini <[email protected]> * chore(starknet_os): move write_split_result --------- Signed-off-by: Dori Medini <[email protected]>
…-libs#5288) Looks like our test runner doesn't support running integration tests that depend on the crate with `testing` feature, so we can only use integration tests that don't need `testing` on the same crate :( This is almost all a MOVE with only one required change: the bootstrap_e2e has gone stale and is broken, so i added an `ignore`. The reason is that it wasn't run in the CI since required-features doesn't add features, it just doesn't run the test if the feature isn't there :/ Will fix the test in an upcoming PR. Co-authored-by: Gilad Chase <[email protected]>
Makes it easier to debug (tooltips show the value) Co-authored-by: Gilad Chase <[email protected]>
starkware-libs#5298) * chore(deployment): update the service url in the app configs * chore(deployment): remove unnecessary storage allocation from services
) * refactor(starknet_l1_provider): use consts Makes it easier to debug (tooltips show the value) * chore(starknet_l1_provider): not related to mempool Right now we're duplicating this util, until we decide on a central super-lean crate for sharing common stuff --------- Co-authored-by: Gilad Chase <[email protected]>
starkware-libs#5291) Co-authored-by: Gilad Chase <[email protected]>
gabrielbosio
approved these changes
Mar 26, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Syncs main with the upsteam main until this commit
There are no diffs whatsoever