refactor(fee): remove more occurnces of resource mapping

[nimrod-starkware]

This change is 

[nimrod-starkware]

• refactor(fee): remove more occurnces of resource mapping #547 : 2 dependent PRs (#557 , #600 ) 👈
• build(fee): tx_context and create_tx_info cannot_fail #538
• refactor(fee): use valid resource bounds in blockifier #537
• refactor(fee): tx_args holds valid resource bounds #508 : 1 other dependent PR (#509 )
• refactor(fee): block context creator may fail #507
• refactor(fee): transaction info creator may fail #506
• build(fee): current tx_info holds validResourceBounds #505
• refactor(fee): remove resource mapping usage in rpc and adjust to use all resources #580
• main

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

Join @nimrod-starkware and the rest of your teammates on Graphite

[dorimedini-starkware]

Reviewed 4 of 4 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @nimrod-starkware and @ShahakShama)

---

crates/papyrus_storage/src/serialization/serializers.rs line 369 at r1 (raw file):

        pub max_amount: u64,
        pub max_price_per_unit: u128,
    }

@ShahakShama ?

[dorimedini-starkware]

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

[github-actions]

Benchmark movements:
tree_computation_flow performance regressed!
tree_computation_flow time: [66.816 ms 68.706 ms 71.124 ms]
change: [+1.2019% +3.8448% +7.9669%] (p = 0.01 < 0.05)
Performance has regressed.
Found 8 outliers among 100 measurements (8.00%)
2 (2.00%) high mild
6 (6.00%) high severe

[github-actions]

Benchmark movements:
tree_computation_flow performance improved 😺
tree_computation_flow time: [65.642 ms 65.932 ms 66.405 ms]
change: [-8.5613% -5.4521% -2.6476%] (p = 0.00 < 0.05)
Performance has improved.
Found 10 outliers among 100 measurements (10.00%)
5 (5.00%) high mild
5 (5.00%) high severe

[github-actions]

Benchmark movements:
tree_computation_flow performance improved 😺
tree_computation_flow time: [65.702 ms 65.787 ms 65.884 ms]
change: [-8.0540% -4.8219% -2.1132%] (p = 0.00 < 0.05)
Performance has improved.
Found 4 outliers among 100 measurements (4.00%)
1 (1.00%) high mild
3 (3.00%) high severe

[ShahakShama]

Reviewed 1 of 2 files at r7, all commit messages.
Reviewable status: 7 of 8 files reviewed, 1 unresolved discussion (waiting on @dorimedini-starkware and @nimrod-starkware)

mikonos

[nimrod-starkware]

Reviewed 1 of 2 files at r7.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @nimrod-starkware)

[nimrod-starkware]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @dorimedini-starkware)

---

crates/papyrus_storage/src/serialization/serializers.rs line 369 at r1 (raw file):

Previously, dorimedini-starkware wrote…

@ShahakShama ?

Done.

[nimrod-starkware]

Reviewed 2 of 4 files at r1, 1 of 3 files at r2, 1 of 2 files at r3, 1 of 1 files at r5, 1 of 1 files at r6, 1 of 2 files at r7, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @dorimedini-starkware)

[nimrod-starkware]

Dismissed @dorimedini-starkware from a discussion.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @nimrod-starkware)