Nltk: TextTiling not support python 3

Created on 16 Oct 2017 · 4Comments · Source: nltk/nltk

In line 315 clip = min(max(len(scores)/10, 2), 5) you should wrap this value with a int because python 3 division type is float.

Thanks

bug corpus goodfirstbug python3

Source

talbaumel

Most helpful comment

https://github.com/nltk/nltk/pull/1863
This bug was fixed :)

e5pecial on 23 Oct 2017

👍2

All 4 comments

Related #965 #1537 and https://github.com/nltk/nltk/issues?utf8=✓&q=is%3Aissue%20is%3Aopen%20texttiling

I think this is a long standing bug that hasn't been fixed and the fix could be pretty simple. Let's try to patch this by the next release ;P

alvations on 16 Oct 2017

I would like to work on this :)

zorroblue on 23 Oct 2017

https://github.com/nltk/nltk/pull/1863
This bug was fixed :)

e5pecial on 23 Oct 2017

👍2

@e5pecial Thanks for the note!

@zorroblue Thank you for your interest, if you're up to contribute on something else, you can try browsing the goodfirstbug tag: https://github.com/nltk/nltk/labels/goodfirstbug . Or if you're interested to work on the related issues, there's #965 and #1537.

Resolved in #1863

alvations on 23 Oct 2017

Was this page helpful?

0 / 5 - 0 ratings