InjectiveLabs / sdk-python

Injective Exchange API for Python clients
Apache License 2.0
41 stars 27 forks source link

Feat/message broadcaster without simulation #226

Closed aarmoa closed 1 year ago

aarmoa commented 1 year ago
achilleas-kal commented 1 year ago

Effectively the calculation should include:

  1. Default_gas_limit = gas for the tx itself since fields such as memo and timeout_height cost gas.
  2. Gas per object
  3. You can consider a "buffer" variable than can also be modifiable be the clients so we can have more strict gas per object and have a buffer to account for different ops in IAVL.
aarmoa commented 1 year ago

@achilleas-kal let me see what I can do for point 1. Regarding point 2, since those messages are still present in the SDK and I have already implemented the code to estimate their gas cost, I don't see the need to remove them now. We can remove them when the messages are no longer present in the SDK.