Skip to content
This repository has been archived by the owner on May 13, 2022. It is now read-only.

ValueError: need more than 2 values to unpack (on tumbler.py) #538

Closed
marcoagner opened this issue May 25, 2016 · 4 comments
Closed

ValueError: need more than 2 values to unpack (on tumbler.py) #538

marcoagner opened this issue May 25, 2016 · 4 comments

Comments

@marcoagner
Copy link

marcoagner commented May 25, 2016

I couldn't find an issue with this error, so here it is:

I'm running under Fedora 23 and bitcoin-rpc as blockchain_source.
The last run of tumbler.py (called with all three addresses, so not related to another issue I saw around here) was taking about 5+ hours than estimated, I thought that was odd and finished the process. Then, I started the tumble.py with -m 2 for the mixdepth the remaining coins were in.

It started creating some TXs, all seemed fine, then I saw this error and no more transactions. It's already 3+ hours more than estimated time:

2016-05-25 19:04:10,090 [Thread-9    ] [DEBUG]  orderlist = 

2016-05-25 19:04:10,091 [Thread-9    ] [DEBUG]  ERROR not enough liquidity in the orderbook
Exception in thread Thread-9:
Traceback (most recent call last):
  File "/usr/lib64/python2.7/threading.py", line 804, in __bootstrap_inner
    self.run()
  File "/usr/lib64/python2.7/threading.py", line 1076, in run
    self.function(*self.args, **self.kwargs)
  File "/home/user/Projects/joinmarket/joinmarket/blockchaininterface.py", line 420, in bitcoincore_timeout_callback
    timeoutfun(uc_called)
  File "tumbler.py", line 125, in timeout_callback
    self.create_tx()
  File "tumbler.py", line 242, in create_tx
    self.ignored_makers)
ValueError: need more than 2 values to unpack

I let tumbler.py running for some time after the exception and got just IRC logs.
Known issue or any thoughts on what else I could investigate to make this more helpful?
Could not reproduce it again, btw.

@marcoagner marcoagner changed the title ValueError: need more than 2 values to unpack (on tumble.py) ValueError: need more than 2 values to unpack (on tumbler.py) May 25, 2016
@dreboli
Copy link

dreboli commented May 27, 2016

Same here. scrubbed log

@chris-belcher
Copy link
Collaborator

Thanks for the report

The error is in this line: https://github.com/JoinMarket-Org/joinmarket/blob/master/joinmarket/support.py#L363

It needs to return None, 0, 0
So three values not two, then it wont crash with that ValueError

@AdamISZ
Copy link
Member

AdamISZ commented May 31, 2016

It's sad really, I already fixed that bug but I can't remember where :)

chris-belcher added a commit that referenced this issue Jun 1, 2016
@chris-belcher
Copy link
Collaborator

lets see how this goes 8fa6b0a

AdamISZ pushed a commit to AdamISZ/joinmarket that referenced this issue Jun 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants