Add simulation support #931
Conversation
webmaster128
force-pushed
the
add-simulate
branch
2 times, most recently
from
4ded93f
to
4d61793
Compare
| }), | ||
| }; | ||
| const memo = "Go go go"; | ||
| const gasUsed = await client.simulate(alice.address0, [executeContractMsg], memo); |
Yeah, this is the next question. I think the flow is very inconvenient. I'm thinking about an "auto" option that does the simulation internally and use a constant multiplier but still allow a custom gas limit. I'll write an example that shows how the current diff can be used. |
webmaster128
force-pushed
the
add-simulate
branch
from
10f45c9
to
27a4f1b
Compare
|
@ethanfrey could you have a look at |
webmaster128
force-pushed
the
add-simulate
branch
from
27a4f1b
to
49278d1
Compare
| const memo = "With simulate"; | ||
| const gasEstimation = await client.simulate(account.address, [sendMsg], memo); | ||
| const fee = calculateFee(Math.round(gasEstimation * 1.3), gasPrice); | ||
| const result = await client.sendTokens(account.address, recipient, amount, fee, memo); |
There was a problem hiding this comment.
Wow, this next to last argument can be:
- "auto"
- number -> multiplier after simulation
StdFee-> caller fixed gas/fee
The top two using a gas price set in the constructor.
Is that correct?
| @@ -367,7 +391,7 @@ export class SigningCosmWasmClient extends CosmWasmClient { | |||
| senderAddress: string, | |||
| recipientAddress: string, | |||
| amount: readonly Coin[], | |||
| fee: StdFee, | |||
| fee: StdFee | "auto" | number, | |||
There was a problem hiding this comment.
Okay, nice arg here... very flexible and I hope the caller uses TS
webmaster128
force-pushed
the
add-simulate
branch
from
49278d1
to
b4aa877
Compare
Closes #828