search

mila-iqia / blocks

A Theano framework for building and training neural networks
Other
1.16k stars 351 forks source link

VisibleDeprecationWarning now results an error #1179

Closed SwordYork closed 7 years ago

SwordYork commented 7 years ago

Hi,

After upgrading python to 3.5, the following warning results an error. I think it is better to change the type of mask or change outputs = [list(output[:mask.sum()]) (line 323) to outputs = [list(output[:int(mask.sum())]). I'm not sure which way is better.

python3.4/site-packages/blocks/search.py:324: VisibleDeprecationWarning: using a non-integer number instead of an integer will result in an error in the future for output, mask in equizip(outputs, masks)]

Thanks.

dmitriy-serdyuk commented 7 years ago

Thank you! Could you make a PR? Perhaps, we should start testing with 3.5 version.