mzucker / page_dewarp

Text page dewarping using a "cubic sheet" model
https://mzucker.github.io/2016/08/15/page-dewarping.html
MIT License
1.44k stars 241 forks source link

Error: Need more than 2 values to unpack #16

Open stevethearkiv opened 4 years ago

stevethearkiv commented 4 years ago

Sorry I cannot get this to work, and there is nothing in the readme about additional values?

Cheers for any help you can give me

loaded boston_cooking_a.jpg with size 2448x3264 and resized to 490x653 Traceback (most recent call last): File "../page_dewarp.py", line 915, in main() File "../page_dewarp.py", line 867, in main cinfo_list = get_contours(name, small, pagemask, 'text') File "../page_dewarp.py", line 450, in get_contours cv2.CHAIN_APPROX_NONE) ValueError: need more than 2 values to unpack

Andry-Bal commented 4 years ago

In earlier versions of openCV function findContours() used to return 3 values (image, contours, hierarchy). In openCV 4 it returns two values (contours, hierarchy) and this is what causing the error. Just use openCV 3.

bjornmosten commented 4 years ago

You can also just remove the first _ on line 450, making the line

contours, _ = cv2.findContours(mask, cv2.RETR_EXTERNAL, cv2.CHAIN_APPROX_NONE)