Skip to content

feat(blockifier): add data gas accounts to VC #5154

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

Merged
merged 1 commit into from
Mar 26, 2025

Conversation

nimrod-starkware
Copy link
Contributor

No description provided.

@nimrod-starkware nimrod-starkware self-assigned this Mar 23, 2025
@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Contributor Author

@nimrod-starkware nimrod-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Python side:
https://reviewable.io/reviews/starkware-industries/starkware/37403

Reviewable status: 0 of 11 files reviewed, all discussions resolved (waiting on @nimrod-starkware)

@nimrod-starkware nimrod-starkware force-pushed the nimrod/data_gas_accounts/add_to_vc branch 2 times, most recently from c7e36d1 to 762f180 Compare March 23, 2025 14:20
Copy link
Contributor Author

nimrod-starkware commented Mar 23, 2025

@nimrod-starkware nimrod-starkware force-pushed the nimrod/data_gas_accounts/add_to_vc branch 2 times, most recently from be513b8 to d8b6bb5 Compare March 25, 2025 07:51
Copy link
Collaborator

@Yoni-Starkware Yoni-Starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 10 of 11 files at r2, all commit messages.
Reviewable status: 10 of 11 files reviewed, 1 unresolved discussion (waiting on @nimrod-starkware)


crates/blockifier/resources/blockifier_versioned_constants_0_13_5.json line 146 at r2 (raw file):

            "0x00816dd0297efc55dc1e7559020a3a825e81ef734b558f03c83325d4da7e6253",
            "0x041bf1e71792aecb9df3e9d04e1540091c5e13122a731e02bec588f71dc1a5c3"
        ]

Should be empty here

Code quote:

        "data_gas_accounts": [
            "0x02c8c7e6fbcfb3e8e15a46648e8914c6aa1fc506fc1e7fb3d1e19630716174bc",
            "0x00816dd0297efc55dc1e7559020a3a825e81ef734b558f03c83325d4da7e6253",
            "0x041bf1e71792aecb9df3e9d04e1540091c5e13122a731e02bec588f71dc1a5c3"
        ]

@nimrod-starkware nimrod-starkware force-pushed the nimrod/data_gas_accounts/add_to_vc branch from d8b6bb5 to 565bf41 Compare March 25, 2025 11:18
Copy link
Contributor Author

@nimrod-starkware nimrod-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: 1 of 11 files reviewed, 1 unresolved discussion (waiting on @nimrod-starkware and @Yoni-Starkware)


crates/blockifier/resources/blockifier_versioned_constants_0_13_5.json line 146 at r2 (raw file):

Previously, Yoni-Starkware (Yoni) wrote…

Should be empty here

Done.

@nimrod-starkware nimrod-starkware force-pushed the nimrod/data_gas_accounts/add_to_vc branch from b9e6c91 to 6396d4c Compare March 25, 2025 15:31
Copy link
Collaborator

@Yoni-Starkware Yoni-Starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 10 of 10 files at r3, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @nimrod-starkware)

@nimrod-starkware nimrod-starkware added this pull request to the merge queue Mar 26, 2025
Merged via the queue into main with commit 00c4932 Mar 26, 2025
15 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants