Update gas for storage occupation
Describe your changes
Increases the gas cost component for storage occupation and ties this cost the other ones so that it's not free-floating anymore (which could cause a divergence if the other costs are changed).
Removes the redundant new_from_sub_limit constructor for TxGasMeter.
Indicate on which release or other PRs this topic is based on
#3428 (diff for review: https://github.com/anoma/namada/pull/3510/files/209da0c6ad9cd456a5a6634ec25cc45408e0b72c..33a6cc9437692ccb1ef24e58f2536ac2dfdeb8f3)
Checklist before merging to draft
- [x] I have added a changelog
- [x] Git history is in acceptable state
Let's wait for the CI to come back online before reviewing this cause there's a change we might have a few tests failing due to gas
Codecov Report
Attention: Patch coverage is 78.92377% with 47 lines in your changes missing coverage. Please review.
Project coverage is 53.46%. Comparing base (
8479d38) to head (33a6cc9). Report is 4 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #3510 +/- ##
==========================================
- Coverage 53.48% 53.46% -0.02%
==========================================
Files 320 320
Lines 110000 109964 -36
==========================================
- Hits 58832 58792 -40
- Misses 51168 51172 +4
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Let's wait for the CI to come back online before reviewing this cause there's a change we might have a few tests failing due to gas
Ok I've fixed the broken tests, this PR is now ready for review