Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RecursionError on segment call #33

Open
irmo322 opened this issue May 31, 2021 · 6 comments
Open

RecursionError on segment call #33

irmo322 opened this issue May 31, 2021 · 6 comments

Comments

@irmo322
Copy link

irmo322 commented May 31, 2021

Hi,

I'm having trouble with following code:

import wordsegment

wordsegment.load()
text = "The article went on to say, “For in the pizza shops rich and poor harmoniously congregate; they are the only places where the members of Neapolitan aristocracy—far haughtier than those of any other part of Italy—may be seen (eating) their favorite delicacy side by side with their own coachmen and valets and barbers.”"
wordsegment.segment(text)

It fails with a RecursionError.
RecursionError: maximum recursion depth exceeded while calling a Python object

I'm using python 3.8 on ubuntu 20.04.

@grantjenks
Copy link
Owner

Can you share more of the trace back?

@irmo322
Copy link
Author

irmo322 commented May 31, 2021

Trace back error : error.txt

@grantjenks
Copy link
Owner

Works for me:

$ ipython
Python 3.9.1 (v3.9.1:1e5d33e9b9, Dec  7 2020, 12:10:52) 
Type 'copyright', 'credits' or 'license' for more information
IPython 7.24.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: import wordsegment

In [2]: wordsegment.load()

In [3]: text = "The article went on to say, “For in the pizza shops rich and poor harmoniously congregate; they are the only places where 
   ...: the members of Neapolitan aristocracy—far haughtier than those of any other part of Italy—may be seen (eating) their favorite deli
   ...: cacy side by side with their own coachmen and valets and barbers.”"
   ...: 

In [4]: wordsegment.segment(text)
Out[4]: 
['the',
 'article',
 'went',
 'on',
 'to',
 'say',
 'for',
 'in',
 'the',
 'pizza',
 'shops',
 'rich',
 'and',
 'poor',
 'harmoniously',
 'congregate',
 'they',
 'are',
 'the',
 'only',
 'places',
 'where',
 'the',
 'members',
 'of',
 'neapolitan',
 'aristocracy',
 'far',
 'haugh',
 'tier',
 'than',
 'those',
 'of',
 'any',
 'other',
 'part',
 'of',
 'italy',
 'may',
 'be',
 'seen',
 'eating',
 'their',
 'favorite',
 'delicacy',
 'side',
 'by',
 'side',
 'with',
 'their',
 'own',
 'coachmen',
 'and',
 'valets',
 'and',
 'barbers']

@grantjenks
Copy link
Owner

Weird, it doesn't work for me in Python 3.8.

@grantjenks
Copy link
Owner

See the PR. If I set the CHUNK_SIZE to 200 then it works for me in 3.8

@irmo322
Copy link
Author

irmo322 commented Jun 1, 2021

Thank you for the support :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants