Implement basic gas metering

informalsystems / basecoin-rs

An example ABCI application making use of tendermint-rs and ibc-rs

Apache License 2.0

54 stars 17 forks source link

Implement basic gas metering #30

Open hu55a1n1 opened 2 years ago

hu55a1n1 commented 2 years ago

[ ] Define a new wrapper store type MeteredStore<S> that intercepts store ops and calculates gas costs.
[ ] Copy Cosmos SDK's meter values -> https://github.com/cosmos/cosmos-sdk/blob/master/store/types/gas.go#L219

hxrts commented 2 years ago

should be aware of this issue in the sdk if you haven't come across it yet https://github.com/cosmos/cosmos-sdk/issues/4938

hu55a1n1 commented 2 years ago

Thanks for sharing the issue @hxrts! 🙏 I wasn't aware of it. Just started looking at it, will make sure to keep this in mind before I start implementing gas estimation in basecoin-rs.