Issue 721 in nltk: Tinyyy patch with greaaat effect.

0 views
Skip to first unread message

nl...@googlecode.com

unread,
Oct 5, 2011, 7:17:38 PM10/5/11
to nltk-...@googlegroups.com
Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 721 by khame...@gmail.com: Tinyyy patch with greaaat effect.
http://code.google.com/p/nltk/issues/detail?id=721

I've been able to compile amazing NLTK under brilliant PyPy with a small
patch described here:
https://groups.google.com/group/nltk-dev/browse_thread/thread/7dd4048beacb832a

"But if you just change line 282 in .../nltk/sourcedstring.py into:"
class SourcedString(object):

And it all worked just fine.

With this patch you can try the delicious NLTK with awesome speed of PyPy :)

See also:
https://bugs.pypy.org/issue846


nl...@googlecode.com

unread,
Oct 5, 2011, 7:34:05 PM10/5/11
to nltk-...@googlegroups.com
Updates:
Status: Accepted
Owner: StevenBird1

Comment #1 on issue 721 by StevenBird1: Tinyyy patch with greaaat effect.
http://code.google.com/p/nltk/issues/detail?id=721

Thanks for the patch.


nl...@googlecode.com

unread,
Oct 5, 2011, 7:38:11 PM10/5/11
to nltk-...@googlegroups.com

Comment #2 on issue 721 by khame...@gmail.com: Tinyyy patch with greaaat
effect.
http://code.google.com/p/nltk/issues/detail?id=721

thanks, Steven!

nl...@googlecode.com

unread,
Oct 10, 2011, 11:09:02 PM10/10/11
to nltk-...@googlegroups.com
Updates:
Status: Fixed

Comment #3 on issue 721 by StevenBird1: Tinyyy patch with greaaat effect.
http://code.google.com/p/nltk/issues/detail?id=721

Adopted patch.


nl...@googlecode.com

unread,
Oct 11, 2011, 6:17:09 AM10/11/11
to nltk-...@googlegroups.com

Comment #4 on issue 721 by khame...@gmail.com: Tinyyy patch with greaaat
effect.
http://code.google.com/p/nltk/issues/detail?id=721

thanks!

nl...@googlecode.com

unread,
Apr 22, 2013, 10:25:13 AM4/22/13
to nltk-...@googlegroups.com

Comment #5 on issue 721 by zoo...@gmail.com: Tinyyy patch with greaaat
effect.
http://code.google.com/p/nltk/issues/detail?id=721

StevenBird1 said "Adopted patch" in Oct 10, 2011, but was the patch
actually merged? The same problem seems to be present in ntlk 2.0.1rc1.

--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
Reply all
Reply to author
Forward
0 new messages