Fixing issue 3153.

[orizi]

This change is 

[spapinistarkware]

Reviewed all commit messages.
Reviewable status: 0 of 3 files reviewed, 1 unresolved discussion (waiting on @gilbens-starkware and @orizi)

---

crates/cairo-lang-sierra-gas/src/gas_info.rs line 72 at r1 (raw file):

            let other_val = other.function_costs.get(key);
            assert!(
                match (self_val, other_val) {

Consider adding a wrapper type for the costs, and implement ParitalEq using sub_maps

[orizi]

Reviewable status: 0 of 3 files reviewed, 1 unresolved discussion (waiting on @gilbens-starkware and @spapinistarkware)

---

crates/cairo-lang-sierra-gas/src/gas_info.rs line 72 at r1 (raw file):

Previously, spapinistarkware (Shahar Papini) wrote…

Consider adding a wrapper type for the costs, and implement ParitalEq using sub_maps

this is spread all around, so would be a lot more changes - and i'm not sure this is actually required,
as this check fails just because it compares 2 different methods to calculate the same thing.

[spapinistarkware]

Reviewed 3 of 3 files at r1.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @gilbens-starkware)