Skip to content
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.

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

ticketbuyer.balancetomaintainabsolute not respected #2055

Open
chappjc opened this issue May 25, 2021 · 11 comments
Open

ticketbuyer.balancetomaintainabsolute not respected #2055

chappjc opened this issue May 25, 2021 · 11 comments

Comments

@chappjc
Copy link
Member

chappjc commented May 25, 2021

The ticketbuyer.balancetomaintainabsolute setting does not seem to be respected.

My balance is routinely drawn down below this limit when purchasing tickets.

@amass01
Copy link
Member

amass01 commented Sep 30, 2022

@jrick may i investigate and try fix this one ?

@jrick
Copy link
Member

jrick commented Sep 30, 2022

doesn't need much investigation because we understand why this happens -- as utxos are selected, the confirmed balance might dip below this configuration value, since the change will be unconfirmed.

I'm not sure if this is anything we need to fix. Talking about balances (as this config option does) without also talking about the number of confirmations is meaningless.

@chappjc
Copy link
Member Author

chappjc commented Sep 30, 2022

In my experience (at the time) if you let the wallet run indefinitely, you will get to a point where there is no (or very little) unconfirmed, yet you've fully depleted your balance on tickets.

@amass01
Copy link
Member

amass01 commented Sep 30, 2022

Aha, I see

@jrick
Copy link
Member

jrick commented Sep 30, 2022

@chappjc this is because the unconfirmed balance is going to a different account, e.g. unmixed. This option predates the mixing support and this use case.

@chappjc
Copy link
Member Author

chappjc commented Sep 30, 2022

That's not my recollection. I suggest running testnet wallets with and without mixing, with a non zero balancetomaintainabsolute. I think in both case it's not just unconfirmed or unmixed change. Maybe I'm wrong and this can be closed, but the wallet would become way over bought into tickets, where either unconfirmed or unmixed account balances don't even come close to the requested setting.

@chappjc
Copy link
Member Author

chappjc commented Sep 30, 2022

My testnet ticketbuyer is presently not mixing. Will set it and try now.

@jrick
Copy link
Member

jrick commented Sep 30, 2022

thanks. let's find out if the option works as i'm recalling, otherwise we should discuss how we'd prefer it to work. if it is "working" then at the very least we need to clarify what the option really does because as is the description is extremely vague.

@chappjc

This comment was marked as resolved.

@chappjc
Copy link
Member Author

chappjc commented Sep 30, 2022

Oh dang, ticketbuyer.limit is the wrong setting. Where did that come from? 😠

@chappjc
Copy link
Member Author

chappjc commented Sep 30, 2022

Ok, sorry for the noise, this was just a screw up on my part it seems. With --ticketbuyer.balancetomaintainabsolute=200 and no mixing it's working as expected.

There may be a warranted clarification when mixing is in use, but it seem PEBKAC.

   --ticketbuyer.balancetomaintainabsolute= Amount of funds to keep in wallet when purchasing tickets (default: 0 DCR)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants