Ge Gao

Results 37 comments of Ge Gao

LG, thanks for the fix!

@bwann52 this is the change of the assets yet, changed that in https://github.com/MystenLabs/sui/pull/3226 and tested locally there as well

@sblackshear I saw that eth_call does not cost gas, and presumably sui_call will be executed on fullnodes without any gas cost either, is it a concern that sui_call would be...

@leofisG > fetch all of the user objects this looks like just data fetch, if so can https://docs.sui.io/sui-jsonrpc#sui_getObjectsOwnedByAddress work instead? also is this info used for txn construction (otherwise maybe...

@leofisG we chatted briefly in an internal thread and the [code pointer](https://github.com/hippospace/tutorial-lending/blob/be2cd038b2cd0f6bbfd87a5e105890fa8624539d/sources/lending.move#L362) will very likely hit the txn execution limit, maybe also the limit of Sui Move object. I am...

@PaulFidika we have https://docs.sui.io/sui-jsonrpc#sui_dryRunTransaction for txn simulation already. another endpoint that I am adding is dryRunMoveCall, https://github.com/MystenLabs/sui/issues/4967

@PaulFidika with dryRunTransaction, ppl can simulate txn runs, which can be move Move function or native function like TransferSui, but it needs to be a signed txn, and requires a...

@5kbpers Thanks for flagging, about the payment issue, there is a pending PR here https://github.com/MystenLabs/sui/pull/4452 that will solve this problem by wrapping all merging, splitting and transfer into one txn...

@lxfind thanks a lot for the review! Agreed that I should add integration tests covering both the execution itself and also the gas deduction logic, doing it now! > Another...

per discussion with @lxfind and @longbowlu, I made changes in the latest commit: - simplify the pay_sui "debit coin and transfer" logic to start with total coins - separate the...