refactor(blockifier): min compiler version for sierra gas type

[AvivYossef-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[AvivYossef-starkware]

• refactor(blockifier): remove versioned runnable contract class #2811
• refactor(blockifier): use sierra version in native blockifier #2809
• refactor(blockifier): min compiler version for sierra gas type #2752 👈 (View in Graphite)
• chore(starknet_api): add sierra version utils #2828
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[github-actions]

Artifacts upload workflows:

• Started at 2024-12-18T13:52:27Z
• Started at 2024-12-18T14:03:39Z
• Started at 2024-12-18T14:58:07Z
• Started at 2024-12-19T09:01:11Z
• Started at 2024-12-19T10:17:03Z
• Started at 2024-12-19T11:38:32Z
• Started at 2024-12-19T11:39:29Z
• Started at 2024-12-19T13:21:58Z
• Started at 2024-12-19T13:49:06Z
• Started at 2024-12-19T16:06:56Z

[dorimedini-starkware]

Reviewed 27 of 27 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @AvivYossef-starkware, @noaov1, and @TzahiTaub)

---

crates/blockifier/src/execution/contract_class.rs line 312 at r1 (raw file):

    pub fn try_from_json_string(raw_contract_class: &str) -> Result<CompiledClassV1, ProgramError> {
        let casm_contract_class: CasmContractClass = serde_json::from_str(raw_contract_class)?;
        let sierra_version = SierraVersion::default();

is this ok...? where is try_from_json_string used?

Code quote:

let sierra_version = SierraVersion::default();

[github-actions]

Benchmark movements:
tree_computation_flow performance improved 😺
tree_computation_flow time: [33.852 ms 33.890 ms 33.931 ms]
change: [-5.0047% -3.5249% -2.2314%] (p = 0.00 < 0.05)
Performance has improved.
Found 12 outliers among 100 measurements (12.00%)
3 (3.00%) low severe
1 (1.00%) low mild
4 (4.00%) high mild
4 (4.00%) high severe

[dorimedini-starkware]

Reviewed 3 of 3 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @AvivYossef-starkware, @noaov1, and @TzahiTaub)

[github-actions]

Benchmark movements:
tree_computation_flow performance improved 😺
tree_computation_flow time: [33.898 ms 34.015 ms 34.196 ms]
change: [-5.0240% -3.3278% -1.8262%] (p = 0.00 < 0.05)
Performance has improved.
Found 7 outliers among 100 measurements (7.00%)
7 (7.00%) high severe

[AvivYossef-starkware]

Reviewable status: 22 of 28 files reviewed, 1 unresolved discussion (waiting on @dorimedini-starkware, @noaov1, and @TzahiTaub)

---

crates/blockifier/src/execution/contract_class.rs line 312 at r1 (raw file):

Previously, dorimedini-starkware wrote…

is this ok...? where is try_from_json_string used?

No, it's not okay. It is used here Here. I'll open another PR for it, and in the meantime, I'll use SierraVersion::deprecated to avoid activating the feature on the Python gateway.

[AvivYossef-starkware]

Should add python PR

Reviewable status: 22 of 28 files reviewed, 1 unresolved discussion (waiting on @dorimedini-starkware, @noaov1, and @TzahiTaub)

[dorimedini-starkware]

Reviewed 6 of 6 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @noaov1 and @TzahiTaub)

[dorimedini-starkware]

Reviewed 1 of 1 files at r4, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @noaov1 and @TzahiTaub)

[github-actions]

Benchmark movements:
full_committer_flow performance improved 😺
full_committer_flow time: [29.649 ms 29.691 ms 29.737 ms]
change: [-1.9807% -1.6431% -1.3511%] (p = 0.00 < 0.05)
Performance has improved.
Found 4 outliers among 100 measurements (4.00%)
4 (4.00%) high mild

[dorimedini-starkware]

Reviewed 6 of 6 files at r5, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @noaov1 and @TzahiTaub)

[dorimedini-starkware]

Reviewed 2 of 2 files at r6, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @noaov1 and @TzahiTaub)

[github-actions]

Benchmark movements:
tree_computation_flow performance improved 😺
tree_computation_flow time: [34.370 ms 34.423 ms 34.482 ms]
change: [-4.9094% -3.3945% -2.0928%] (p = 0.00 < 0.05)
Performance has improved.
Found 4 outliers among 100 measurements (4.00%)
3 (3.00%) high mild
1 (1.00%) high severe

[dorimedini-starkware]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @noaov1 and @TzahiTaub)