Skip to content

update deso.ros to include coin change in transfer operations #33

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
lazynina wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

update deso.ros to include coin change in transfer operations #33

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
68 changes: 52 additions & 16 deletions rosetta-cli-conf/testnet/deso.ros
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,26 +39,46 @@ transfer(10){
max_fee_amount = "1200";
send_buffer = {{dust_amount}} + {{max_fee_amount}};

sender = {
"account_identifier": {
"address": "tBCKWCLyYKGa4Lb4buEsMGYePEWcaVAqcunvDVD4zVDcH8NoB5EgPF"
}
};
// We look for a coin of value >= the reserved_amount to create
// a transfer with change (reserved_amount is max_fee_amount + dust_amount x 2).
reserved_amount = "2400";
sender = find_balance({
"minimum_balance":{
"value": {{reserved_amount}},
"currency": {{currency}}
},
"require_coin": true
});

// Send between dust_amount and 10 DESO
// The amount we send to the recipient is a random value
// between the dust_amount and the value of the entire coin (minus
// the amount reserved for fee payment and covering the dust minimum
// of the change UTXO).
receivable_amount = {{sender.balance.value}} - {{send_buffer}};
recipient_amount = random_number({
"minimum": {{dust_amount}},
"maximum": "10000000000"
"maximum": {{receivable_amount}}
});
print_message({
"recipient_amount":{{recipient_amount}}
});

// The change amount is what we aren't sending to the recipient
// minus the maximum fee. Don't worry, we will adjust this
// amount to avoid overpaying the fee after the dry run
// completes.
raw_change_amount = {{sender.balance.value}} - {{recipient_amount}};
change_amount = {{raw_change_amount}} - {{max_fee_amount}};
print_message({
"change_amount":{{change_amount}}
});

// The last thing we need to do before creating the transaction
// is to find a recipient with a *types.AccountIdentifier that
// is not equal to the sender.
recipient = find_balance({
"not_account_identifier":[{{sender.account_identifier}}],
"not_coins":[{{sender.coin}}],
"minimum_balance":{
"value": "0",
"currency": {{currency}}
Expand All @@ -67,22 +87,28 @@ transfer(10){
"create_probability": 50
});

sender_amount = 0 - {{recipient_amount}};
sender_amount -= {{send_buffer}};
sender_amount = 0 - {{sender.balance.value}};
transfer_dry_run.confirmation_depth = "1";
transfer_dry_run.dry_run = true;
transfer_dry_run.operations = [
{
"operation_identifier":{"index":0},
"type":"INPUT",
"account":{{sender.account_identifier}},
"amount":{"value":{{recipient_amount}},"currency":{{currency}}}
"amount":{"value":{{sender_amount}},"currency":{{currency}}},
"coin_change":{"coin_action":"coin_spent", "coin_identifier":{{sender.coin}}}
},
{
"operation_identifier":{"index":1},
"type":"OUTPUT",
"account":{{recipient.account_identifier}},
"amount":{"value":{{recipient_amount}},"currency":{{currency}}}
},
{
"operation_identifier":{"index":2},
"type":"OUTPUT",
"account":{{sender.account_identifier}},
"amount":{"value":{{change_amount}},"currency":{{currency}}}
}
];
},
Expand All @@ -97,21 +123,28 @@ transfer(10){
"suggested_fee":{{suggested_fee}}
});

output_amount = {{recipient_amount}} - {{suggested_fee.value}};
change_amount = {{raw_change_amount}} - {{suggested_fee.value}};
transfer.network = {{transfer_dry_run.network}};
transfer.confirmation_depth = {{transfer_dry_run.confirmation_depth}};
transfer.operations = [
{
"operation_identifier":{"index":0},
"type":"INPUT",
"account":{{sender.account_identifier}},
"amount":{"value":{{sender_amount}},"currency":{{currency}}}
"amount":{"value":{{sender_amount}},"currency":{{currency}}},
"coin_change":{"coin_action":"coin_spent", "coin_identifier":{{sender.coin}}}
},
{
"operation_identifier":{"index":1},
"type":"OUTPUT",
"account":{{recipient.account_identifier}},
"amount":{"value":{{output_amount}},"currency":{{currency}}}
"amount":{"value":{{recipient_amount}},"currency":{{currency}}}
},
{
"operation_identifier":{"index":2},
"type":"OUTPUT",
"account":{{sender.account_identifier}},
"amount":{"value":{{change_amount}},"currency":{{currency}}}
}
];
}
Expand All @@ -130,7 +163,8 @@ return_funds(10){
"minimum_balance":{
"value": {{reserved_amount}},
"currency": {{currency}}
}
},
"require_coin": true
});

// We send the maximum amount available to the recipient. Don't worry
Expand All @@ -150,7 +184,8 @@ return_funds(10){
"operation_identifier":{"index":0},
"type":"INPUT",
"account":{{sender.account_identifier}},
"amount":{"value":{{sender_amount}},"currency":{{currency}}}
"amount":{"value":{{sender_amount}},"currency":{{currency}}},
"coin_change":{"coin_action":"coin_spent", "coin_identifier":{{sender.coin}}}
},
{
"operation_identifier":{"index":1},
Expand Down Expand Up @@ -182,7 +217,8 @@ return_funds(10){
"operation_identifier":{"index":0},
"type":"INPUT",
"account":{{sender.account_identifier}},
"amount":{"value":{{sender_amount}},"currency":{{currency}}}
"amount":{"value":{{sender_amount}},"currency":{{currency}}},
"coin_change":{"coin_action":"coin_spent", "coin_identifier":{{sender.coin}}}
},
{
"operation_identifier":{"index":1},
Expand Down